Closed Bug 974627 Opened 6 years ago Closed 6 years ago

Decorative images in Firefox accounts page don't need assistive text

Categories

(Firefox for Android :: Android Sync, defect, P1)

ARM
Android
defect

Tracking

()

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
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.
https://hg.mozilla.org/mozilla-central/rev/1a14c3cdfa7e
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Product: Android Background Services → Firefox for Android
You need to log in before you can comment on or make changes to this bug.