PASTA-ELN / pasta-eln

PASTA-ELN with new frontend
https://pasta-eln.github.io/pasta-eln/
Other
7 stars 2 forks source link

Closing the "You will loose all new data..." warning pop-up #224

Closed HTsybenko closed 3 months ago

HTsybenko commented 4 months ago
  1. Open an item to edit.
  2. Click cancel
  3. Warning “You will loose all new data. Do you want to save it for next time?” appears.

If the user clicks on "X" in the top right corner, it will close both the warning and the editing window. The expected behavior would be to close the warning only with the editing window still being open.

SteffenBrinckmann commented 3 months ago

@HTsybenko still an issue or closed after our discussion?

HTsybenko commented 3 months ago

@SteffenBrinckmann, I think we agreed not to close this one.

The other point of our discussion was whether the result of "X" on the editing form should be the same as for "Cancel". And for that one, we decided to keep it as is.

HTsybenko commented 3 months ago

The change was implemented.