Skip to content

Add tooltip for "Upload File" button when user is not authenticated #3497

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 1 commit into
base: develop
Choose a base branch
from

Conversation

takshittt
Copy link
Contributor

Fixes #3079

Changes:

  • Added a tooltip to the "Upload File" button to inform users that authentication is required before uploading.
  • This enhances the user experience by providing guidance instead of leaving the button inactive without explanation.
Screenshot 2025-05-27 at 12 45 46 AM

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 #3079

@yugalkaushik
Copy link
Contributor

@takshittt Since the "Upload File" button isn’t shown at all when the user is not logged in, adding a tooltip about authentication isn’t needed. Users won’t see the button unless they’re logged in, so the tooltip wouldn’t be visible or helpful.

@takshittt
Copy link
Contributor Author

Hi @yugalkaushik
Thanks for pointing that out! I’ve actually updated the behavior so that the "Upload File" button is always visible, even when the user is not logged in — it's just disabled in that state. A tooltip or inline message explains that login is required to use it.

This approach improves accessibility and user experience by making the UI more predictable. Users can immediately see what actions are available, and once they log in, the button becomes active without needing to rediscover it.

Let me know if you have any feedback!

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.

Notation for the need of login to upload (image) files
2 participants