-
Notifications
You must be signed in to change notification settings - Fork 120
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
spv: spv mode indicator on sidebar #2666
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.
Thanks for this PR. I think it looks good, but can you also add it when the sidebar is reduced or in the bottom?
So I think that we should use the label SPV instead of the icon. What do you think @vctt94? |
Hrm I think the icon with a tooltip should suffice for now. We can tweak if people seem to be confused after release. |
IMO, on mobile view, adding the spv icon would pollute the current UI. Maybe we should go with the icon + tooltip for the full-width view, and use the label for collapsed views. |
Fine by me |
0142f01
to
657c46b
Compare
657c46b
to
66e1760
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.
LGTM
This diff closes #2640 by adding the SPV icon on "Latest Block" sidebar section.
(on hover)