Skip to content

feat: add FormGroup component documentation to sistent components page#7666

Open
Xynash wants to merge 2 commits intolayer5io:masterfrom
Xynash:sistent-formgroup-docs
Open

feat: add FormGroup component documentation to sistent components page#7666
Xynash wants to merge 2 commits intolayer5io:masterfrom
Xynash:sistent-formgroup-docs

Conversation

@Xynash
Copy link
Copy Markdown
Contributor

@Xynash Xynash commented Apr 26, 2026

Description
This PR closes #7663. Adds documentation for the FormGroup component to the Sistent components page.

Three MDX files created at src/collections/sistent/components/formgroup/:

  • index.mdx — Overview and usage of FormGroup
  • guidance.mdx — Best practices, layout variants, and accessibility guidelines
  • code.mdx — Code examples with ThemeWrapper and SistentThemeProvider

Follows the same structure as existing components like Button and Card.

Notes for Reviewers

  • Vertical and horizontal layout examples included
  • Switch component usage documented
  • Accessibility patterns with FormControl and FormLabel covered
  • All CodeBlock snippets use SistentThemeProvider

Signed commits

  • Yes, I signed my commits.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 26, 2026

Preview deployment for PR #7666 removed.

This PR preview was automatically pruned because we keep only the 6 most recently updated previews on GitHub Pages to stay within deployment size limits.

If needed, push a new commit to this PR to generate a fresh preview.

@Xynash
Copy link
Copy Markdown
Contributor Author

Xynash commented Apr 26, 2026

This PR is ready for review! All three tabs are rendering correctly on the preview .

Comment thread src/collections/sistent/components/formgroup/guidance.mdx Outdated
Comment thread src/collections/sistent/components/formgroup/code.mdx Outdated
@Bhumikagarggg
Copy link
Copy Markdown
Contributor

@Xynash Thank you for your contribution! Let's discuss this during the website call tomorrow at 5:30 PM IST | 7 AM CST Add it as an agenda item to the meeting minutes, if you would 🙂

@Daniel-1600
Copy link
Copy Markdown
Contributor

Hi @Xynash thanks for working on this issue, could you squash all your commits into one for easier review thanks!

Signed-off-by: Ansh Sharma <anshsharma161105@gmail.com>
@Xynash
Copy link
Copy Markdown
Contributor Author

Xynash commented Apr 29, 2026

Hi @Xynash thanks for working on this issue, could you squash all your commits into one for easier review thanks!

Done! Squashed all commits into one .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Sistent] Add FormGroup component to the sistent components page

4 participants