-
Notifications
You must be signed in to change notification settings - Fork 214
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
Update the CTA buttons #1049
Update the CTA buttons #1049
Conversation
Size Change: -205 B (0%) Total Size: 844 kB
ℹ️ View Unchanged
|
Full-stack documentation: https://docs.openverse.org/_preview/1049 Please note that GitHub pages takes a little time to deploy newly pushed code, if the links above don't work or you see old versions, wait 5 minutes and try again. You can check the GitHub pages deployment action list to see the current status of the deployments. |
eb15997
to
64374df
Compare
Based on this PR, I added a new comment in #860 suggesting a spacing update. |
f2c3ce3
to
e37ce56
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.
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.
Everything looks correct. Great work ✨
The center-aligned version definitely looks better @zackkrida, but the problem is rooted in the control button size. It's not part of this update, but it should be tackled after updating the components list. To me is not a blocker as we are aware of this issue and need to solve it after updating the button components. But I will leave it to @obulat to decide. |
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.
Fantastic!
* Use Python to group items by license to speed up the query Signed-off-by: Olga Bulat <[email protected]> * Remove unnecessary type conversions Signed-off-by: Olga Bulat <[email protected]> * Lowercase the license names; save the invalid values to S3 Signed-off-by: Olga Bulat <[email protected]> * Use a tempfile for invalid items Signed-off-by: Olga Bulat <[email protected]> --------- Signed-off-by: Olga Bulat <[email protected]>
Fixes
Related to #1020 by @obulat
Description
This PR applies the new button styles to the CTAs according to the Figma files: https://www.figma.com/file/es303VxVr0C7rFTcOe3OXL/Core-interface-improvement?node-id=950-7110&t=ofmQkfpwZhitIllp-4
The external icons are removed because we can add them by setting
show-external-icon
prop, which will be passed to theVLink
component.Testing Instructions
The CI should work, and the snapshots look correct and match the Figma mockups.
Checklist
Update index.md
).main
) ora parent feature branch.
errors.
Developer Certificate of Origin
Developer Certificate of Origin