Skip to content

Added Menu Documentation #7087

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 3 commits into
base: master
Choose a base branch
from
Open

Added Menu Documentation #7087

wants to merge 3 commits into from

Conversation

@SplinterSword
Copy link
Contributor

@SplinterSword SplinterSword commented Oct 29, 2025 •
edited
Loading

Description

This PR fixes #5952

Notes for Reviewers
image
image
image

Signed commits

  • Yes, I signed my commits.

@SplinterSword
Added Menu Documentation
dc7b087 
Signed-off-by: SplinterSword <sparshj2003@gmail.com>
@l5io
Copy link
Contributor

l5io commented Oct 29, 2025

🚀 Preview for commit dc7b087 at: https://69020a273f76e561f896dff0--layer5.netlify.app

Rajesh-Nagarajan-11
Rajesh-Nagarajan-11 reviewed Nov 1, 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.

Merge conflicts

@l5io
Copy link
Contributor

l5io commented Nov 1, 2025

🚀 Preview for commit 07e42b5 at: https://6905b1688bc7258e811521e3--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
😊

@SplinterSword
Fixed Merge Conflicts
43ff4d9 
Signed-off-by: SplinterSword <sparshj2003@gmail.com>
@SplinterSword SplinterSword reopened this Nov 4, 2025
@l5io
Copy link
Contributor

l5io commented Nov 4, 2025

🚀 Preview for commit 43ff4d9 at: https://69098cc21ecbee4a0fcff53f--layer5.netlify.app

@l5io
Copy link
Contributor

l5io commented Nov 4, 2025

🚀 Preview for commit 43ff4d9 at: https://69098d2c3c5fb83dce5b8ce4--layer5.netlify.app

@SplinterSword
Added MenuItem and MenuList documentation to the Menu Page
ee8732c 
Signed-off-by: SplinterSword <sparshj2003@gmail.com>
@l5io
Copy link
Contributor

l5io commented Nov 4, 2025

🚀 Preview for commit ee8732c at: https://69099d255618fb4f3e5010fe--layer5.netlify.app

@Rajesh-Nagarajan-11
Copy link
Member

Rajesh-Nagarajan-11 commented Nov 8, 2025

Y all drop-down of menu's not sticky ? @SplinterSword

@SplinterSword
Copy link
Contributor Author

SplinterSword commented Nov 8, 2025

@Rajesh-Nagarajan-11 do you mean that the drop down menu does down with the scroll ??

@SplinterSword
Copy link
Contributor Author

SplinterSword commented Nov 8, 2025

I have just used the code given in the MUI library, so I don't know what is causing this. But I did notice that when the menu is opened in the MUI library, it is just disabling the scroll.

@SplinterSword
Copy link
Contributor Author

SplinterSword commented Nov 8, 2025

If you want I can disable the scroll till the menu is closed ??

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 Awaiting requested review from Rajesh-Nagarajan-11

@ChetanFTW ChetanFTW Awaiting requested review from ChetanFTW

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 Menu component to the sistent components page

4 participants

@SplinterSword @l5io @Rajesh-Nagarajan-11 @ChetanFTW