feat(app): improve UI styling and accessibility - #37
Conversation
zachmcnair
commented
Dec 16, 2025
zachmcnair
commented
- Replace Work Sans font with Inter for better readability
- Update dark mode color scheme for better contrast
- Improve dropdown styling with proper background colors
- Add max-width constraint (640px) to chat messages
- Update selected chat item styling in light mode
- Reduce sidebar navigation text size by 2pt
- Fix orphaned words in setup completion modal
- Update logo to use dark mode variant when appropriate
- Replace Work Sans font with Inter for better readability - Update dark mode color scheme for better contrast - Improve dropdown styling with proper background colors - Add max-width constraint (640px) to chat messages - Update selected chat item styling in light mode - Reduce sidebar navigation text size by 2pt - Fix orphaned words in setup completion modal - Update logo to use dark mode variant when appropriate
comppaz
left a comment
comppaz
left a comment
There was a problem hiding this comment.
Review Summary
Good improvements overall, but please address these consistency issues before merge:
- Hardcoded colors - Use design tokens instead of hex values
- Extension dark mode not synced - Colors don't match the app
- Title bar feature - Consider separating into its own PR
See line comments below for specifics.
Line-specific feedback:
linear-gradient(to bottom right, hsl(var(--background)) 0%, hsl(var(--card)) 100%)Instead of hardcoded hex values.
|
…d UI improvements - Replace hardcoded colors with design tokens across app and extension - Sync extension dark mode colors with app (blue-tinted grays) - Update extension popup and sidebar styling to match app design - Fix background gradients to use CSS variables consistently - Add logout button to settings page (floats right of title) - Fix animated background orb colors to use correct blue hue - Add protocol handler (think://) for opening app from extension - Update extension icons for light/dark modes - Improve font loading in extension popup - Fix button width consistency in extension popup
|
@comppaz Thanks for the feedback! I've done a more thorough update across extension and app following your feedback. Mind reviewing? |
|
@zachmcnair I will review it by tmr 👍 If it looks fine in general and you don't mind, I would take it over and split it up in a few small PRs! |
|
@zachmcnair So in general what the PR aims to achieve looks good to me. But it should be actually at least 3 PRs not one, and not so many hardcoded / inline styles. If you like I can take that over, split it and clean it up. |
|
@comppaz Thanks for your grace. I'll make sure to make smaller PRs in the future. As for this PR, I'll work to split it up. I don't want to burden you. If I do a shit job in breaking it apart, I'll gladly accept your help! :) |
2 participants
