Open
Conversation
…em is being hovered over for fill styles
Collaborator
|
Reassigned back to you to make changes we discussed @angelplusultra |
shatfield4
requested changes
Feb 13, 2026
Collaborator
shatfield4
left a comment
There was a problem hiding this comment.
Be sure to not be using the custom css theme variables anymore when implementing designs. Use tailwind classes provided from the designs always being sure to use the light: prefix for light mode classes.
Also avoid using onMouseEnter and onMouseLeave when tracking hover states. This usually adds an unneeded useState and you can usually use group and group-hover tailwind classes to accomplish the same thing using css only.
frontend/src/components/Sidebar/ActiveWorkspaces/ThreadContainer/ThreadItem/index.jsx
Outdated
Show resolved
Hide resolved
…ght color | make icons correct color in light mode
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request Type
screenrecording-2026-02-12_14-51-29.mp4
Relevant Issues
resolves #4982
What is in this change?
Sidebar container (
Sidebar/index.jsx)light:bg-slate-200backgroundFooter icons (
Footer/index.jsx)color="var(--theme-sidebar-footer-icon-fill)"prop, replaced with Tailwind classestext-white light:text-slate-800Settings button icons (
SettingsButton/index.jsx)colorprop replaced withtext-white light:text-slate-800classesActive workspaces (
ActiveWorkspaces/index.jsx)light:outline-2 light:outline-blue-400) to a fill-based style (light:bg-blue-200)light:hover:bg-slate-300; removedhover:font-boldcolor="var(--theme-sidebar-item-workspace-active)"prop with conditional className (light:text-blue-800when active)light:text-blue-900when activegroup/upload,group/gear) for distinct active vs inactive statesThread items (
ThreadContainer/ThreadItem/index.jsx)light:border-blue-800, inactive useslight:border-slate-400light:bg-blue-200; inactive hover getslight:hover:bg-slate-300light:text-blue-900when active withfont-semibold,light:text-slate-800withfont-mediumwhen inactiveNew thread button (
ThreadContainer/index.jsx)light:hover:bg-slate-300bg-white/20tobg-zinc-800 light:bg-slate-50font-semiboldSearch box & new workspace button (
SearchBox/index.jsx)placeholder:text-theme-settings-input-placeholdertoplaceholder:text-white/50 light:placeholder:text-slate-500 placeholder:font-semiboldlight:hover:bg-slate-300hover state; Plus icon getslight:text-slate-500Additional Information
All light mode overrides use Tailwind
light:variant classes directly in JSX rather than introducing new CSS variables. CSS variablecolorprops on Phosphor icons have been replaced with Tailwind text color classes, keeping the approach consistent with the rest of the codebase.Developer Validations
yarn lintfrom the root of the repo & committed changes