Skip to content

Add Deepness theme stylesheet (css/deepness.css) - #37

Merged
bmfmancini merged 1 commit into
mainfrom
feature/deepness-theme-css
Oct 6, 2026
Merged

bmfmancini merged 1 commit into
mainfrom
feature/deepness-theme-css

Conversation

@TheWitness

Copy link
Copy Markdown
Member

Summary

Adds a css/deepness.css stylesheet so this plugin's elements are tuned for the
new Deepness dark-emerald theme (proposed in Cacti/cacti#8202).

It mirrors the plugin's existing dark.css (low-opacity severity tints with
light text so the dark page shows through) and aligns it with the Deepness
design intent:

  • Success / "OK" / "Up" / "restored" colours shift to the Deepness emerald accent.
  • Dark surface colours are aligned to the Deepness panel palette.
  • Error / warning / notice / acknowledge colours are kept (already dark-appropriate).

No behaviour change: Cacti only loads css/<theme>.css when that theme is the
selected theme, so this file is inert unless the Deepness theme is active.

Copilot AI balanced review requested due to automatic review settings October 6, 2026 21:49

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The scoped color overrides match existing selectors and theme-loading behavior, with no blocking issues identified.

Review effort: Balanced
Findings: None

What changed in this PR

Adds Deepness-specific styling for QuickTree graph action glyphs.

Changes:

  • Uses emerald for add actions and red for remove actions.
  • Applies only when the Deepness theme is selected.
File Description
css/​deepness.css Defines Deepness colors for add/remove glyphs.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@bmfmancini
bmfmancini merged commit 8a20a27 into main Oct 6, 2026
6 checks passed
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.

3 participants