Skip to content

fix: remove !important from workspace session tab custom background to allow color overrides#11

Merged
amirlehmam merged 1 commit into
amirlehmam:masterfrom
H37IX:fix/workspace-custom-colors
May 10, 2026
Merged

fix: remove !important from workspace session tab custom background to allow color overrides#11
amirlehmam merged 1 commit into
amirlehmam:masterfrom
H37IX:fix/workspace-custom-colors

Conversation

@H37IX

@H37IX H37IX commented May 10, 2026

Copy link
Copy Markdown
Contributor

Fix: workspace session tab background now updates correctly when selecting a color

This fixes an issue where the workspace did not update its background color when using the custom color choosers.

Root Cause

The CSS rule for the workspace used !important, which prevented background styles (color picker) from having effect.

Fix Applied

Removed !important so inline styles take precedence:

.workspace-row--active {
  background: #0091FF; /* removed !important */
}

Fixes #10 

… custom color overrides from the custom color picker
@amirlehmam amirlehmam merged commit 52999c2 into amirlehmam:master May 10, 2026
amirlehmam added a commit that referenced this pull request May 11, 2026
- Fix #12: workspacePrefs (defaultShell, placement, auto-reorder) had no
  persistence layer, so any value the user picked in Settings was lost on
  the next launch — the "Default shell" toggle felt broken even when it
  saved into state during the session. Adds a scoped localStorage layer
  for just this slice (key: wmux-workspace-prefs); failures are swallowed
  so sandboxed/locked storage can't break the renderer boot.
- Annotates the .workspace-row--active rule (fix for #10 landed via #11)
  so the next person doesn't reintroduce !important and break custom
  workspace colors again.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants