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

warning before overwrite #8

Open
Pupo1 opened this issue Dec 1, 2022 · 2 comments
Open

warning before overwrite #8

Pupo1 opened this issue Dec 1, 2022 · 2 comments

Comments

@Pupo1
Copy link

Pupo1 commented Dec 1, 2022

When I choose a custom url for my note, but an old note with exactly this url existed already, it will just be overwritten. I think, a warning would be nice.

@shelltr
Copy link
Owner

shelltr commented Dec 2, 2022

Are these the repro steps?

  1. open up tab 1 with url /test
  2. open up tab 2 with url /test
  3. save contents on tab 1
  4. save contents on tab 2 -- tab 1 content is overridden

So there should be a warning at step 4?

@Pupo1
Copy link
Author

Pupo1 commented Dec 8, 2022

Yes and no.
The repro steps are correct.
I would rather issue the warning at step 2 ("Warning: A textpad with this url is already open!").
At step 4 would be okay, too, but then you have already typed a bunch of text and don't know what to do with it.
Or, leave steps 1-4 as they are, but when afterwards I enter tab 1 and click on the text window to type something, issue a warning ("Warning: The text has been altered in another window. Saving this text will delete the other one.").

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