Skip to content
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

feat(Nakedcard): new component #819

Merged
merged 17 commits into from
Jul 20, 2023
Merged

feat(Nakedcard): new component #819

merged 17 commits into from
Jul 20, 2023

Conversation

atabel
Copy link
Collaborator

@atabel atabel commented Jul 14, 2023

No description provided.

@github-actions
Copy link

github-actions bot commented Jul 14, 2023

Size stats

master this branch diff
Total JS 9.36 MB 9.36 MB +7.45 kB
JS without icons 875 kB 882 kB +7.37 kB
Lib overhead 122 kB 122 kB 0 B
Lib overhead (gzip) 31.9 kB 31.9 kB 0 B

@github-actions
Copy link

github-actions bot commented Jul 14, 2023

Accessibility report
✔️ No issues found

ℹ️ You can run this locally by executing yarn audit-accessibility.

@github-actions
Copy link

github-actions bot commented Jul 14, 2023

Deploy preview for mistica-web ready!

✅ Preview
https://mistica-3r0icumox-tuentisre.vercel.app

Built with commit 533571b.
This pull request is being automatically deployed with vercel-action

@github-actions
Copy link

github-actions bot commented Jul 14, 2023

Screenshot tests report

✔️ All passing

src/card.tsx Show resolved Hide resolved
src/__stories__/naked-card-story.tsx Outdated Show resolved Hide resolved
src/image.tsx Show resolved Hide resolved
src/image.tsx Show resolved Hide resolved
@atabel atabel requested a review from yceballost July 18, 2023 09:57
@yceballost yceballost self-requested a review July 18, 2023 12:58

return (
<CardContainer
width={width}
height={height}
dataAttributes={{...dataAttributes, 'component-name': 'PosterCard'}}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe you can use getPrefixedDataAttributes here

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, that's already done inside CardContainer

src/card.tsx Outdated
<div style={{position: 'relative'}}>
{isTouchable && (
<div
className={classNames(styles.touchableMediaOverlay, {
Copy link
Contributor

@aweell aweell Jul 19, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The media part of the naked differs from the media hover definition, and uses the same overlay that displayMedia and poster cards with the following tokens:

coverBackgroundHover, coverBackgroundPressed

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed, looks much better now!

@atabel atabel requested a review from aweell July 19, 2023 11:28
Copy link
Contributor

@aweell aweell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@atabel atabel added this pull request to the merge queue Jul 20, 2023
Merged via the queue into master with commit 688bb8e Jul 20, 2023
10 checks passed
@atabel atabel deleted the atoledano-naked-card branch July 20, 2023 08:06
tuentisre pushed a commit that referenced this pull request Jul 20, 2023
# [14.18.0](v14.17.1...v14.18.0) (2023-07-20)

### Bug Fixes

* **DisplayMediaCard, PosterCard:** add label to video control ([#823](#823)) ([dc765ef](dc765ef))
* **row:** add missing dataAttributes ([#815](#815)) ([47e7cab](47e7cab))

### Features

* **AdvancedDataCard:** new community component ([#780](#780)) ([2803b9c](2803b9c))
* **Blocks:** new Mistica Community Components ([#770](#770)) ([8fc368a](8fc368a))
* **Header:** small version, allow removing vertical padding ([#822](#822)) ([448387e](448387e))
* **Nakedcard:** new component ([#819](#819)) ([688bb8e](688bb8e))
@tuentisre
Copy link
Collaborator

🎉 This PR is included in version 14.18.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants