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.

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

Confusion when using save button on edited shareable link diagram #19

Closed
amoosbr opened this issue Jun 2, 2015 · 3 comments
Closed
Assignees

Comments

@amoosbr
Copy link

amoosbr commented Jun 2, 2015

Hi Skanaar,

thank you for starting this great project. I really like using it.
While using nomnoml in the team, one function lead to some confusion. It is related to the 'save' button functionality on a shareable link diagram (the one at the top middle).

The one interpretation is, that no changes are saved to local storage after clicking on ok.
And the other, in my team, more common interpretation, was that changes 'by default' are not saved automatically to local storage, but will be saved when clicking on the save button.
Part of the confusion is probably related to the confirm message dialog, which reads as
"Do you want to overwrite the diagram in localStorage with the currently viewed diagram?".
So, most of us thought that by clicking save the modified document would then be saved to local storage. Instead, the local storage was set to the content of the shareable link.

I just wanted to give you some feedback, from what we experienced by using nomnoml.

In my fork, I modified the save button to save the edited content to local storage. But if you want to keep functionality as is, perhaps you can modify the confirm message and make clear, that possible changes will be lost.

Thanks

@skanaar
Copy link
Owner

skanaar commented Jun 4, 2015

Thanks for reporting, will look into this

@skanaar skanaar self-assigned this Jun 4, 2015
@skanaar
Copy link
Owner

skanaar commented Jun 6, 2015

@amoosbr : your fork looked great, deployed a fix just now

@skanaar skanaar closed this as completed Jun 6, 2015
@amoosbr
Copy link
Author

amoosbr commented Jun 6, 2015

Thank you. I would have made a pull request, but I wasn't sure if the save worked as it did on purpose.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants