-
Notifications
You must be signed in to change notification settings - Fork 14
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
Export switch subtext setter #162
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few questions
app/ui/view_nbgl.c
Outdated
@@ -292,7 +278,9 @@ static void settings_screen_callback(uint8_t index, nbgl_content_t *content) { | |||
#ifdef APP_BLINDSIGN_MODE_ENABLED | |||
switches[BLINDSIGN_MODE].initState = app_mode_blindsign(); | |||
switches[BLINDSIGN_MODE].text = "Blind sign"; | |||
switches[BLINDSIGN_MODE].subText = ""; | |||
if ((switches[BLINDSIGN_MODE].subText) == NULL) { | |||
switches[BLINDSIGN_MODE].subText = "Allows signing transactions without seeing all fields"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there enough space to display this on flex/stax? instead of "seeing", maybe "reviewing"? Otherwise, let's ask AI what it thinks about this description haha.
5e317d3
to
22bc419
Compare
22bc419
to
35dc757
Compare
view_set_switch_subtext
so that any app can modify the subtext of a switch according to its needs.🔗 zboto Link