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
[DDW-685] Stake Pools - Adjustments #1420
Conversation
a3e1bc6
to
d059290
Compare
@daniloprates Netlify build is fixed 👍 |
|
Rest is very good! 👍 |
I'll remove the fixed Search component until we find a better solution. ATM it's taking too much space. |
@a-rukin in this case, I can't just ignore the right margin, as it would make it even when there's the scrollbar, but uneven when there is not. Let's leave this one like this and I'll open a card for us to study handling the scrollbar in a better way. Maybe we can apply one of those Javascript solutions globally in the app. |
Sure 👌 |
Hey reviewers, all the requests were made and are available here: https://daniloprates.github.io/daedalus/?selectedKind=Staking&selectedStory=Stake%20Pools&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybooks%2Fstorybook-addon-knobs |
source/renderer/app/components/staking/stake-pools/StakePoolTooltip.scss
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.
@daniloprates great work! I have left you only one very tiny change request: #1420 (comment)
@MarcusHurney please review this one too 🙏 |
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.
Great work @daniloprates 🎉
…com/input-output-hk/daedalus into fix/ddw-685-stake-pools-adjustments
@daniloprates there are some issues with AppVeyor. @cleverca22 will help us resolve them in the scope of 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.
OK
0628a1e
to
8b85a1f
Compare
…com/input-output-hk/daedalus into fix/ddw-685-stake-pools-adjustments
This PR adds fine adjustments to the Stake Pools screen.
Todos
https://docs.google.com/document/d/10UwMUoDBalrRS4QvLZq2dtXJ42ICdpfgDPIUo3Fcxeg/edit#
Tooltip positioning and Search/Filtering will be handled on different PR's
Temporary Storybook URL, while Netifly is failing.
Screenshots
Testing Checklist
Review Checklist
Basics
feature
/bug
/chore
,release-x.x.x
)yarn test
)yarn dev
)yarn package
/ CI builds)yarn flow:test
)yarn lint
)yarn prettier:check
)yarn manage:translations
produces no changes)yarn storybook
)yarn.lock
file is updatedCode Quality
Testing
After Review
done
column on the YouTrack board