Skip to content

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

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

yugalkaushik
Copy link
Contributor

Fixes #3511

Changes:

Web-Editor.mp4

I have verified that this pull request:

  • has no linting errors (npm run lint)
  • has no test errors (npm run test)
  • is from a uniquely-named feature branch and is up to date with the develop branch.
  • is descriptively named and links to an issue number, i.e. Fixes #123

@raclim
Copy link
Collaborator

raclim commented Jun 10, 2025

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?

@yugalkaushik
Copy link
Contributor Author

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?

@raclim
Copy link
Collaborator

raclim commented Jun 10, 2025

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?

@yugalkaushik
Copy link
Contributor Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Download fails silently when logged out
2 participants