Skip to content

Add: add dark mode theming - #5330

Open
Chri6s wants to merge 2 commits into
greenbone:mainfrom
Chri6s:main
Open

Add: add dark mode theming#5330
Chri6s wants to merge 2 commits into
greenbone:mainfrom
Chri6s:main

Conversation

@Chri6s

@Chri6s Chri6s commented Jun 4, 2026

Copy link
Copy Markdown

What

  • Adds a global dark mode color scheme using the native Mantine ThemeSwitch inside the header.
  • Aligns assets (SVGs, conditional animations like :hover) to the selected colorTheme.
  • Adds safety checks to handle CSS variables during d3-color chart computations to prevent crashes.

Why

  • Better user experience.

References

  • N/A

Checklist

  • Tests (aligned the ones that were not compliant to this change)

@Chri6s
Chri6s requested a review from a team as a code owner June 4, 2026 17:31
@greenbonebot
greenbonebot enabled auto-merge (rebase) June 4, 2026 17:31
auto-merge was automatically disabled June 5, 2026 11:59

Head branch was pushed to by a user without write access

@greenbonebot
greenbonebot enabled auto-merge (rebase) June 5, 2026 12:00
auto-merge was automatically disabled June 7, 2026 22:09

Head branch was pushed to by a user without write access

@greenbonebot
greenbonebot enabled auto-merge (rebase) June 7, 2026 22:09
auto-merge was automatically disabled June 7, 2026 22:10

Head branch was pushed to by a user without write access

@greenbonebot
greenbonebot enabled auto-merge (rebase) June 7, 2026 22:11
auto-merge was automatically disabled June 10, 2026 11:15

Head branch was pushed to by a user without write access

@greenbonebot
greenbonebot enabled auto-merge (rebase) June 10, 2026 11:15
@Chri6s Chri6s changed the title feat: add dark mode support Add: add dark mode theming Jun 10, 2026
auto-merge was automatically disabled June 10, 2026 14:54

Head branch was pushed to by a user without write access

@greenbonebot
greenbonebot enabled auto-merge (rebase) June 10, 2026 14:54
auto-merge was automatically disabled June 10, 2026 18:58

Head branch was pushed to by a user without write access

@greenbonebot
greenbonebot enabled auto-merge (rebase) June 10, 2026 18:58
@Chri6s
Chri6s marked this pull request as draft June 15, 2026 13:18
auto-merge was automatically disabled June 15, 2026 13:18

Pull request was converted to draft

Chri6s added 2 commits June 15, 2026 15:18
- Enable built-in theme switch component in AppHeader and default to
system preferences.
- Add safety logic to d3-color operations in chart components to prevent
runtime crashes with CSS variables.
Fixed coloring of widget close button, coloring of chart slice labels,
line charts and matched tests accordingly.
@Chri6s
Chri6s marked this pull request as ready for review June 15, 2026 13:19
@Chri6s

Chri6s commented Jul 6, 2026

Copy link
Copy Markdown
Author

@bjoernricks Can I request a review for this PR as there are new UI changes which could cause wrong coloring in specific views/icons etc.?

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.

1 participant