search

MD-Anderson-Bioinformatics / pathway-mapper

An interactive and collaborative graphical curation tool for cancer pathways
https://www.pathwaymapper.org/
GNU Affero General Public License v3.0
0 stars 0 forks source link

Pathway and related information erased if user opens new panels in sections containing PathwayMapper #6

Open jmelott opened 3 years ago

marohrdanz commented 3 years ago

This is caused by a fragile original fix!

We noticed this problem before, and created this hacky commit to address it: https://github.com/MD-Anderson-Bioinformatics/NG-CHM/commit/63c6c837538fefa2e8cfcf20b49f982edb58df1c

This hack checks for 'PathwayMapper' in the pane text content to decide if the user should be warned about losing the PathwayMapper state. BUT, when I set up the custom.js file for the testing, I used 'Pathway Mapper' as the name of the plugin (instead of 'PathwayMapper'). So the code did not give the user a warning about losing pathway mapper info.

We need to have a more robust fix for this bug.