-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Add tooltip for unauthenticated download button #3514
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
Conversation
Thanks so much for your work on this @yugalkaushik, I think this looks great so far!
I think my only concern is whether the implementation for this tooltip is WCAG 2.2 compliant. I think the relevant criterion might be 1.4.3 Content on Hover or Focus?
So I tried making the tooltip WCAG 2.2 compliant, but it looks like we'd need to change the whole functionality to make it work properly or we can keep the basic implementation. What do you suggest?
Thanks for taking a look into it!
We're aiming to become WCAG 2.2 compliant, so I think it might be better to take a step back to come up with a tooltip that meets the criterion before pushing anything out. This is also probably linked to @takshittt's PR (#3497). Maybe this could be something to look into altogether?
I'm thinking of creating a separate, common tooltip modal component that we can reuse wherever needed. This should help reduce the need to manually implement tooltips in each component. Once that's done, we can use that tooltip here as well.
awood0727
commented
Aug 6, 2025
Hi @raclim, my team and I, which consists of @npNSU, @kaybcodes, and @SilasVM have looked into issues #3709 and #3511. We agree they could be modified to be more WGAC-compliant, and have done some work/research into doing so. Some potential improvements are as follows: ensure the user can dismiss the prompt themselves (utilizing their keyboard and mouse to exit), and ensure the labels are written as simply as possible for screen reader users.
We were also able to match the tooltip color from this pull request and PR #3497. We explored an option that can make unavailable buttons grayed out while the user is not logged in, and additionally located the area where it is possible to change; however, it changes the entire list color, so new lines of code will need to be added to specify the list element in question. Additionally, we believe that the amount of work needed to complete this will take a lot of time and effort. This is why we believe the best course of action would be to allow multiple contributors to help with both issues to make them more WGAC-compliant. With the ability my team has, we believe we can only gray out the buttons that are in question, and if you would like to, we can make a pull request for that and continue to allow other contributors to edit our changes and continue with the outline we have suggested.
Fixes #3511
Changes:
Web-Editor.mp4
I have verified that this pull request:
npm run lint
)npm run test
)develop
branch.Fixes #123