Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Sign up
Appearance settings

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.

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

Open
naikmubashir wants to merge 19 commits into layer5io:master
base: master
Choose a base branch
Loading
from naikmubashir:master

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.
 

Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
Copy link
Contributor

l5io commented Oct 21, 2025

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

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

Copy link
Member

please resolve the merge conflicts >

 

Signed-off-by: Naik Mubashir <124430100+naikmubashir@users.noreply.github.com>
Copy link
Contributor

l5io commented Oct 22, 2025

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

Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
Copy link
Contributor

l5io commented Oct 22, 2025

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

Copy link
Member

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 reacted with thumbs up emoji 

Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
Copy link
Contributor

l5io commented Oct 29, 2025

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

Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
Copy link
Contributor

l5io commented Oct 29, 2025

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

Copy link
Contributor

l5io commented Oct 31, 2025

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

Copy link
Member

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
😊

 

Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
Signed-off-by: Naik Mubashir <naikmubashir095@gmail.com>
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>
Copy link
Member

Any updates ?

Copy link
Contributor

@naikmubashir can you come to the next week's websites meeting, so we can finalise on this . Else I will go ahead to inspect the changes and make necessary updates to complete this PR

Copy link
Member

Merge conflicts...

Copy link
Contributor

Thank you for your contribution!
Let’s discuss this during the website call today (December 1st) at 8:00AM CT | 6:30PM IST.
Please add it as an agenda item to the meeting minutes
😊

 

Signed-off-by: Naik Mubashir <124430100+naikmubashir@users.noreply.github.com>
@leecalcote leecalcote removed the request for review from Namanv0509 December 3, 2025 16:14
Copy link
Contributor

@naikmubashir
closing this pr due to inactivity

Copy link
Contributor Author

naikmubashir commented Dec 23, 2025 •
edited
Loading

@kishore08-07 Can we just reopen this pr. I would to work on this further
@Rajesh-Nagarajan-11 @saurabhraghuvanshii

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

@Fireentity Fireentity Awaiting requested review from Fireentity

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

Assignees

No one assigned

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

AltStyle によって変換されたページ (->オリジナル) /