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)

ARM
Android
defect

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. :)
Sure. Android complains if there's no string, so I put in a string.
Component: General → Android Sync
OS: Linux → Android
Product: Firefox for Android → Android Background Services
Hardware: x86 → ARM
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.
(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.
OK, thanks! :)
Assignee: nobody → nalexander
Blocks: 972888
Priority: -- → P1
Attached file github PR (deleted) —
Not really feeling it, so here are some small tweaks.
Attachment #8385681 - Flags: review?(rnewman)
Attachment #8385681 - Flags: review?(rnewman) → review+
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
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.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Product: Android Background Services → Firefox for Android
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: