Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consider adding an accessibility disclaimer to the Baseline badge #10350

Closed
3 tasks done
captainbrosset opened this issue Jan 19, 2024 · 5 comments · Fixed by mdn/content#33515
Closed
3 tasks done

Consider adding an accessibility disclaimer to the Baseline badge #10350

captainbrosset opened this issue Jan 19, 2024 · 5 comments · Fixed by mdn/content#33515
Labels
involves: Content Requires the attention of the Content team.

Comments

@captainbrosset
Copy link
Contributor

Summary

The context for this issue comes from web-platform-dx/web-features#498.

Sorry, this is a long issue to read, but I think the gist of it can be understood by reading these comments:

  1. Is accessibility support included? web-platform-dx/web-features#498 (comment)
  2. Is accessibility support included? web-platform-dx/web-features#498 (comment)
  3. Is accessibility support included? web-platform-dx/web-features#498 (comment)
  4. Is accessibility support included? web-platform-dx/web-features#498 (comment)

In short, the web-features repo does not currently have a clear guideline around accessibility that's used to decide the Baseline status of a feature. We're discussing ways to change this, but it was suggested in the linked issue above that, in the meantime, it would be good for MDN to add a disclaimer to the Baseline badge.

See web-platform-dx/web-features#498 (comment) for an example of the proposal. Once expanded, the badge could have something like

 There is no guarantee this feature is
 <a href="https://www.w3.org/TR/WCAG22/#dfn-accessibility-supported">
  accessibility supported
 </a>, however, so support testing is recommended.

Because the UI rendering of the Baseline data is owned by MDN, I'm opening this issue for the MDN team to discuss and consider if they want to change the UI and add a note about accessibility.

URL

N/A

Reproduction steps

N/A

Expected behavior

Accessibility should either be included in the baseline status for the feature, or a disclaimer that it's not should be present in the badge.

Actual behavior

N/A

Device

Desktop

Browser

Chrome

Browser version

Stable

Operating system

Android

Screenshot

No response

Anything else?

No response

Validations

@github-actions github-actions bot added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Jan 19, 2024
@argl argl added involves: Content Requires the attention of the Content team. and removed needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Jan 30, 2024
@LeoMcA
Copy link
Member

LeoMcA commented Feb 14, 2024

I don't think this is a good idea, for a number of reasons:

First, I reckon it'll fail to achieve what it sets out to do: people likely won't see this disclaimer, currently very few people actually open the Baseline banner: our metrics show between 2-3% of pageviews with a Baseline banner shown also have that Baseline banner toggled open.

Second, a11y support isn't the only non-goal of Baseline, do we add disclaimers for all those others in the banner as well? This will get rather overwhelming rather quickly, and fail on the aim of Baseline to be easily digestible.

Indeed, Baseline doesn't override general web development best practices: you should still test your code, regardless of what the compatibility data suggests. Perhaps some kind of "how to use Baseline" document would be a better place for a disclaimer of this type, along with the other non-goals.

@foolip
Copy link
Contributor

foolip commented Feb 15, 2024

Accessibility isn't a non-goal of Baseline, the delineation that was made in web-platform-dx/web-features#519 is that if the browser does the wrong thing we consider that a user-affecting bug like any other, and it might lead to the feature not being considered ready. It's bugs outside of the browser, in the ATs, that we're not tracking.

@ddbeck
Copy link
Contributor

ddbeck commented Feb 15, 2024

Indeed, Baseline doesn't override general web development best practices: you should still test your code, regardless of what the compatibility data suggests. Perhaps some kind of "how to use Baseline" document would be a better place for a disclaimer of this type, along with the other non-goals.

I would like to echo this sentiment. Perhaps the Baseline glossary page is the right place to cover some of this stuff. That Extra considerations section could use some attention, along the lines of: don't forget to do accessibility, usability, performance, and other testing, or you'll be sorry whether or not there was a blue or green mark.

@github-actions github-actions bot added the idle label Mar 16, 2024
@caugner
Copy link
Contributor

caugner commented Apr 19, 2024

I would like to echo this sentiment. Perhaps the Baseline glossary page is the right place to cover some of this stuff. That Extra considerations section could use some attention, along the lines of: don't forget to do accessibility, usability, performance, and other testing, or you'll be sorry whether or not there was a blue or green mark.

We discussed this internally and support Daniel's proposal to reflect this on the Baseline glossary page. 👍

@ddbeck
Copy link
Contributor

ddbeck commented May 9, 2024

I opened a PR for this in content: mdn/content#33515

bsmth added a commit to mdn/content that referenced this issue May 13, 2024
…ns (#33515)

* Baseline glossary entry: mention accessibility and other considerations

Fixes mdn/yari#10350

* Apply copy editing suggestions

Co-authored-by: Brian Thomas Smith <brian@smith.berlin>

---------

Co-authored-by: Brian Thomas Smith <brian@smith.berlin>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
involves: Content Requires the attention of the Content team.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants