Skip to content

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

Closed
takshittt wants to merge 1 commit intoprocessing:developfrom
takshittt:takshit/login-to-upload-file
Closed

Add tooltip for "Upload File" button when user is not authenticated#3497
takshittt wants to merge 1 commit intoprocessing:developfrom
takshittt:takshit/login-to-upload-file

Conversation

@takshittt
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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!

@raclim
Copy link
Copy Markdown
Collaborator

raclim commented Feb 19, 2026

I'm sorry that this PR ended up falling out of date! Closing since the original issue has been addressed.

@raclim raclim closed this Feb 19, 2026
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

3 participants