Skip to content

Add menuchart#7674

Open
indevi wants to merge 6 commits into
layer5io:masterfrom
indevi:menucard
Open

Add menuchart#7674
indevi wants to merge 6 commits into
layer5io:masterfrom
indevi:menucard

Conversation

@indevi
Copy link
Copy Markdown
Contributor

@indevi indevi commented Apr 29, 2026

Description

This PR fixes #7662

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Shivam Tyagi <tyagishivam.dev@gmail.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 29, 2026

🚀 Preview deployment: https://layer5.io/pr-preview/pr-7674/

Note: Preview may take a moment (GitHub Pages deployment in progress). Please wait and refresh. Track deployment here

@Bhumikagarggg
Copy link
Copy Markdown
Contributor

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

@indevi
Copy link
Copy Markdown
Contributor Author

indevi commented May 3, 2026

@Bhumikagarggg sure added

Comment thread src/collections/sistent/components/menu/guidance.mdx Outdated
Signed-off-by: Shivam Tyagi <tyagishivam.dev@gmail.com>

import { useState } from "react";
import { Menu, MenuItem, MenuList, Button, Box, Typography, Divider, ListItemIcon, ListItemText } from "@sistent/sistent";
import { MoreVert as MoreVertIcon, Edit as EditIcon, Delete as DeleteIcon, Share as ShareIcon, Download as DownloadIcon, FileCopy as FileCopyIcon, Print as PrintIcon } from "@mui/icons-material";
Copy link
Copy Markdown
Member

@rishiraj38 rishiraj38 May 4, 2026

Choose a reason for hiding this comment

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

We’ve these icons exported from sistent. Please use sistent to import them and if you didn't find any icon exported from sistent open a sub-issue here and add them.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@indevi Any Updates??

Signed-off-by: Shivam Tyagi <tyagishivam.dev@gmail.com>
@Bhumikagarggg
Copy link
Copy Markdown
Contributor

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

@indevi
Copy link
Copy Markdown
Contributor Author

indevi commented May 11, 2026

@rishiraj38 @saurabhraghuvanshii requesting review please

@rishiraj38
Copy link
Copy Markdown
Member

@indevi, please make a commit so the preview can be built.

indevi added 2 commits May 11, 2026 22:01
Signed-off-by: Shivam Tyagi <tyagishivam.dev@gmail.com>
@indevi
Copy link
Copy Markdown
Contributor Author

indevi commented May 11, 2026

@rishiraj38 hope this is the preview commit requested

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.

Please check out this PR for fixing this detachment issue. (#7524)

Screen.Recording.2026-05-11.at.10.38.19.PM.mov

and Please center those menu buttons in overview and guidance

and also you have added too much content and it doesn't feel like they are in sequence please improve on that too

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Sistent] Add Menu component to the sistent components page

4 participants