Skip to content

feat(styles): align product section colors with mergify.com#11237

Merged
mergify[bot] merged 1 commit intomainfrom
devs/jd/worktree-color-product/align-product-section-colors-mergify-com--b908772f
Apr 23, 2026
Merged

feat(styles): align product section colors with mergify.com#11237
mergify[bot] merged 1 commit intomainfrom
devs/jd/worktree-color-product/align-product-section-colors-mergify-com--b908772f

Conversation

@jd
Copy link
Copy Markdown
Member

@jd jd commented Apr 22, 2026

Updates the existing section accent hex values (merge-queue, ci-insights,
merge-protections) to match the canonical product palette defined in
mergify.com's src/lib/products.ts, and adds section colors for the two
remaining products that didn't have them: test-insights (orange) and
stacks (coral).

  • Wires section-test-insights and section-stacks body classes in
    BaseLayout so product pages get their accent applied to the ToC.
  • Adds matching nav-group-test-insights and nav-group-stacks classes to
    NavGroup, following the existing left-sidebar pattern (transparent
    hover, neutral gray for the active page). The product accent itself
    is applied via the ToC header-link rules keyed off the body
    section-* class, not the sidebar.

Copilot AI review requested due to automatic review settings April 22, 2026 18:18
@mergify mergify Bot had a problem deploying to Mergify Merge Protections April 22, 2026 18:18 Failure
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 22, 2026

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 🤖 Continuous Integration

Wonderful, this rule succeeded.
  • all of:
    • check-success = build
    • check-success = lint
    • check-success = test
    • any of:
      • check-success = test-broken-links
      • label = ignore-broken-links
    • any of:
      • check-success=Cloudflare Pages
      • -head-repo-full-name~=^Mergifyio/

🟢 👀 Review Requirements

Wonderful, this rule succeeded.
  • any of:
    • #approved-reviews-by >= 2
    • author = dependabot[bot]
    • author = mergify-ci-bot

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?:

🟢 🔎 Reviews

Wonderful, this rule succeeded.
  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

🟢 📕 PR description

Wonderful, this rule succeeded.
  • body ~= (?ms:.{48,})

@jd jd marked this pull request as ready for review April 22, 2026 18:18
@mergify mergify Bot requested a review from a team April 22, 2026 18:20
Copy link
Copy Markdown

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

Aligns the docs site’s product-section accent colors with the canonical mergify.com product palette, and extends section theming to the remaining products so the ToC and sidebar can be styled consistently per product section.

Changes:

  • Updated existing product accent hex values (merge-queue, ci-insights, merge-protections) and added new palette entries for test-insights and stacks.
  • Added section-specific ToC styling blocks for test-insights and stacks.
  • Wired new section-* body classes in BaseLayout and nav-group-* classes in NavGroup for the new product routes.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/styles/index.css Updates/extends section accent CSS variables and adds section-specific ToC/nav styling for test-insights and stacks.
src/layouts/BaseLayout.astro Adds section-test-insights and section-stacks body classes based on route prefix.
src/components/LeftSidebar/NavGroup.astro Adds nav-group-test-insights and nav-group-stacks classes based on nav group path prefix.

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

Comment thread src/styles/index.css
Updates the existing section accent hex values (merge-queue, ci-insights,
merge-protections) to match the canonical product palette defined in
mergify.com's src/lib/products.ts, and adds section colors for the two
remaining products that didn't have them: test-insights (orange) and
stacks (coral).

- Wires section-test-insights and section-stacks body classes in
  BaseLayout so product pages get their accent applied to the ToC.
- Adds matching nav-group-test-insights and nav-group-stacks classes to
  NavGroup, following the existing left-sidebar pattern (transparent
  hover, neutral gray for the active page). The product accent itself
  is applied via the ToC header-link rules keyed off the body
  section-* class, not the sidebar.

Change-Id: Ib908772fed46af5eb9bbf6d7f8a283151e3989d2
@jd jd force-pushed the devs/jd/worktree-color-product/align-product-section-colors-mergify-com--b908772f branch from f744b22 to 42d1712 Compare April 22, 2026 18:39
@jd
Copy link
Copy Markdown
Member Author

jd commented Apr 22, 2026

Revision history

# Type Changes Date
1 initial f744b22 2026-04-22 18:39 UTC
2 rebase f744b22 → 42d1712 2026-04-22 18:39 UTC

@mergify mergify Bot deployed to Mergify Merge Protections April 22, 2026 18:40 Active
@mergify mergify Bot requested a review from a team April 23, 2026 07:21
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 23, 2026

Merge Queue Status

This pull request spent 2 minutes 32 seconds in the queue, including 2 minutes 12 seconds running CI.

Required conditions to merge

mergify Bot added a commit that referenced this pull request Apr 23, 2026
@mergify mergify Bot added the queued label Apr 23, 2026
@mergify mergify Bot merged commit 23bc91d into main Apr 23, 2026
9 of 14 checks passed
@mergify mergify Bot deleted the devs/jd/worktree-color-product/align-product-section-colors-mergify-com--b908772f branch April 23, 2026 09:15
@mergify mergify Bot removed the queued label Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants