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

Remove semantics from the cocoon dashboard. #3706

Merged
merged 1 commit into from
May 6, 2024

Conversation

matanlurey
Copy link
Contributor

Closes flutter/flutter#145849.

This was originally added by Casey so he could use some Chrome extension to navigate the dashboard (#1729), but it appears to cause serious problems for mouse-users.

I've filed flutter/flutter#147881 for the web team to take a look.

Copy link
Member

@gaaclarke gaaclarke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@stuartmorgan
Copy link
Contributor

What are the accessibility implications of this? Will it break screenreaders, for instance?

@gaaclarke
Copy link
Member

What are the accessibility implications of this? Will it break screenreaders, for instance?

In an internal chat Jonah noted that it doesn't work with screenreaders today anyways.

@stuartmorgan
Copy link
Contributor

Okay, in that case I'm fine with trying this for now then, and gathering actual feedback about what (if anything) regresses that people are using.

@matanlurey matanlurey added the autosubmit Merge PR when tree becomes green via auto submit App. label May 6, 2024
@auto-submit auto-submit bot merged commit fb212e4 into flutter:main May 6, 2024
3 checks passed
@matanlurey matanlurey deleted the disable-semantics-dashboard branch May 9, 2024 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autosubmit Merge PR when tree becomes green via auto submit App.
Projects
None yet
3 participants