Closed
Bug 974627
Opened 11 years ago
Closed 11 years ago
Decorative images in Firefox accounts page don't need assistive text
Categories
(Firefox for Android Graveyard :: Android Sync, defect, P1)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: rimas, Assigned: nalexander)
References
()
Details
Attachments
(1 file)
Bug 965376 added the following strings to sync_strings.dtd a couple weeks ago:
<!-- Localization note: these describe icons and graphics, and are
consumed by assistive devices. -->
<!ENTITY fxaccount_checkbox_contentDescription2 'Firefox accounts checkbox graphic'>
<!ENTITY fxaccount_icon_contentDescription2 'Firefox accounts icon'>
<!ENTITY fxaccount_intro_contentDescription2 'Firefox accounts introduction graphics'>
<!ENTITY fxaccount_mail_contentDescription2 'Firefox accounts envelope graphic'>
From the comment, it seems like these strings are meant to be used as alt-text for purely decorative images. Which is a bad idea. Not only is such text not helpful to visually-impaired people, it is actually counter-productive and will only get in the way.
Please remove these strings from localization and use empty alt attributes instead. These images convey absolutely no meaning and are there for decoration purposes only.
See http://dev.w3.org/html5/alt-techniques/#sec5 if I don't sound credible enough. :)
Assignee | ||
Comment 1•11 years ago
|
||
Sure. Android complains if there's no string, so I put in a string.
Updated•11 years ago
|
Component: General → Android Sync
OS: Linux → Android
Product: Firefox for Android → Android Background Services
Hardware: x86 → ARM
Reporter | ||
Comment 2•11 years ago
|
||
Nick, even if a string is required, can't you use an empty string there, or a no-break space? You wouldn't have to make that localizable, so you'd save time for localizers as well users.
Assignee | ||
Comment 3•11 years ago
|
||
(In reply to Rimas Kudelis from comment #2)
> Nick, even if a string is required, can't you use an empty string there, or
> a no-break space? You wouldn't have to make that localizable, so you'd save
> time for localizers as well users.
Sorry, I should have been more explicit: yes, I will use an empty string for all these content descriptions.
Reporter | ||
Comment 4•11 years ago
|
||
OK, thanks! :)
Assignee | ||
Updated•11 years ago
|
Assignee | ||
Comment 5•11 years ago
|
||
Not really feeling it, so here are some small tweaks.
Attachment #8385681 -
Flags: review?(rnewman)
Updated•11 years ago
|
Attachment #8385681 -
Flags: review?(rnewman) → review+
Assignee | ||
Comment 6•11 years ago
|
||
Status: NEW → ASSIGNED
Comment 7•11 years ago
|
||
hi nick, sorry had to backout for the suspicion this caused test failures like https://tbpl.mozilla.org/php/getParsedLog.php?id=35928754&tree=Fx-Team
Assignee | ||
Comment 8•11 years ago
|
||
Reland:
https://hg.mozilla.org/integration/fx-team/rev/1a14c3cdfa7e
Let's see if the fixes pushed in Bug 979388 (or blind luck!) are sufficient to avoid the earlier issues.
Comment 9•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Product: Android Background Services → Firefox for Android
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•