1551711 - Add alt text property to snippet schema

Status: VERIFIED FIXED

(Firefox :: Messaging System, enhancement, P1)

Description

[Emily McMinn :emcminn]

Add optional alt text to snippet icons for accessibility purposes. See if this is reasonable for the Snippets team.

Comment 1

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/activity-stream/pull/5102

Comment 2

[Ed Lee :Mardak]

https://hg.mozilla.org/mozilla-central/rev/0dff9f803849

Comment 3

[Marius Coman [:mcoman], Ecosystem QA]

Hi Emily, are these server-side changes or we can see them in the browser?

Comment 4

[Emily McMinn :emcminn]

(In reply to Marius Coman [:mcoman], Experiments QA from comment #3)

Hi Emily, are these server-side changes or we can see them in the browser?

It's a front-end change that sets us up to accept the prop from the server once it's available - right now the front end will be using the fallback string because the data isn't present. (The fallback is just an empty string, to flag images/icons as decorative for screen readers.) It's also an optional prop, so there's no need to go back and fill it in; it's just there to be used going forward :)

Comment 5

[Marius Coman [:mcoman], Ecosystem QA]

I have verified this issue with the latest Firefox Nightly (69.0a1 Build ID - 20190619214046) installed, on Windows 10 x64, Arch Linux and Mac 10.14.5. Now, "Alt" text is set to an empty string as specified in comment 4.