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

spv: spv mode indicator on sidebar #2666

Merged
merged 4 commits into from
Oct 7, 2020

Conversation

victorgcramos
Copy link
Member

This diff closes #2640 by adding the SPV icon on "Latest Block" sidebar section.

Screen Shot 2020-09-11 at 10 31 06 AM

(on hover)
Screen Shot 2020-09-11 at 10 31 00 AM

Copy link
Member

@vctt94 vctt94 left a 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?

@victorgcramos
Copy link
Member Author

victorgcramos commented Sep 16, 2020

So I think that we should use the label SPV instead of the icon. What do you think @vctt94?

@alexlyp
Copy link
Member

alexlyp commented Sep 16, 2020

Hrm I think the icon with a tooltip should suffice for now. We can tweak if people seem to be confused after release.

@victorgcramos
Copy link
Member Author

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.

@vctt94
Copy link
Member

vctt94 commented Sep 17, 2020

Fine by me

@victorgcramos victorgcramos force-pushed the 2640-spv-mode-label branch 2 times, most recently from 0142f01 to 657c46b Compare September 17, 2020 13:51
@victorgcramos
Copy link
Member Author

Collapsed view looks like this:
Screen Shot 2020-09-17 at 11 37 23 AM

Copy link
Member

@vctt94 vctt94 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alexlyp alexlyp merged commit df39847 into decred:master Oct 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add SPV mode label
3 participants