Skip to content

feat(ui): add dark theme#843

Merged
AndrejsPon00 merged 3 commits into
mainfrom
kof-ui-dark-theme
Mar 9, 2026
Merged

feat(ui): add dark theme#843
AndrejsPon00 merged 3 commits into
mainfrom
kof-ui-dark-theme

Conversation

@AndrejsPon00
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a light/dark theme toggle to the Collector UI and wires themed rendering into JSON viewers so the UI can support a dark theme mode.

Changes:

  • Introduces a ThemeProvider that applies light/dark classes and persists preference in localStorage.
  • Adds a ModeToggle control in the sidebar and wraps the app with the theme provider.
  • Refactors JSON rendering to use a themed CustomJsonView wrapper and updates call sites.

Reviewed changes

Copilot reviewed 12 out of 13 changed files in this pull request and generated 10 comments.

Show a summary per file
File Description
kof-operator/webapp/collector/src/components/shared/tabs/RawJsonTab.tsx Refactors JSON view into a themed wrapper component and adds optional text shortening.
kof-operator/webapp/collector/src/components/shared/ThemeProvider.tsx New theme context/provider that persists theme and applies root classes.
kof-operator/webapp/collector/src/components/shared/ModeToggle.tsx New icon-only button to toggle between light/dark themes.
kof-operator/webapp/collector/src/components/shared/JsonViewCard.tsx Switches JSON card rendering to use the themed JSON view wrapper.
kof-operator/webapp/collector/src/components/pages/victoriaPage/victoria-details/VictoriaDetailsPage.tsx Updates raw JSON tab to use the new JSON view wrapper.
kof-operator/webapp/collector/src/components/pages/dashboards/DetailTabs.tsx Updates raw JSON detail tab to use the new JSON view wrapper.
kof-operator/webapp/collector/src/components/pages/collectorPage/components/collector-details/CollectorContent.tsx Updates collector raw JSON rendering to use the new JSON view wrapper.
kof-operator/webapp/collector/src/components/features/TargetsList.tsx Uses the new JSON view wrapper for per-target JSON expansion rows.
kof-operator/webapp/collector/src/components/features/MainPage.tsx Removes hardcoded white background to allow theme-controlled backgrounds.
kof-operator/webapp/collector/src/components/features/AppSidebar.tsx Adds the theme mode toggle to the sidebar layout.
kof-operator/webapp/collector/src/App.tsx Wraps the app in ThemeProvider so theme state is available globally.
kof-operator/webapp/collector/package.json Adds a new dependency entry related to Radix UI.
kof-operator/webapp/collector/package-lock.json Locks newly added dependency tree.
Files not reviewed (1)
  • kof-operator/webapp/collector/package-lock.json: Language not supported

You can also share your feedback on Copilot code review. Take the survey.

Comment thread kof-operator/webapp/collector/src/components/shared/ModeToggle.tsx Outdated
Comment thread kof-operator/webapp/collector/package.json
Comment thread kof-operator/webapp/collector/src/components/shared/ModeToggle.tsx Outdated
Comment thread kof-operator/webapp/collector/src/components/shared/tabs/RawJsonTab.tsx Outdated
Comment thread kof-operator/webapp/collector/src/components/shared/ThemeProvider.tsx Outdated
Comment thread kof-operator/webapp/collector/src/components/shared/tabs/RawJsonTab.tsx Outdated
Comment thread kof-operator/webapp/collector/src/components/shared/ThemeProvider.tsx Outdated
Comment thread kof-operator/webapp/collector/src/components/shared/ThemeProvider.tsx Outdated
@AndrejsPon00 AndrejsPon00 merged commit 778bfa2 into main Mar 9, 2026
8 checks passed
@AndrejsPon00 AndrejsPon00 deleted the kof-ui-dark-theme branch March 9, 2026 13:58
@github-project-automation github-project-automation Bot moved this to Done in k0rdent Mar 9, 2026
gmlexx pushed a commit that referenced this pull request Mar 27, 2026
* feat(ui): add dark theme
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants