Skip to content

bugfix: css issues after 7297#7308

Open
liquidely wants to merge 3 commits intomainfrom
bugfix/css-issues-after-7297
Open

bugfix: css issues after 7297#7308
liquidely wants to merge 3 commits intomainfrom
bugfix/css-issues-after-7297

Conversation

@liquidely
Copy link
Contributor

Summary

  • Change headings color
  • Change tabs active background color

How did you test this change?

Live site and inspect tool

Copilot AI review requested due to automatic review settings March 26, 2026 20:30
@liquidely liquidely requested review from a team as code owners March 26, 2026 20:30
@liquidely liquidely mentioned this pull request Mar 26, 2026
6 tasks
Copy link
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

Fixes theme-related visual regressions introduced after change 7297 by adjusting heading and tab active-state colors to use appropriate, existing CSS variables for both light and dark themes.

Changes:

  • Update active tab styling in light theme (background + text color) to align with the page surface/background tokens.
  • Adjust heading text colors in light/dark themes to use the intended theme/neutral palette variables.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
stylesheets/commons/Tabs.scss Tweaks .tabs-static active tab colors in light theme to use --clr-background and a valid neutral text token.
stylesheets/commons/Headings.scss Updates heading text colors to use --clr-wiki-theme-20 (light) and --clr-secondary-90 (dark).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ElectricalBoy ElectricalBoy added the stylesheets Changes to stylesheets label Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stylesheets Changes to stylesheets

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants