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.

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

Refactor ContextMenu.js into a class hierarchy for MapContextMenu, LayerContextMenu and potentially similar #845

Open
prushforth opened this issue May 5, 2023 · 0 comments
Open

Refactor ContextMenu.js into a class hierarchy for MapContextMenu, LayerContextMenu and potentially similar #845

prushforth opened this issue May 5, 2023 · 0 comments
Labels
idea

Comments

@prushforth
Copy link
Member

ContextMenu.js is complicated and large, and hard to understand. It may be worthwhile to refactor it into separate handler subclasses so that the logic and content of different context menus is separated into different files, yet they share common infrastructure code.
@prushforth prushforth added the idea label May 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
idea
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@prushforth