Weirdness if tests with same name applied.

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

Weirdness if tests with same name applied. #1

Closed marohrdanz closed 4 years ago

marohrdanz commented 4 years ago

When the same name is used for a test, that should either overwrite the existing values for that test, or add a '1', '2', etc. to the test name and show all the results. Instead, the existing test result box(es) appear off the edge of the PathwayMapper node. An example for the t-test is shown in the screenshot. Screen Shot 2020-07-21 at 3 24 35 PM

marohrdanz commented 4 years ago

The 'weirdness' was addressed in e52bcf6, but still need to give the user a confirmation dialog to let them decide if they want to overwrite the existing dataset with that name.

marohrdanz commented 4 years ago

Made separate issue (#4) for the confirmation box. Closing this because the 'weirdness' is fixed by the pull request.