Skip to content

feat: implement dynamic routing for Sistent components#7408

Open
rishiraj38 wants to merge 28 commits intolayer5io:masterfrom
rishiraj38:feat/sistent-refactor-mdx
Open

feat: implement dynamic routing for Sistent components#7408
rishiraj38 wants to merge 28 commits intolayer5io:masterfrom
rishiraj38:feat/sistent-refactor-mdx

Conversation

@rishiraj38
Copy link
Contributor

@rishiraj38 rishiraj38 commented Feb 5, 2026 •
edited
Loading

Description

This PR refactors the Sistent component documentation system to migrate from a hardcoded data source (content.js) to a dynamic, MDX-driven approach using Gatsby's data layer. This change improves scalability and maintainability of the design system documentation.

Components to Refactor:

  • accordion
  • avatar
  • avatar-group
  • backdrop
  • badge
  • box
  • button
  • button-group
  • chip
  • circularprogress
  • collapase
  • container
  • dialog
  • divider
  • drawer
  • formcontrollabel
  • grid
  • iconbutton
  • icons
  • link
  • list
  • modal
  • pagination
  • paper
  • popper
  • radiogroup
  • select
  • stack
  • stepper
  • switch
  • tabs
  • text-field
  • text-input
  • toggle-button
  • toolbar
  • tooltip

Signed commits

  • Yes, I signed my commits.

@github-actions github-actions bot added area/projects An issue relating to Layer5 initiatives (projects) area/site-config project/sistent labels Feb 5, 2026
@rishiraj38
feat: implement dynamic routing for Sistent components
b7bc06b 
Migrate Sistent components to MDX-based dynamic routing. This removes the hardcoded content.js and replaces it with a scalable, GraphQL-driven approach suitable for large component libraries.

Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@rishiraj38 rishiraj38 force-pushed the feat/sistent-refactor-mdx branch from 860abe1 to b7bc06b Compare February 5, 2026 13:12
@rishiraj38
Copy link
Contributor Author

rishiraj38 commented Feb 5, 2026

@Rajesh-Nagarajan-11 Could you please review and give some feedback?

@l5io
Copy link
Contributor

l5io commented Feb 5, 2026

🚀 Preview for commit b7bc06b at: https://69849a8800c6f13b7bbb8e77--layer5.netlify.app

@rishiraj38
feat: standardize sistent component layouts and refactor to MDX
e33a175 
Migrated all Sistent components to dynamic MDX routing. Standardized component header layouts by moving descriptions to frontmatter. Cleaned up legacy content.js and deprecated code.

Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@rishiraj38
Copy link
Contributor Author

rishiraj38 commented Feb 6, 2026

@Rajesh-Nagarajan-11 Could you please review this? Is this the correct way?

@rishiraj38
Copy link
Contributor Author

rishiraj38 commented Feb 6, 2026

This needs a bit more focus. I just need a review to see if I’m moving in the right direction or if I’m increasing your workload. 😅

@Rajesh-Nagarajan-11
Copy link
Member

Rajesh-Nagarajan-11 commented Feb 6, 2026

present it in monday's website meeting please

@l5io
Copy link
Contributor

l5io commented Feb 6, 2026

🚀 Preview for commit e33a175 at: https://6985ed49e6962ec326ad1365--layer5.netlify.app

@rishiraj38
Copy link
Contributor Author

rishiraj38 commented Feb 6, 2026

present it in monday's website meeting please

OK, but what was your initial thought about this? Does it look good?

@rishiraj38 rishiraj38 closed this Feb 6, 2026
@rishiraj38 rishiraj38 reopened this Feb 6, 2026
@rishiraj38
Copy link
Contributor Author

rishiraj38 commented Feb 6, 2026

Like all components in the sistent, each has three sub-components: overview, guidance and code. I tried adding everything into a single index.mdx file but it became too large. It was too difficult to find and fix anything so I divided them into three separate files.

@l5io
Copy link
Contributor

l5io commented Feb 6, 2026

🚀 Preview for commit e33a175 at: https://698605f323fdac1a22f3038b--layer5.netlify.app

@Bhumikagarggg
Copy link
Contributor

Bhumikagarggg commented Feb 8, 2026

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

@l5io
Copy link
Contributor

l5io commented Feb 13, 2026

🚀 Preview for commit 83035c1 at: https://698ece6c6623f82919a30f8f--layer5.netlify.app

@rishiraj38
Revert linting changes to focus on logic
42b3496 
Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@rishiraj38 rishiraj38 force-pushed the feat/sistent-refactor-mdx branch from 4d638b7 to 42b3496 Compare February 16, 2026 14:17
@rishiraj38
Merge remote-tracking branch 'origin/master' into feat/sistent-refact…
d10d755 
…or-mdx

# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
@rishiraj38
Merge branch 'master' into feat/sistent-refactor-mdx
43f7ff8
@l5io
Copy link
Contributor

l5io commented Feb 16, 2026

🚀 Preview for commit 43f7ff8 at: https://69932c5945919983b1785c12--layer5.netlify.app

@rishiraj38
refactor: inline Accordion demos and clean gatsby-node.js
5c3ca69 
Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@rishiraj38
Copy link
Contributor Author

rishiraj38 commented Feb 16, 2026

@Rajesh-Nagarajan-11 I understand I couldn’t explain everything because of a sore throat which makes speaking difficult. This will make reviewing challenging. Could you please review one component, the accordion, and provide some feedback? This will help me adjust all other components similarly.

@Rajesh-Nagarajan-11
Copy link
Member

Rajesh-Nagarajan-11 commented Feb 16, 2026

Did you use AI to migrate contents ?

@Rajesh-Nagarajan-11
Copy link
Member

Rajesh-Nagarajan-11 commented Feb 16, 2026 •
edited
Loading

If so no issues , but please double check the doc contents are replica , becuz AI might change the content of documention

@l5io
Copy link
Contributor

l5io commented Feb 16, 2026

🚀 Preview for commit 5c3ca69 at: https://69933d87b32a291bd40f03b8--layer5.netlify.app

@rishiraj38
Copy link
Contributor Author

rishiraj38 commented Feb 18, 2026 •
edited
Loading

I made the commit without -- no verify that why lints change added

@rishiraj38
updating and fixing all errors and also updated backdrop
d116802 
Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@rishiraj38 rishiraj38 force-pushed the feat/sistent-refactor-mdx branch from cd91a01 to d116802 Compare February 18, 2026 16:56
@rishiraj38
restore lost progress: fix imports and re-apply collapsible
e34744e 
Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@rishiraj38
fix: add missing collapsible prop to usage and installation pages
6f38801 
Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@l5io
Copy link
Contributor

l5io commented Feb 18, 2026

🚀 Preview for commit 6f38801 at: https://6995f6bfa0e8a119f5f8d513--layer5.netlify.app

@rishiraj38
style: lower sidebar navigation position
b14f050 
Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@l5io
Copy link
Contributor

l5io commented Feb 18, 2026

🚀 Preview for commit b14f050 at: https://69960048104b0643307304cb--layer5.netlify.app

@rishiraj38
revertCTA
d0515e0 
Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@rishiraj38
revertBanner
4fd3a6f 
Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@rishiraj38
revertCTAmeet
1697d3e 
Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@l5io
Copy link
Contributor

l5io commented Feb 18, 2026

🚀 Preview for commit 1697d3e at: https://699611965749bb008d07b81a--layer5.netlify.app

@l5io
Copy link
Contributor

l5io commented Feb 19, 2026

🚀 Preview for commit 7993cc0 at: https://699697711fc39d7a54954e09--layer5.netlify.app

@rishiraj38
updated:BadageandBox
c3d6684 
Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@Rajesh-Nagarajan-11
Copy link
Member

Rajesh-Nagarajan-11 commented Feb 19, 2026 •
edited
Loading

I suggest you to bump into single commit , we need to reduce the commit as much as possible

@l5io
Copy link
Contributor

l5io commented Feb 19, 2026

🚀 Preview for commit c3d6684 at: https://69973b4119062e11d67a3665--layer5.netlify.app

@l5io
Copy link
Contributor

l5io commented Feb 20, 2026

🚀 Preview for commit a7f8287 at: https://6998b71d13148b1be1c669a3--layer5.netlify.app

@rishiraj38
style(sistent): refactor Dialog, Divider, Drawer, FormControlLabel, G…
973a1c4 
…rid, IconButton, Icons, Link components

Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@l5io
Copy link
Contributor

l5io commented Feb 21, 2026

🚀 Preview for commit 973a1c4 at: https://6999e375e8085e1d208fffb1--layer5.netlify.app

@Bhumikagarggg
Copy link
Contributor

Bhumikagarggg commented Feb 22, 2026

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

@rishiraj38
fix(programs): lazy initialize activeOption state; Updated components…
68e15a7 
…: Select, Stack, List, Modal, Pagination, Popper, RadioGroup

Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@l5io
Copy link
Contributor

l5io commented Feb 22, 2026

🚀 Preview for commit 68e15a7 at: https://699b4056e8085e01368fff6a--layer5.netlify.app

@l5io
Copy link
Contributor

l5io commented Feb 24, 2026

🚀 Preview for commit 899d577 at: https://699ddae2021b99332359a5ec--layer5.netlify.app

@rishiraj38
fix(programs): lazy initialize activeOption state
8dc8186 
Updated text-input, text-field, toggle-button, toolbar, tooltip, and stack components

Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@rishiraj38
Copy link
Contributor Author

rishiraj38 commented Feb 26, 2026

@Rajesh-Nagarajan-11 Done with all the components.

@l5io
Copy link
Contributor

l5io commented Feb 26, 2026

🚀 Preview for commit 8dc8186 at: https://69a062bba89eb8357c326a87--layer5.netlify.app

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

Reviewers

Copilot code review Copilot Copilot left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

area/projects An issue relating to Layer5 initiatives (projects) area/site-config project/sistent

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@rishiraj38 @l5io @Rajesh-Nagarajan-11 @Bhumikagarggg