Skip to content

fix: [docs] sync props tables with components - #971

Draft
Shreyag02 wants to merge 1 commit into
mainfrom
fix/docs-props-sync
Draft

Shreyag02 wants to merge 1 commit into
mainfrom
fix/docs-props-sync

Conversation

@Shreyag02

@Shreyag02 Shreyag02 commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • List.Item has no align prop, so the docs no longer list it and the "Aligning rows" example is removed. List lists its required children.

  • Indicator documents aria-label instead of ariaLabel.

  • CodeBlock.LanguageSelect takes the props of a single-selection Select root, so its table lists open, defaultOpen, onOpenChange and disabled instead of className.

  • Props that the components accept but the docs left out: pickerGroupClassName on RangePicker, optionalText and requiredText on Field.Label, disabled on Chip, decorative on FloatingActions.Separator, leadingIcon on Select.Item and toastManager on Toast.Provider.

  • Deprecated props (classNames, and the calendar inputProps, inputsProps, calendarProps and popoverProps) stay out of the tables.

  • I have followed the pull request section of the contributing guide.

@Shreyag02
Shreyag02 added this pull request to stack #972 October 12, 2026 07:32
@vercel

vercel Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
apsara Ready Ready Preview Oct 12, 2026 7:36am UTC

@coderabbitai

coderabbitai Bot commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 58 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 788ee8a0-37ea-4b22-b9dd-85b3772e10ff

📥 Commits

Reviewing files that changed from the base of the PR and between fdc5aa0 and acc605a.


📒 Files selected for processing (13)
  • apps/www/src/content/docs/components/calendar/props.ts
  • apps/www/src/content/docs/components/chip/props.ts
  • apps/www/src/content/docs/components/code-block/index.mdx
  • apps/www/src/content/docs/components/code-block/props.ts
  • apps/www/src/content/docs/components/field/props.ts
  • apps/www/src/content/docs/components/floating-actions/props.ts
  • apps/www/src/content/docs/components/indicator/index.mdx
  • apps/www/src/content/docs/components/indicator/props.ts
  • apps/www/src/content/docs/components/list/demo.ts
  • apps/www/src/content/docs/components/list/index.mdx
  • apps/www/src/content/docs/components/list/props.ts
  • apps/www/src/content/docs/components/select/props.ts
  • apps/www/src/content/docs/components/toast/props.ts

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@pkg-pr-new

pkg-pr-new Bot commented Oct 12, 2026

Copy link
Copy Markdown

Open in StackBlitz

pnpm add https://pkg.pr.new/@raystack/apsara@971

commit: acc605a

@Shreyag02
Shreyag02 removed this pull request from stack #972 October 12, 2026 07:57
@Shreyag02
Shreyag02 changed the base branch from feat/export-prop-types to main October 12, 2026 07:57
@Shreyag02 Shreyag02 self-assigned this Oct 12, 2026
@Shreyag02 Shreyag02 changed the title Fix/docs props sync fix: [docs] sync props tables with components Oct 12, 2026
@Shreyag02
Shreyag02 marked this pull request as draft October 12, 2026 09:55

This branch was successfully deployed

1 active deployment
Preview — acc605a4 Deployed Oct 12, 2026 by vercel[bot]
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