Skip to content
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

Notebook should use the Menu API for context menus #5713

Closed
ozyx opened this issue Aug 25, 2022 · 3 comments · Fixed by #6767
Closed

Notebook should use the Menu API for context menus #5713

ozyx opened this issue Aug 25, 2022 · 3 comments · Fixed by #6767

Comments

@ozyx
Copy link
Contributor

ozyx commented Aug 25, 2022

The Notebook object has a number of context menus which look and feel like menus generated from the Menu API, but are really generated by a PopupMenu component exclusive to the Notebook, whose existence predates the Menu API.

The Notebook should be updated to use the Menu API to help ensure consistent context menu behavior across the application.

@unlikelyzero
Copy link
Contributor

@ozyx
Copy link
Contributor Author

ozyx commented Jul 21, 2023

Testing Instructions

  1. Check the menus in Notebooks and verify they look and feel the same as other context menus in Open MCT

@khalidadil
Copy link
Contributor

The delete section/page menus look and work as expected. Verified in Testathon on 07/21/23

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

Successfully merging a pull request may close this issue.

4 participants