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

feat: made advanced settings for token creation read-only #534

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

tushar-saxena-yral
Copy link
Contributor

Closes #529

Disables the fields, but the ux is a bit compromised. Should I add a bit styling and a white text saying something along the lines of you can't change these settings? or is this good enough

@rupansh-sekar-yral
Copy link
Contributor

@saikatdas0790 thoughts on UX suggestions?

@saikatdas0790
Copy link
Contributor

@tushar-saxena-yral Let's bring this up during today's meeting?

@rupansh-sekar-yral rupansh-sekar-yral dismissed their stale review November 27, 2024 06:56

pending UI changes

@tushar-saxena-yral
Copy link
Contributor Author

tushar-saxena-yral commented Dec 12, 2024

https://www.figma.com/file/qly4tjTKMMbdOO4IrLzOeC?node-id=766:2199&locale=en&type=design

  • Create the banner
  • Implement the settings card
  • Add tooltips with description

@tushar-saxena-yral
Copy link
Contributor Author

@harshita-srivastava-yral i cut some corners with the tooltip to save dev time. lemme know if you want it to be pixel perfect, otherwise i feel it will suffice

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.

Make the advanced setting fields read only
4 participants