-
Notifications
You must be signed in to change notification settings - Fork 25
refactor: transitions and animations to tailwind #1141
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
Conversation
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.
Pull Request Overview
This pull request refactors the codebase to use Tailwind CSS classes for transitions and animations instead of custom CSS. The purpose is to standardize animation and transition patterns while maintaining the visual behavior of the UI.
- Replaced custom CSS transitions with Tailwind utility classes
- Moved animation keyframes from individual component styles to a centralized location
- Updated test snapshots to reflect the new class names
Reviewed Changes
Copilot reviewed 36 out of 36 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| design-system/src/styles/tailwind.css | Added centralized animation keyframes (shake, shimmer, loading-bar) |
| design-system/src/styles/layers.css | Added transition classes to oc-card component |
| design-system/src/components/*/ | Multiple components updated to use Tailwind transition classes |
| web-runtime/src/components/ | Transition classes updated in navigation and notification components |
| web-pkg/src/components/ | Various sidebar and filter components updated |
| test snapshots | Updated to reflect new Tailwind class names |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
628d8fc to
558bfe4
Compare
…ilwind refactor: transitions and animations to tailwind
The few complex transitions are still written in pure CSS, I don't feel like it would be fun (or even possible) to write them via Tailwind classes.
refs #937