-
Notifications
You must be signed in to change notification settings - Fork 46
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
Multi: v2 staking design #1016
Multi: v2 staking design #1016
Conversation
Description needs to be updated, because from what i'm seeing it looks like the work is incomplete, and that might have been intentional as follow up PRs are planned, but i'm not sure, can the description state the overall expectation of this PR(including what's purposely left out) and it we are expecting follow up PRs on this |
The PR resolves everything mentioned above. The next PR is the tx details page, and that does not really have anything to do with this PR. |
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.
- Navigating to the staking page cause a overall lag of the app for a couple of seconds.
-
All tickets are showing at once which isn't efficient and most likely the cause of the lag. the list should be paginated.
-
clicking on the settings icon on the staking page causes a crash
5. there should be more right margin after the settings icon, and stake
should be in bold
6. This section should be titledTickets
and the icons don't seem to be on par with the provided figma design
|
|
98fded7
to
0ef219e
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.
clicking the settings icon isn't working, is that intentional?
if you are using a watch only wallet, yes it wont work as its watch only. but other wallets works. |
- clean up redundant files and lines of code
Update stake shuffle to use single wallet selector approach Fix setting button margin bug Fix app crash when settings button is clicked due to watch only wallet selection Fix ticket list hover alignment bug Add title to ticket list section general code clean up
alright, there's nothing indicating that, maybe we should use a toast? |
or shouldn't show it |
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.
I dont understand this comment please. Can you share a screenshot of the UI that did not align? |
- Fix default app container size as it wax too small on full screen. - Adjust pading and localize and fix string variables typo
LGTM @JustinBeBoy are you still having concerns? |
Resolves #981
Fix #1014
Fix #976
Fix #1005
Fix #1022
Fix #978