-
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
re-align RadioListItem button to top of component #769
Conversation
Thanks for opening this pull request! Someone will review it soon 🔍 |
Hi, any update on this one? |
Hi @stvtortora, sorry for the delay, I am checking now. I noticed the spacing wasn’t exactly the same as the Figma file (1px vs. 4px). The default on |
Hi @bpierre. No worries. I removed that line, but I don't think its what we want because no default value took its place. As far as I can tell, there does not appears to be a file (or any code for that matter) with "Figma" in this repository. |
Ah sorry I should have been more explicit, I was referring to the link to Figma in the issue: https://www.figma.com/file/bZ1j7omfhdMCbtFahP3nEJ9B/Client?node-id=525%3A1141 I was just testing and it seems correct now? |
@bpierre Ah, ok, thanks for clarifying! It looks good to me 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.
💯
Fixes aragon/client#1340