Skip to content

docs(doc-1409): constrain Highlight colors to brand palette and fix :target step layout#2127

Open
djwfyi wants to merge 3 commits into
mainfrom
DOC-1409/highlight-color
Open

docs(doc-1409): constrain Highlight colors to brand palette and fix :target step layout#2127
djwfyi wants to merge 3 commits into
mainfrom
DOC-1409/highlight-color

Conversation

@djwfyi
Copy link
Copy Markdown
Collaborator

@djwfyi djwfyi commented May 14, 2026

Content Description

Constrains the <Highlight> component to the approved brand color palette and fixes a layout bug where step numbers were displaced when navigating to an anchor link pointing to a heading inside a Flow component.

Preview Link

Internal Reference

Closes DOC-1409

AI review: mention @claude in a comment to request a review or changes. See CONTRIBUTING.md for available commands.

FORK LIMITATION: @claude does not work on pull requests opened from forks. GitHub Actions cannot access the required secrets for fork-originated PRs. To use AI review, push your branch directly to this repository.

@netlify /docs

…target step layout bug

- replace open-ended color prop with constrained set: secondary (#FFE0CC),
  success, info, warning, danger; default falls back to #E6E7E9 (divider gray)
- text color is always #050B24 per brand guidelines
- add console.warn for unrecognized color values
- rename color="green" → color="secondary" across all MDX usages
- add flow-step global class to Flow <li> elements and exclude it from the
  :target sibling highlight selectors to fix step number displacement on
  anchor navigation

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@djwfyi djwfyi requested a review from a team as a code owner May 14, 2026 19:06
@netlify
Copy link
Copy Markdown

netlify Bot commented May 14, 2026

Deploy Preview for vcluster-docs-site ready!

Name Link
🔨 Latest commit 78df0bc
🔍 Latest deploy log https://app.netlify.com/projects/vcluster-docs-site/deploys/6a062d5c0233b300075985a2
😎 Deploy Preview https://deploy-preview-2127--vcluster-docs-site.netlify.app/docs
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@djwfyi djwfyi self-assigned this May 14, 2026
@github-actions
Copy link
Copy Markdown
Contributor

djwfyi and others added 2 commits May 14, 2026 16:02
…ned docs

version-0.34.0 is the canonical URL target (lastVersion) and still uses
color="green" in its fragments. Without this alias the color falls back
to the default gray. Versioned docs are auto-backported so they cannot
be edited directly.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…nsole.warn, add skill docs

- switch success/info/warning/danger to --ifm-color-*-lightest for
  accessibility — all variants now use light tints with black text
- remove console.warn (silent fallback is sufficient)
- document Highlight usage, color options, and legacy green alias in
  the vcluster-docs-writer skill reference

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant