-
-
Notifications
You must be signed in to change notification settings - Fork 629
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
hold to confirm animation for mercury ui #3776
hold to confirm animation for mercury ui #3776
Conversation
|
902aa4c
to
ff59b1d
Compare
|
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.
Some comments.
core/embed/rust/src/ui/model_mercury/flow/confirm_reset_create.rs
Outdated
Show resolved
Hide resolved
core/embed/rust/src/ui/model_mercury/component/hold_to_confirm.rs
Outdated
Show resolved
Hide resolved
core/embed/rust/src/ui/model_mercury/component/hold_to_confirm.rs
Outdated
Show resolved
Hide resolved
core/embed/rust/src/ui/model_mercury/component/hold_to_confirm.rs
Outdated
Show resolved
Hide resolved
core/embed/rust/src/ui/model_mercury/component/hold_to_confirm.rs
Outdated
Show resolved
Hide resolved
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.
Build fails
core/embed/rust/src/ui/model_mercury/component/hold_to_confirm.rs
Outdated
Show resolved
Hide resolved
core/embed/rust/src/ui/model_mercury/component/hold_to_confirm.rs
Outdated
Show resolved
Hide resolved
@@ -161,7 +161,7 @@ impl Component for Bip39Input { | |||
let style = self.button.style(); | |||
|
|||
// First, paint the button background. | |||
self.button.render_background(target, style); | |||
self.button.render_background(target, style, 0xFF); |
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.
Note: this line was removed in #3832 so it will cause conflict later.
@@ -192,7 +192,7 @@ impl Component for Slip39Input { | |||
let style = self.button.style(); | |||
|
|||
// First, paint the button background. | |||
self.button.render_background(target, style); | |||
self.button.render_background(target, style, 0xFF); |
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.
Note: this line was removed in #3832 so it will cause conflict later.
73c2df4
to
4808c34
Compare
4808c34
to
2fbe4a0
Compare
fda164d
to
ee43705
Compare
2fbe4a0
to
38c6b52
Compare
d1bf5a3
to
b8409a3
Compare
38c6b52
to
8c02187
Compare
604c04b
to
6455f69
Compare
[no changelog]
[no changelog]
[no changelog]
6455f69
to
25762fd
Compare
d0c02dd
into
tychovrahe/ui-t3t1/brightness
This PR adds hold to confirm animation to mercury UI.
Draft for now, as it relies on several other PRs which should go in first, namely: #3775 #3763 , and also contains i2c fix that should be removed in the final PR.