-
Notifications
You must be signed in to change notification settings - Fork 10
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
fix(Cards): CardAction type #831
Conversation
Size stats
|
Deploy preview for mistica-web ready! ✅ Preview Built with commit c16c335. |
Accessibility report ℹ️ You can run this locally by executing |
@@ -391,21 +391,6 @@ export const cardActionMedia = style([ | |||
}, | |||
]); | |||
|
|||
export const videoAction = style([ |
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.
this was not being used :/
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.
This logic became obsolete when we changed the hover/active states, and the top action for the cards with videos/images became more generic. Before that, the top action for videos used to have a different style (this one), and it was used in line 306 of src/card.tsx
.
Good catch!
🎉 This PR is included in version 14.19.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
No description provided.