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

Release Announcement component #154

Merged
merged 29 commits into from
Apr 3, 2024

Conversation

florianduros
Copy link
Member

@florianduros florianduros commented Mar 21, 2024

Figma release announcement https://www.figma.com/file/WGM39wEVQSJxnSFf6eeVry/Fixing-Threads-notifs?type=design&node-id=1124-2469&mode=design

This component is under the release announcement folder in storybook.
image

Following element-hq/compound#298, we are using floating-ui instead of radix to avoid accessibility issues on floating/popover/tooltip behavior.

Copy link

cloudflare-workers-and-pages bot commented Mar 21, 2024

Deploying compound-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3a3ef67
Status: ✅  Deploy successful!
Preview URL: https://352ead76.compound-web.pages.dev
Branch Preview URL: https://florianduros-announcement-co.compound-web.pages.dev

View logs

@florianduros florianduros force-pushed the florianduros/announcement-component branch from 44f47a8 to 3833930 Compare March 25, 2024 15:57
@florianduros florianduros force-pushed the florianduros/announcement-component branch from 26d03ff to 3bffb9b Compare March 28, 2024 13:26
@florianduros florianduros marked this pull request as ready for review March 28, 2024 13:30
@florianduros florianduros requested a review from a team as a code owner March 28, 2024 13:30
@florianduros florianduros removed the request for review from a team March 28, 2024 13:30
Copy link
Member

@robintown robintown left a comment

Choose a reason for hiding this comment

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

Looking good, here are my thoughts:

}

.description {
color: var(--cpd-color-gray-500);
Copy link
Member

Choose a reason for hiding this comment

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

This seems to now use color/text/on-solid/primary in the designs

Copy link
Member Author

Choose a reason for hiding this comment

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

That's the case of the header but the description is on Light/color/gray/500

src/components/ReleaseAnnouncement/ReleaseAnnouncement.tsx Outdated Show resolved Hide resolved
src/components/ReleaseAnnouncement/ReleaseAnnouncement.tsx Outdated Show resolved Hide resolved
@robintown robintown removed the request for review from dbkr March 28, 2024 19:07
@florianduros florianduros merged commit 9805166 into main Apr 3, 2024
6 checks passed
@florianduros florianduros deleted the florianduros/announcement-component branch April 3, 2024 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants