Skip to content

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

Open
TheWitness wants to merge 1 commit into
developfrom
feature/deepness-theme-css
Open

TheWitness wants to merge 1 commit into
developfrom
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
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The required changelog entry for the new theme support is missing.

Review effort: Balanced
Findings: 1 Low severity

Open (1)
What changed in this PR

Adds plugin styling for Cacti’s proposed Deepness theme.

Changes:

  • Mirrors dark-theme threshold and device styling.
  • Applies emerald accents to healthy states and action glyphs.
File Description
css/​deepness.css Adds Deepness-specific status, glyph, and legend styling.

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

Comment thread css/deepness.css
@@ -0,0 +1,380 @@
/*
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