Skip to content

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

Closed
naikmubashir wants to merge 22 commits intolayer5io:masterfrom
naikmubashir:master
Closed

feat(sistent): add Table to components documentation#7062
naikmubashir wants to merge 22 commits intolayer5io:masterfrom
naikmubashir:master

Conversation

@naikmubashir
Copy link
Copy Markdown
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
Copy Markdown
Member

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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member

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
Copy Markdown
Member

l5io commented Oct 22, 2025

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

@Rajesh-Nagarajan-11
Copy link
Copy Markdown
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
Copy Markdown
Member

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
Copy Markdown
Member

l5io commented Oct 29, 2025

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

@l5io
Copy link
Copy Markdown
Member

l5io commented Oct 31, 2025

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

@Rajesh-Nagarajan-11
Copy link
Copy Markdown
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>
@Rajesh-Nagarajan-11
Copy link
Copy Markdown
Member

Rajesh-Nagarajan-11 commented Nov 13, 2025

Any updates ?

@Namanv0509
Copy link
Copy Markdown
Contributor

Namanv0509 commented Nov 26, 2025

@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

@leecalcote
Copy link
Copy Markdown
Member

leecalcote commented Dec 1, 2025

Merge conflicts...

@Fireentity
Copy link
Copy Markdown
Contributor

Fireentity commented Dec 1, 2025

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
😊

@naikmubashir
Merge branch 'master' into master
d07d998 
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
@Fireentity Fireentity self-requested a review December 7, 2025 14:50
@kishore08-07
Copy link
Copy Markdown
Contributor

kishore08-07 commented Dec 15, 2025

@naikmubashir
closing this pr due to inactivity

@naikmubashir
Copy link
Copy Markdown
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

@kishore08-07 kishore08-07 reopened this Dec 23, 2025
saurabhraghuvanshii and others added 4 commits December 23, 2025 15:32
@saurabhraghuvanshii
Merge branch 'master' into master
7ce5fbf
@naikmubashir
Merge branch 'master' into master
5cda73a 
Signed-off-by: Naik Mubashir <124430100+naikmubashir@users.noreply.github.com>
@l5io
Updated feature data from spreadsheet
b311a47 
Signed-off-by: l5io <ci@layer5.io>
@l5io
Updated feature data from spreadsheet
e76c3e2 
Signed-off-by: l5io <ci@layer5.io>
rishiraj38
rishiraj38 reviewed Apr 9, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@rishiraj38 rishiraj38 left a comment

Choose a reason for hiding this comment

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

Thanks @naikmubashir, could you please take a look at the updated version of this issue? #7607

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 23, 2026

PR Preview Action v1.6.3
Preview removed because the pull request was closed.
2026-04-23 09:09 UTC

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

+1 more reviewer

@rishiraj38 rishiraj38 rishiraj38 left review comments

Reviewers whose approvals may not affect merge requirements

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

9 participants

@naikmubashir @l5io @Rajesh-Nagarajan-11 @Namanv0509 @leecalcote @Fireentity @kishore08-07 @rishiraj38 @saurabhraghuvanshii