Skip to content

Enable dark mode - #846

Open
Madhurchandran wants to merge 2 commits into
kubernetes:mainfrom
Madhurchandran:enable-dark-mode
Open

Enable dark mode#846
Madhurchandran wants to merge 2 commits into
kubernetes:mainfrom
Madhurchandran:enable-dark-mode

Conversation

@Madhurchandran

@Madhurchandran Madhurchandran commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Summary

enable dark-mode styling across the contributor site and fixes several visual regressions found while testing the KEP and Community Groups pages.

Changes

  • Improve dark-mode styling for KEP page containers and borders.
  • Update KEP filter controls to use Bootstrap theme variables instead of hardcoded light-mode colors.
  • Improve KEP DataTables pagination styling in dark mode.
  • Refine Community Groups dark-mode styles:
  • Update KEP component colors to better match the dark theme.

Remaining work

  • Review Bootstrap 5.3.x changes and address any remaining styling regressions.
  • Replace remaining hardcoded colors and reduce/remove unnecessary !important usage.
  • squashing the commit after work done for cleaner history.

Testing

  • Built and served the site locally with Hugo.
  • Used browser DevTools to inspect computed styles and identify CSS specificity/override issues.

Related

Ref: #745

AI Disclosure
An AI assistant was used to help identify and fix issues during regression testing. All changes were reviewed and validated before submission.

@kubernetes-prow kubernetes-prow Bot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Aug 9, 2026
@kubernetes-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Madhurchandran
Once this PR has been reviewed and has the lgtm label, please assign palnabarun for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubernetes-prow kubernetes-prow Bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Aug 9, 2026
feat: resolve community groups dark mode regressions

feat(theme): resolve dark mode DataTables regressions

feat: improve dark mode theme toggle visibility

feat: improve KEP dark mode styling and organize styles

feat: replace remaining hardcoded colors with CSS vars across all SCSS files

feat: refine KEP table dark mode styles

feat: address review comments

feat: address review feedback

feat:Improve Community Groups dark theme readability

feat: refine dark mode component styling
@Madhurchandran

Copy link
Copy Markdown
Contributor Author

Hey @TineoC, I’ve been working through the dark mode PR and have done some cleanup to reduce unnecessary overrides and !important occurrences. What approach would you recommend for the remaining hardcoded colors?

@TineoC

TineoC commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

/hold

Until the RSS errors are resolved

@kubernetes-prow kubernetes-prow Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants