Thank you for this very useful module.

I didn't find a way to select or an option to disable target="" in the Only local images are allowed. tag

I find this can be necessary at times. Could this be added to the next release?

Thanks,

Comments

soundsational’s picture

soundsational commented

Just to add: it's because it doesn't validate for Strict xHTML doctype

johnfyoung’s picture

johnfyoung commented

Sounds reasonable. Will add it.

johnfyoung’s picture

johnfyoung commented
Status: Active » Fixed
soundsational’s picture

soundsational commented

Thanks it works great.

For others that are wondering where this new setting is. Go to your content type manage field (where you added a field for this module). There is a new settings called 'Enable custom target' tick the checkbox next to it. Now when you add a new image, under 'target:', you'll have a new option in the drop down called 'Use Default (_self)'. This will remove the target element from the < a > tag if you do not wish to have it there.

Thanks johnfyoung for including this option.

P.S. Do you think the 'Use Default (_self)' should be renamed to 'Do not show'? It's just that now we have two _self and it can sound a little confusing?

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.