Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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
Member

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
Collaborator

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

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

Successfully merging a pull request may close this issue.

4 participants