Skip to content

feat(sistent): add Table to components documentation #7062

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

feat(sistent): add Table to components documentation #7062

wants to merge 14 commits into from

Conversation

@naikmubashir
Copy link
Contributor

@naikmubashir naikmubashir commented Oct 21, 2025 •
edited by Rajesh-Nagarajan-11
Loading

Description
This PR adds comprehensive documentation for the Table component (ResponsiveDataTable) to the Sistent components documentation page at layer5.io/projects/sistent/components. The documentation follows the established patterns used by other components like Button and includes complete examples, guidance, and code references.

Changes Made:

  • src/sections/Projects/Sistent/components/table/
    • index.js # Overview page with basic examples
    • guidance.js # Design guidelines and best practices
    • code.js # Comprehensive code examples
  • src/sections/Projects/content.js
  • src/components/SistentNavigation/content.js

This PR fixes #5962

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

@naikmubashir
feat(sistent): add Table to SistentNavigation
022e1f1 
Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
@naikmubashir
feat(sistent): add Table to component documentation
1cbbeec 
Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
@naikmubashir
feat(sistent): add Table usage guidance and best practices
bba7b31 
Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
@naikmubashir
feat(sistent): add Table interative code examples
1a8f7c9 
Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
@naikmubashir
feat(sistent): add Table to components data list
aa6b67c 
Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
@l5io
Copy link
Contributor

l5io commented Oct 21, 2025

🚀 Preview for commit aa6b67c at: https://68f7d6902dc5951120f39855--layer5.netlify.app

Rajesh-Nagarajan-11
Rajesh-Nagarajan-11 reviewed Oct 22, 2025
View reviewed changes
Copy link
Member

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

atleast add these too(in guidance) @naikmubashir :

  • What are options values...
  • How to enable column sort
  • what are multiselect and none select
  • disabled row
  • sub tables

@Rajesh-Nagarajan-11
Copy link
Member

Rajesh-Nagarajan-11 commented Oct 22, 2025

please resolve the merge conflicts >

@naikmubashir
Merge branch 'master' into master
9f354bf 
Signed-off-by: Naik Mubashir <124430100+naikmubashir@users.noreply.github.com>
@l5io
Copy link
Contributor

l5io commented Oct 22, 2025

🚀 Preview for commit 9f354bf at: https://68f8710ed357e4603f74c747--layer5.netlify.app

@naikmubashir
fix(sistent): fix eslint errors in SistentNavigation
11a1617 
Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
@l5io
Copy link
Contributor

l5io commented Oct 22, 2025

🚀 Preview for commit 11a1617 at: https://68f875ebee2407df2194c8ea--layer5.netlify.app

@Rajesh-Nagarajan-11
Copy link
Member

Rajesh-Nagarajan-11 commented Oct 27, 2025

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

@naikmubashir
merge new changes
33afef3 
Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
@l5io
Copy link
Contributor

l5io commented Oct 29, 2025

🚀 Preview for commit 33afef3 at: https://69029b3ea3e9bd4e3d717b1b--layer5.netlify.app

@naikmubashir
fix(sistent): fix new updates
13e39bf 
Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
@l5io
Copy link
Contributor

l5io commented Oct 29, 2025

🚀 Preview for commit 13e39bf at: https://69029eda1a935c538b76af63--layer5.netlify.app

@l5io
Copy link
Contributor

l5io commented Oct 31, 2025

🚀 Preview for commit 0ecfa69 at: https://6905041008c9c155405bcd50--layer5.netlify.app

@Rajesh-Nagarajan-11
Copy link
Member

Rajesh-Nagarajan-11 commented Nov 2, 2025

Thank you for your contribution!
Let’s discuss this during the website call tomorrow (November 3) at 5:30 PM IST | 7:00 AM CT.
Please add it as an agenda item to the meeting minutes
😊

@naikmubashir
fix: resolve merge conflict
068b501 
Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
@naikmubashir
Merge branch 'master' of https://github.com/layer5io/layer5
16cca18
@naikmubashir
fix: update code.js
b6dc275 
Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
@naikmubashir
fix(sistent): prevent SSR issues with Table component rendering
ef450e0 
Added client-side mounting check to ResponsiveDataTable components to fix
Gatsby static HTML build failure. The MUI theme.alpha function was causing
errors during SSR. Tables now only render after component mounts in browser.

Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 left review comments

@Namanv0509 Namanv0509 Awaiting requested review from Namanv0509

Assignees

No one assigned

Labels

project/sistent

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Sistent] Add Table component to the sistent components page

3 participants

@naikmubashir @l5io @Rajesh-Nagarajan-11