[Sistent] Add FormControl component documentation to Sistent components page by shteypandey28-hue · Pull Request #7445 · layer5io/layer5

shteypandey28-hue · 2026-03-02T13:42:41Z

Description

This PR adds documentation for the FormControl component to the Sistent components section.

The following has been implemented:

Overview page
Guidance page
Code examples page
Navigation entry in Sistent sidebar
Component registration in components listing
Gatsby route wrappers for proper routing

This ensures FormControl is properly documented similar to other Sistent components.

Closes #5935

Notes for Reviewers

Documentation follows the same structure as Stack and FormGroup components.
Tested locally using npm run develop.
All commits are signed.

Signed-off-by: shteypandey28-hue <shteypandey28@gmail.com>

shteypandey28-hue · 2026-03-02T13:59:40Z

I have made the form control component, kindly review it and merge.

l5io · 2026-03-02T14:00:34Z

🚀 Preview for commit e4b1f7c at: https://69a597b89f9e154a4fa40b3f--layer5.netlify.app

l5io · 2026-03-02T17:15:33Z

🚀 Preview for commit 89643c4 at: https://69a5c58659ef5364910479a8--layer5.netlify.app

Rajesh-Nagarajan-11 · 2026-03-02T17:17:11Z

src/sections/Projects/Sistent/components/formcontrol/code.js

+  Radio,
+} from "@sistent/sistent";
+import {
+  FormControl,


Please use the Sistent FormControl component, example -> import { FormControl } from "@sistent/sistent";.

changes have been made

Rajesh-Nagarajan-11 · 2026-03-02T17:31:25Z

Placeholder not works well

l5io · 2026-03-02T18:13:06Z

🚀 Preview for commit 7ef1216 at: https://69a5d2ee2b301d9f37ab5804--layer5.netlify.app

shteypandey28-hue · 2026-03-02T18:14:59Z

@Rajesh-Nagarajan-11 both the changes have been made , kindly go through it and tell if any further assistant is needed.

YASHMAHAKAL

@shteypandey28-hue Still no improvements :(

l5io · 2026-03-03T02:57:14Z

🚀 Preview for commit d74cd36 at: https://69a64dd2e3ffef73b38a39bf--layer5.netlify.app

rishiraj38 · 2026-03-03T03:10:59Z

@shteypandey28-hue Please revert the lint changes and when committing use --no-verify. Furthermore, the dropdown isn’t working for the overview and guidance section in your current commit.

Screen.Recording.2026-03-03.at.8.30.21.AM.mov

matrixkavi

Please ensure that you are changing only the lines you intend to change. A lot of changes are there in this PR as well as the previous PR 7444 where you have added formatting related changes to js files. It can be cumbersome to review and can result in unwanted changes creeping in unknowingly.

rishiraj38

Please revert the lint changes and when committing use --no-verify. Furthermore, the dropdown isn’t working for the overview and guidance section in your current commit.

l5io · 2026-03-03T10:38:06Z

🚀 Preview for commit d77413b at: https://69a6b9cc450c3964e2be8e4b--layer5.netlify.app

Add complete FormControl documentation with Overview, Guidance, and Code tabs. Includes examples for Select, Checkbox, and Radio controls with proper state management using controlled components. Signed-off-by: shteypandey28-hue <shteypandey28@gmail.com>

l5io · 2026-03-03T13:15:04Z

🚀 Preview for commit fdceeaa at: https://69a6dea1e3ffef8f648a3cfe--layer5.netlify.app

shteypandey28-hue · 2026-03-03T13:51:15Z

Made the changes , please review it and tell me if any further assistant is need .. if not merge it .

Signed-off-by: shteypandey28-hue <shteypandey28@gmail.com>

l5io · 2026-03-03T14:32:19Z

🚀 Preview for commit bc109f0 at: https://69a6f0a5ee811432e437f6a2--layer5.netlify.app

l5io · 2026-03-03T19:37:11Z

🚀 Preview for commit 1cfe82c at: https://69a73836a1f8cfb76337a94c--layer5.netlify.app

…-node.js Signed-off-by: shteypandey28-hue <shteypandey28@gmail.com>

l5io · 2026-03-03T20:01:20Z

🚀 Preview for commit 0674a67 at: https://69a73ddb7e83ec7b8b40820b--layer5.netlify.app

rishiraj38 · 2026-03-04T05:11:42Z

src/collections/news/2026/2026-01-21-docker-kanvas-kubernetes-dominance/index.mdx

@shteypandey28-hue Could you explain the changes made here?

T he maintainer already flagged them — In an earlier cleanup commit (Fix: remove unrelated news files from FormControl PR), those files were explicitly removed because they don't belong.
They're completely unrelated — docker-kanvas-infoq.webp and index.mdx are a news article about

@saurabhraghuvanshii Could you please take a look? I don’t think we need to remove that recent news article we added.

@shteypandey28-hue please revert this, and open Pr only releted to issue. No need to remove new articles and anything. Your work in this is only to add new sistent components.

rishiraj38 · 2026-03-04T05:12:14Z

...collections/news/2026/2026-01-21-docker-kanvas-kubernetes-dominance/docker-kanvas-infoq.webp

rishiraj38 · 2026-03-04T05:12:43Z

src/components/SistentNavigation/content.js

revert the lints

rishiraj38 · 2026-03-04T05:13:19Z

src/sections/Projects/Sistent/components/content.js

revert the lints

rishiraj38 · 2026-03-04T05:16:00Z

gatsby-node.js

            createPage({
              path: pagePath,
-              component: require.resolve(componentPath),
+              component: path.resolve(componentPath),


why this change

rishiraj38 · 2026-03-04T05:19:49Z

src/components/SistentNavigation/content.js

+  },
+
+  {
+    id: 121,


The navigation for FormControl should be placed above the FormControlLabel.

- Revert gatsby-node.js to use require.resolve (matching upstream) - Revert unintended lint reformatting in SistentNavigation/content.js - Revert unintended lint reformatting in components/content.js - Re-add FormControl navigation entries (IDs 121-123) in original style - Re-add FormControl components data entry (ID 37) in original style Signed-off-by: shteypandey28-hue <shteypandey28@gmail.com>

l5io · 2026-03-06T13:36:31Z

🚀 Preview for commit f06b149 at: https://69aad8117dc51e72c0166ff8--layer5.netlify.app

Bhumikagarggg · 2026-03-08T21:02:43Z

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

github-actions bot added the project/sistent label Mar 2, 2026

docs(sistent): add FormControl component documentation

e4b1f7c

Signed-off-by: shteypandey28-hue <shteypandey28@gmail.com>

shteypandey28-hue force-pushed the sistent-formcontrol-docs branch from d199951 to e4b1f7c Compare March 2, 2026 13:44

Merge branch 'master' into sistent-formcontrol-docs

89643c4

Rajesh-Nagarajan-11 requested changes Mar 2, 2026

View reviewed changes

YASHMAHAKAL reviewed Mar 2, 2026

View reviewed changes

shteypandey28-hue force-pushed the sistent-formcontrol-docs branch from c6b6e50 to d74cd36 Compare March 3, 2026 02:40

matrixkavi suggested changes Mar 3, 2026

View reviewed changes

rishiraj38 suggested changes Mar 3, 2026

View reviewed changes

shteypandey28-hue force-pushed the sistent-formcontrol-docs branch from d77413b to fdceeaa Compare March 3, 2026 12:57

shteypandey28-hue added 2 commits March 3, 2026 19:45

Fix: clean FormControl example and remove unintended formatting changes

014b158

Signed-off-by: shteypandey28-hue <shteypandey28@gmail.com>

Fix: remove unrelated news files from FormControl PR

bc109f0

Signed-off-by: shteypandey28-hue <shteypandey28@gmail.com>

github-actions bot added the area/news A noteworthy article, event, happening label Mar 3, 2026

github-actions bot added the area/site-config label Mar 3, 2026

docs: fix component page resolution by using absolute paths in gatsby…

0674a67

…-node.js Signed-off-by: shteypandey28-hue <shteypandey28@gmail.com>

shteypandey28-hue force-pushed the sistent-formcontrol-docs branch from 1cfe82c to 0674a67 Compare March 3, 2026 19:44

rishiraj38 suggested changes Mar 4, 2026

View reviewed changes

Conversation

shteypandey28-hue commented Mar 2, 2026

Description

Notes for Reviewers

Uh oh!

shteypandey28-hue commented Mar 2, 2026

Uh oh!

l5io commented Mar 2, 2026

Uh oh!

l5io commented Mar 2, 2026

Uh oh!

Rajesh-Nagarajan-11 Mar 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Rajesh-Nagarajan-11 commented Mar 2, 2026

Uh oh!

l5io commented Mar 2, 2026

Uh oh!

shteypandey28-hue commented Mar 2, 2026

Uh oh!

YASHMAHAKAL left a comment

Choose a reason for hiding this comment

Uh oh!

l5io commented Mar 3, 2026

Uh oh!

rishiraj38 commented Mar 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

matrixkavi left a comment

Choose a reason for hiding this comment

Uh oh!

rishiraj38 left a comment

Choose a reason for hiding this comment

Uh oh!

l5io commented Mar 3, 2026

Uh oh!

l5io commented Mar 3, 2026

Uh oh!

shteypandey28-hue commented Mar 3, 2026

Uh oh!

l5io commented Mar 3, 2026

Uh oh!

l5io commented Mar 3, 2026

Uh oh!

l5io commented Mar 3, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

l5io commented Mar 6, 2026

Uh oh!

Bhumikagarggg commented Mar 8, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

8 participants

Rajesh-Nagarajan-11 Mar 2, 2026 •

edited

Loading

rishiraj38 commented Mar 3, 2026 •

edited

Loading