Skip to content

docs: say the panel is dark only and drop the light scheme checks - #193

Merged
erkamyaman merged 1 commit into
santoshyadavdev:mainfrom
erkamyaman:docs/panel-dark-only
Oct 1, 2026
Merged

erkamyaman merged 1 commit into
santoshyadavdev:mainfrom
erkamyaman:docs/panel-dark-only

Conversation

@erkamyaman

Copy link
Copy Markdown
Collaborator

The panel only has a dark theme, but CONTRIBUTING.md, the development docs, the devtools-verify skill, the a11y-reviewer role and pnpm test:axe all asked for a light color scheme check, which tested the same dark page again. This settles #181 on dark only for now:

Fixes #181

Checks

  • pnpm test:axe, pnpm format:check, pnpm skills:check and the docs build pass

The contributor docs, the verify skill, the a11y reviewer role and
test:axe all asked for a light scheme check, but the panel only has a
dark theme, so that check tested the same page twice. They now say the
panel is dark only, test:axe runs the dark scheme, and the UI
guidelines link the light theme feature in santoshyadavdev#192.

Fixes santoshyadavdev#181
@github-actions github-actions Bot added area: docs The documentation site area: ci Workflows, hooks and repository tooling labels Oct 1, 2026
@coderabbitai

coderabbitai Bot commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

Next included review available in 56 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available. Your 60 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Essentials

Run ID: a3d96234-68c6-45b8-8cef-00457347620b

📥 Commits

Reviewing files that changed from the base of the PR and between 28e51ce and 8ab5f0d.

📒 Files selected for processing (6)
  • .claude/agents/a11y-reviewer.md
  • .claude/skills/devtools-verify/SKILL.md
  • CONTRIBUTING.md
  • apps/docs/src/content/contributing/development.md
  • docs/contributing/ui-guidelines.md
  • scripts/panel-axe.mjs
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Comment @coderabbitai help to get the list of available commands.

@nx-cloud

nx-cloud Bot commented Oct 1, 2026

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit 8ab5f0d

Command Status Duration Result
nx affected -t test build ✅ Succeeded 1m 13s View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗


☁️ Nx Cloud last updated this comment at 2026-10-01 09:47:47 UTC

@erkamyaman
erkamyaman merged commit cd5bfd2 into santoshyadavdev:main Oct 1, 2026
6 checks passed
erkamyaman added a commit to erkamyaman/angular-devtools that referenced this pull request Oct 1, 2026
Brings in santoshyadavdev#189, santoshyadavdev#191, santoshyadavdev#193 and santoshyadavdev#194. Keeps the one-path loop paragraph next to the new Routes lab sentence in the router docs, adds the santoshyadavdev#194 upgrade note to the 0.0.6 changelog, and rebuilds extension/ui.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ci Workflows, hooks and repository tooling area: docs The documentation site

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Panel: support a light theme, or drop the light scheme check from the contributor docs

1 participant