-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Update worker node design and add missing token #1140
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
base: main
Are you sure you want to change the base?
Conversation
| <div | ||
| key={id} | ||
| className={`relative w-full rounded-xl border bg-surface-primary px-sm py-2 ${className || ''} group overflow-visible`} | ||
| className={`rounded-xl bg-surface-tertiary px-sm py-2 relative w-full border ${className || ''} group overflow-visible`} |
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.
(reference comment)
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.
@a7m-1st I am still cleaning up the commit. Somehow the prettier just force my code formate change lol
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.
oh okay, its fine just reference for me to test as I was just skimming the code yesterday
…ibility - Downgrade prettier-plugin-tailwindcss to v0.6.14 for Tailwind v3 compatibility - Add tailwindConfig to .prettierrc.json for proper Tailwind CSS class sorting - Update design tokens: - Change input background to use tertiary surface - Update tag foreground to use information surface - Adjust task running fill to primary surface - Refine border focus color intensity across all themes - Add background CSS variable for consistent theme support This fixes formatting issues caused by prettier-plugin-tailwindcss v0.7.2 which requires Tailwind v4, while the project uses Tailwind v3. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
4631956 to
670bd03
Compare
- Add Copy button to task reports for easy clipboard access - Implement AnimatePresence for smooth UI transitions - Refactor hook declarations for better React compliance - Fix dependency arrays to use direct references instead of extracted variables - Simplify failed task filtering (remove reAssignTo check) - Add early return guard for chatStore safety - Improve handleShowLog function implementation This redesign improves UX for worker node interactions and task report management. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
1c35f68 to
ce5bc05
Compare
a7m-1st
left a comment
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.
The Bg change, Tooltip, and new completion report LGTM thanks @Douglasymlai

Description
close #997
What is the purpose of this pull request?