Skip to content

docs(menu): add component playgrounds for side & multiple #2627

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

Closed
wants to merge 2 commits into from

Conversation

brandyscarney
Copy link
Member

@brandyscarney brandyscarney commented Oct 28, 2022

While working on the menu playwright migration I noticed there were no usage examples in the docs on how to call open on a menu when you have multiple menus.

This PR adds the following two playgrounds:

  • Menu Sides
  • Multiple Menus

The first one just shows an example of how to change the side to end which is more of a buildup to the next one. Multiple menus has two menus on the start side and one on the end side to show how you can use MenuController to open a menu by id or side.

@vercel
Copy link

vercel bot commented Oct 28, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ionic-docs ❌ Failed (Inspect) Jul 18, 2023 3:28pm

@liamdebeasi
Copy link
Contributor

If the multiple menu issue is a blocker, we can probably ship only the side playground.

@thetaPC
Copy link
Contributor

thetaPC commented Jul 14, 2023

Verifying that it's still an issue on Angular & JavaScript StackBlitz. But it works fine when using Framework v7.

@brandyscarney brandyscarney requested review from a team and averyjohnston and removed request for a team and averyjohnston July 18, 2023 15:10
@brandyscarney
Copy link
Member Author

Thanks for bringing this up @thetaPC! I actually managed to fix the demos and get them working for v7 but this branch is too different from main now so I will open a new PR with the changes. I'll comment here when I do that.

@brandyscarney brandyscarney force-pushed the docs-menu-playgrounds branch from 2bbbc94 to 157b758 Compare July 18, 2023 15:23
@brandyscarney
Copy link
Member Author

Closing in favor of #3049

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.

3 participants