-
-
Notifications
You must be signed in to change notification settings - Fork 18
Dark theme fixes and dashboard cards UI improvements #1642
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
5d04872
09d6832
f35a726
c116b41
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -109,8 +109,14 @@ export class DbTableAiPanelComponent implements OnInit, AfterViewInit, OnDestroy | |
|
|
||
| async ngAfterViewInit() { | ||
| const mermaid = await import('mermaid'); | ||
| const mermaidAPI: any = mermaid.default ?? mermaid; | ||
| const isDark = window.matchMedia('(prefers-color-scheme: dark)').matches; | ||
| mermaidAPI.initialize({ | ||
| startOnLoad: false, | ||
| theme: isDark ? 'dark' : 'default', | ||
| }); | ||
|
Comment on lines
+113
to
+117
|
||
| //@ts-expect-error dynamic load of mermaid | ||
| window.mermaid = mermaid.default ?? mermaid; | ||
| window.mermaid = mermaidAPI; | ||
| } | ||
|
|
||
| ngOnDestroy() { | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -150,12 +150,18 @@ | |||||||||||
| top: 9px; | ||||||||||||
| right: 3px; | ||||||||||||
| display: block; | ||||||||||||
| background: rgba(0, 0, 0, 0.87); | ||||||||||||
| background: var(--color-accentedPalette-500); | ||||||||||||
| border-radius: 50%; | ||||||||||||
| height: 5px; | ||||||||||||
| width: 5px; | ||||||||||||
| } | ||||||||||||
|
|
||||||||||||
| @media (prefers-color-scheme: dark) { | ||||||||||||
| .action_active::after { | ||||||||||||
| background: var(--color-accentedPalette-500); | ||||||||||||
| } | ||||||||||||
| } | ||||||||||||
|
|
||||||||||||
|
Comment on lines
+159
to
+164
|
||||||||||||
| @media (prefers-color-scheme: dark) { | |
| .action_active::after { | |
| background: var(--color-accentedPalette-500); | |
| } | |
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
UI copy uses title-case mid-sentence ("Show Demo..."). If the product copy standard is sentence case elsewhere, consider changing this to "Show demo panels for all members" for consistency/readability.