As per: http://lists.drupal.org/pipermail/development/2008-March/029100.html

Attached patch attempts to clarify the image descriptions.

Untested.

CommentFileSizeAuthor
#3 230763_download_table_text.patch5.16 KBdww
project-download-images.patch2.04 KBwebernet

Comments

dww’s picture

dww
we/he/they
commented
Assigned: Unassigned » dww
Status: Needs review » Active

Thanks for starting this. I blame the crappy network at DrupalCon for the duplicate posting. ;)

Anyway, this won't work at all, since it doesn't touch the CSS so you'd lose all the colorized styling stuff.

Furthermore, I've got a bigger patch in the works that a) provides more descriptive message for the title/alt, and b) is using a theme function to make it easy for sites to override this without having to go all the way to locale.module. So, setting this back to active for now, and I'll set it back to CNR when I get a chance to post what I've got.

Cheers,
-Derek

webernet’s picture

webernet commented

Just a quick note -- the red error on the -dev releases has already caused some concerns that there was something wrong with the releases in #drupal -- perhaps something other than the 'error' class would be appropriate...

dww’s picture

dww
we/he/they
commented
Status: Active » Needs review
StatusFileSize
new 230763_download_table_text.patch5.16 KB

This splits up the text for both the table and the alt/title tags on the icon to be theme functions, so that it's easy to alter all of this per-site.

dww’s picture

dww
we/he/they
commented
Status: Needs review » Fixed

Fixed some minor string changes based on feedback from aclight and hunmonk. Committed to HEAD and deployed on d.o.

Anonymous’s picture

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

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