For example my test page "admin/panels/panel-page/Test/edit/content" have 5 panel images without an ALT tag specified. This causes the XHTML validator to complain.

CommentFileSizeAuthor
#3 panels-DRUPAL-5--2.admin-image-alt-t.patch3.45 KBsun
#2 panels_D5_2x_translation_and_xhtml_bugs.patch3.66 KBhass

Comments

sdboyer’s picture

sdboyer commented

Tuning the xhtml output really isn't merlin's or my area of expertise, so as with #277396: Content is jumping / scrollbars flickering on/off - patch?

hass’s picture

hass commented
Title: XHTML: Images don't have ALT tags » XHTML: Images don't have ALT tags / wrong use if t()
Status: Active » Needs review
StatusFileSize
new panels_D5_2x_translation_and_xhtml_bugs.patch3.66 KB

While I fixed this bug, I found there is a major bug about translatability that is also included in this patch.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new panels-DRUPAL-5--2.admin-image-alt-t.patch3.45 KB

Replaced custom image tags with theme('image') functions. Aside from that, this is ready to go.

hass’s picture

hass commented

This looks more better :-)

sdboyer’s picture

sdboyer commented
Status: Reviewed & tested by the community » Fixed

Great, thanks for getting this up so quickly. Committed.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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