-
Notifications
You must be signed in to change notification settings - Fork 195
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
Updates to Extension Settings layout #2874
Comments
@stackingsaunter I got it.
Are we removing the whole section?( settings part ) |
@AdarshRawat1 yes, whole UI parts of the section. |
Hey @AdarshRawat1, are you still working on this issue? If not, then @stackingsaunter, I would like to try tackling this good first issue. |
This one totally slipped my mind. 🤧 @amitamrutiya2210 if you wish to start contributing to Alby, you can start with this good first issue. |
Thanks @AdarshRawat1, for replying. I will work on this issue then. |
Hey @stackingsaunter, I think some of your suggested changes have already been addressed or removed in the current codebase, such as:
Here are the things I need to do:
Is there anything you want to suggest? |
I think there is some confusion.
You have to add the subtitle as it is not present currently. Check the final design in this Figma file and make changes accordingly. ( The points listed in issue description are there to help you find the changes to be made i.e. the difference between current design and desired design.) |
Okay, I overthought this issue. Now I understand it. |
Part of #2871 epic, which full desired effect can be viewed in this Figma
Changes:
Page tile contains a description "Global settings that affect your experience on all connected wallets in - Alby Extension.". Space between title and descriptions should be 4px.
Section headers are text-xl/bold
All buttons and dropdowns should have fixed 256px width
Use Title case in all list item labels and buttons
Delete Nostr section
Delete legacy section
The text was updated successfully, but these errors were encountered: