Skip to content

[WSL2] Dashboard.jsx: residual hardcoded #0d0b12 in info tooltip — light-theme contrast gap (companion to PR #865 / 30c1c34) #310

@yasinBursali

Description

@yasinBursali

Environment

Description

Upstream commit 30c1c34 ("fix: replace hardcoded colors with theme variables for light theme compat") landed alongside PR Light-Heart-Labs#865 and replaced hardcoded #0d0b12 color literals with theme variables (bg-theme-card, etc.) on dashboard/src/pages/Extensions.jsx. The commit message lists "Tooltip backgrounds, toast notifications, input fields, card footers, filter pills, and GPU badges" as the in-scope items.

A grep of the built dashboard JS bundle in the running dream-dashboard container finds one residual #0d0b12 literal in Dashboard-BOq-Uz6E.js (a Vite code-split chunk for the Dashboard page, which 30c1c34 did not touch — git show --stat 30c1c34 lists only Extensions.jsx).

Source location

dream-server/extensions/services/dashboard/src/pages/Dashboard.jsx, line 518 — the group/info info tooltip:

className="... bg-[#0d0b12]/95 px-3 py-2 text-[11px] leading-4 text-theme-text-muted"

Steps to reproduce

docker exec dream-dashboard sh -c 'grep -o ".\{40\}#0d0b12.\{40\}" /usr/share/nginx/html/assets/Dashboard-*.js'
# -> ... bg-[#0d0b12]/95 px-3 py-2 text-[11px] leading-4 text-theme-text-muted ...

grep -n '#0d0b12' /home/rosenrot/DreamServer/dream-server/extensions/services/dashboard/src/pages/Dashboard.jsx
# -> 518: ...bg-[#0d0b12]/95...

Expected behavior

In light theme, this info tooltip should use a theme-aware background (the same bg-theme-card substitution that 30c1c34 used elsewhere) so it stays legible against light backgrounds. Currently it forces a near-black tooltip background regardless of theme.

Actual behavior

The tooltip is hardcoded to #0d0b12 with 95% opacity, identical to the old (pre-30c1c34) Extensions.jsx pattern that was explicitly fixed.

Suggested fix

One-line substitution mirroring 30c1c34's pattern:

- bg-[#0d0b12]/95
+ bg-theme-card/95

(Or whichever theme token 30c1c34 chose for the equivalent Extensions.jsx tooltip.)

Severity

Low. Cosmetic / accessibility — only affects the info tooltip on the Dashboard page and only when the user is in light theme. Does not block any functionality.

Out-of-scope context

This issue is not a regression in PR Light-Heart-Labs#865 — both PR Light-Heart-Labs#865 and 30c1c34 explicitly scoped only Extensions.jsx. Filing this as a follow-up so the same theme cleanup can be applied to Dashboard.jsx. Found during a QA pass on the merged PR stack while validating PR Light-Heart-Labs#865.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingwsl2WSL2-specific issues

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions