Feature: Leaflet mapping and context menu

mtwichan commented 2 years ago

About

This is a new feature for Dash-cytoscape which includes integration with Leaflet.js and a context menu feature.

Description of changes

Users can render their cytoscape graphs on Leaflet map using any tile server supported by Leaflet. This, also allows users to graph Cytoscape nodes using latitude and longitude coordinates. Additionally, they can create context menus that appear on right-click, with or without the Leaflet map.

The context menu is fully customizable. Developers have full control of what options are shown in the context menu and the business logic associated with clicking on that context menu selection.

cytoscape-leaflet-dash

Pre-Merge checklist

[x] The project was correctly built with npm run build:all.
[ ] If there was any conflict, it was solved correctly.
[ ] All changes were documented in CHANGELOG.md.
[ ] All tests on CircleCI have passed.
[ ] All Percy visual changes have been approved.
[ ] Two people have :dancer:'d the pull request. You can be one of these people if you are a Dash Cytoscape core contributor.

Reference Issues

Leaflet feature: https://github.com/zyphr-solutions/dash-cytoscape/pull/2 Context menu feature: https://github.com/zyphr-solutions/dash-cytoscape/pull/1

Other comments

This feature was created in collaboration with @plotly and reviewed by @alexcjohnson

mtwichan commented 2 years ago

Waiting on @kinimesi to complete the following before we can merge the PR:

The order of execution of tasks should be the following:

Solve: https://github.com/plotly/dash-cytoscape/issues/159
Test new Dash Cytoscape features and write tests for the features
Add any new features and bug fixes to Dash Cytoscape CHANGELOG and any other relevant Dash Cytoscape release/versioning files. The new version will be 0.4.0
Merge :tada:
Write documentation for new Dash Cytoscape features (contextMenu & leaflet)
Ping when finished so I can write a release statement for the Plotly forums and newsletter, and so that Plotly can push to PyPi

CC: @alexcjohnson @kully @jackparmer

Farkites commented 7 months ago

Closing this PR in favour of #200 and #196

plotly / dash-cytoscape