"Align" attribute is deprecated since HTML 4.02. I found some bugs caused by the use of this property in IE7. As default Drupal themes (and the mostly used themes, I think) are XHTML strict, and as IFA already uses classes with css "float" properties, I think that this attribute can be safetly removed.

CommentFileSizeAuthor
#1 ifa_remove_align.patch869 bytesdanillonunes
ifa_img_no_align_attribute.patch713 bytesdanillonunes

Comments

danillonunes’s picture

danillonunes commented
StatusFileSize
new ifa_remove_align.patch869 bytes

Here is the patch for the most recent release of IFA.

franz’s picture

franz
any
Location Montréal
commented

Fixed on dev.
Thanks

franz’s picture

franz
any
Location Montréal
commented
Status: Active » Fixed
lourenzo’s picture

lourenzo commented
Status: Fixed » Needs review

When using tinyMCE, the image tag still have this attribute.
Although, the tinyMCE document leaves in an iframe, and uses another DOCTYPE,

Can this difference between wysiwyg cause formatting result issues?

danillonunes’s picture

danillonunes commented

tinyMCE uses this attribute only in it's interface or it's saved at the end result?

I found a issue only in IE7 at a specific situation, when a image has this attribute AND a float declaration.

lourenzo’s picture

lourenzo commented
Assigned: Unassigned » lourenzo
Status: Needs review » Closed (cannot reproduce)

Couldn't reproduce on recent releases.