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

Messaging Components - Colors and Icons #605

Merged
merged 3 commits into from
Jan 19, 2025

Conversation

easyuxd
Copy link
Contributor

@easyuxd easyuxd commented Sep 9, 2024

Messaging components changes as part of a larger Messaging Enhancements effort.

Link to main issue: #606

This PR adds new 2 new icons and 1 new color to components.py. The "sign" icon in psbt_screens.py has also been updated from the FontAwesome placeholder to the new icon.


New icons: info, sign

icons

New color: Dire Warning (#FF5700)

dire-warning

New color: Dire Warning
New icons: info, sign
@newtonick newtonick added this to the 0.9.0 milestone Sep 20, 2024
@newtonick newtonick added the gui label Oct 25, 2024
@newtonick
Copy link
Collaborator

ACK

@easyuxd
Copy link
Contributor Author

easyuxd commented Jan 15, 2025

@kdmukai

Added keyboard icons to OTF and updated components.py for 0.8.5 release:

  • delete
  • space

Note: Space icon is double-width (48x24)

@jdlcdl
Copy link

jdlcdl commented Jan 15, 2025

Nice! I like the two new icons for "space" and "del" (in french they're longer too "espace" and "suppr.") as well as the "sign" icon.

@alvroble
Copy link
Contributor

ACK. This button "normalization" solves a lot of future problematic with multilanguage. Let's see now how well they fit in the keyboard screen.

@kdmukai
Copy link
Contributor

kdmukai commented Jan 15, 2025

Screenshot 2025-01-15 at 4 03 07 PM

@kdmukai
Copy link
Contributor

kdmukai commented Jan 15, 2025

ACK.

I have an in-progress branch that swaps these icons in. But it'll be easier if we merge this PR first.

@newtonick
Copy link
Collaborator

ACK

@newtonick newtonick merged commit be465a3 into SeedSigner:dev Jan 19, 2025
2 checks passed
alvroble added a commit to alvroble/seedsigner that referenced this pull request Jan 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Merged Not Yet Released
Development

Successfully merging this pull request may close these issues.

5 participants