Skip to content

docs(table): remove JS comment from useTableSelection @example block#4324

Merged
josephfarina merged 1 commit into
mainfrom
navi/docs/storybook-compat-usetableselection
Jul 25, 2026
Merged

docs(table): remove JS comment from useTableSelection @example block#4324
josephfarina merged 1 commit into
mainfrom
navi/docs/storybook-compat-usetableselection

Conversation

@josephfarina

Copy link
Copy Markdown
Contributor

What

The getRowLabel JSDoc on useTableSelection had a // comment inside its @example code fence. Storybook's autodocs markdown parser can treat a // line as ending or corrupting the fence, which renders the remainder of the example as raw text rather than code.

Moved the "accessible names" explanation into the JSDoc description above the @example tag and reduced the code block to the single getRowLabel expression. No behavior or type change; JSDoc prose only.

Validation

  • pnpm --filter @astryxdesign/core typecheck:docs passes
  • @example scan reports no blank lines, JS comments, or bare > in the block

Night Watch — Doc Reviewer

Storybook autodocs can misparse a // comment inside an @example code
fence, rendering the rest of the block as raw text. Move the accessible
names note into the JSDoc description above the example and keep the
code block to a single expression.

Found during Night Watch doc review.
@vercel

vercel Bot commented Jul 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
astryx Ready Ready Preview, Comment Jul 25, 2026 9:17am

Request Review

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Meta Open Source bot. label Jul 25, 2026
github-actions Bot added a commit that referenced this pull request Jul 25, 2026
@github-actions

Copy link
Copy Markdown
Contributor

PR Analysis Report

📚 Storybook Preview

View Storybook for this PR
GitHub Pages may take up to a minute to hydrate after deploy.

🧪 Sandbox Preview

View Sandbox for this PR
GitHub Pages may take up to a minute to hydrate after deploy.

Modified Components

Table · View in Storybook
Metric Before After Delta
Bundle Size (ESM) N/A N/A N/A
Lines of Code N/A 1918 -
Complexity N/A Very High (101) -

Bundle Size Summary

Package Size (ESM) Size (CJS) Gzipped
@astryxdesign/core N/A 4.7KB 0B

Accessibility Audit

Status: 1 accessibility violation(s) found — 1 serious.

Table - 1 issue(s)
  • 🟠 serious: Ensure the contrast between foreground and background colors meets WCAG 2 AA minimum contrast ratio thresholds
    • Rule: color-contrast · Affects 4/24 stories · Learn more
    • WCAG: 1.4.3 (Level AA)

Generated by PR Enrichment workflow | Storybook | Sandbox | View full report

@josephfarina
josephfarina merged commit 3290b8a into main Jul 25, 2026
18 checks passed
@github-actions
github-actions Bot deleted the navi/docs/storybook-compat-usetableselection branch July 26, 2026 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant