Skip to content

Conversation

@idoshamun
Copy link
Member

@idoshamun idoshamun commented Dec 31, 2025

@vercel
Copy link

vercel bot commented Dec 31, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
daily-webapp Ready Ready Preview Dec 31, 2025 11:56am
1 Skipped Deployment
Project Deployment Review Updated (UTC)
storybook Ignored Ignored Dec 31, 2025 11:56am

Copy link
Contributor

@rebelchris rebelchris left a comment

Choose a reason for hiding this comment

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

The animation injection is kinda meh 😂

Comment on lines 93 to 102
<style>{`
@keyframes gentlePulse {
0%, 100% { opacity: 1; transform: scale(1); }
50% { opacity: 0.6; transform: scale(1.2); }
}
@keyframes fadeSlideUp {
from { opacity: 0; transform: translateY(8px); }
to { opacity: 1; transform: translateY(0); }
}
`}</style>
Copy link
Contributor

Choose a reason for hiding this comment

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

What's this 😅

Copy link
Member Author

Choose a reason for hiding this comment

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

I already did it in another PR that got merged 🙈
What's the alternative? Adding it to a global css is not ideal as well. You prefer a module css and import?

Copy link
Contributor

Choose a reason for hiding this comment

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

Modules already cleaner or just import style.css, but this is so basic tailwind should support it no?

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed

@idoshamun idoshamun merged commit 5bc5811 into main Dec 31, 2025
5 of 7 checks passed
@idoshamun idoshamun deleted the analyze-cosmetics branch December 31, 2025 11:54
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