Skip to content

[dev] [claudfuen] claudio/ui-tweaks#860

Merged
claudfuen merged 14 commits into
mainfrom
claudio/ui-tweaks
Jun 10, 2025
Merged

[dev] [claudfuen] claudio/ui-tweaks#860
claudfuen merged 14 commits into
mainfrom
claudio/ui-tweaks

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

This is an automated pull request to merge claudio/ui-tweaks into dev.
It was created by the [Auto Pull Request] action.

- import Button component for use in menu items
- simplify menu item filters by combining conditions
- remove commented-out code and unused badge variants
- adjust TooltipProvider structure for better readability
- update Item component to use Button and improve tooltip logic
- eliminate unnecessary padding from navigation component
@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 10, 2025 4:57pm
comp-framework-editor ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 10, 2025 4:57pm
comp-portal ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 10, 2025 4:57pm
comp-trust ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 10, 2025 4:57pm

- update button class logic for better alignment based on isCollapsed state
- conditionally render TooltipContent based on isCollapsed state for cleaner code structure
cursor[bot]

This comment was marked as outdated.

- Simplify color pairs definition to a single string array
- Remove unused `isCollapsed` prop and replace with `size` prop
- Adjust button and avatar styling for consistency
- Remove unnecessary role and aria-label attributes
- Clean up placeholder and text content syntax
- Enhance layout and alignment with flex utilities
cursor[bot]

This comment was marked as outdated.

- change button variant from "outline" to "ghost" for style consistency
- rearrange class names for better readability and maintenance
cursor[bot]

This comment was marked as outdated.

- replace list items with button components for better consistency
- use ghost variant buttons for navigation
- enhance styling for active and disabled states
- integrate Tooltip for disabled items with "Coming soon" message
cursor[bot]

This comment was marked as outdated.

- move "Add Framework" button to center for improved UI alignment
cursor[bot]

This comment was marked as outdated.

- Replace "rounded-xs" with "rounded-sm" for consistent border radius across components
cursor[bot]

This comment was marked as outdated.

…layout

- Replace manual breadcrumb rendering with PageWithBreadcrumb for consistency across task pages.
- Update TaskList and layout components to utilize the new breadcrumb structure.
- Enhance sidebar and main menu components for better styling and responsiveness.
- Adjust button styles and layout in various components for improved user experience.
cursor[bot]

This comment was marked as outdated.

- Replace div containers with Card components in SingleTask and TaskList for better visual structure.
- Adjust TaskFilterHeader layout for improved spacing and alignment.
- Update AnimatedLayout and Sidebar components for consistent background styling.
- Refactor Tabs component to streamline structure and improve performance with memoization and state management.
- change hover highlight height from h-8 to h-9 for better fit
- remove redundant height class from menu item styling
- update text style to remove font-medium from active state
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Bug: Styling Regression in `AssistantButton` Component

The AssistantButton component lacks proper styling because its className prop was removed and the intended replacement styling was committed as a commented-out block.

apps/app/src/components/ai/chat-button.tsx#L21-L28

<Button
variant="ghost"
size="default"
// className={cn(
// "relative w-full max-w-sm justify-start text-muted-foreground",
// "hover:bg-accent hover:text-accent-foreground"
// )}
onClick={() => setAssistantOpen(true)}

Fix in Cursor


Was this report helpful? Give feedback by reacting with 👍 or 👎

@claudfuen claudfuen merged commit d7e5258 into main Jun 10, 2025
11 checks passed
@claudfuen claudfuen deleted the claudio/ui-tweaks branch June 10, 2025 17:07
@claudfuen
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.40.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant