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

Request for Clearing Functionality in TextEdit Undoer for Enhanced Data Security #3436

Closed
rustbasic opened this issue Oct 4, 2023 · 0 comments · Fixed by #3479
Closed

Request for Clearing Functionality in TextEdit Undoer for Enhanced Data Security #3436

rustbasic opened this issue Oct 4, 2023 · 0 comments · Fixed by #3479

Comments

@rustbasic
Copy link
Contributor

rustbasic commented Oct 4, 2023

Dear emilk,

We need a feature to clear all content from the undoer in TextEdit.

For instance:

  1. When editing a new file, where we need to get rid of the previously saved Undo function.
  2. When entering an ID or password and we don't want the Undo function to work.

Using the id_source() function to disable the Undo feature is possible, but for data security, it would be better to have the ability to erase the content in the undoer.

Thanks, emilk.

LoganDark added a commit to LoganDark/egui that referenced this issue Oct 17, 2023
This feels awful, @emilk why does this have to be an Arc?

Should the Arc be replaced with a new one when set_undoer is called,
or should it just replace the undoer inside the Arc?

Fixes emilk#3436
LoganDark added a commit to LoganDark/egui that referenced this issue Oct 17, 2023
This feels awful, @emilk why does this have to be an Arc?

Should the Arc be replaced with a new one when set_undoer is called,
or should it just replace the undoer inside the Arc?

Closes emilk#3436
emilk pushed a commit that referenced this issue Jan 7, 2024
This feels awful, @emilk why does this have to be an Arc?

Should the Arc be replaced with a new one when set_undoer is called, or
should it just replace the undoer inside the Arc?

Closes #3436
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

Successfully merging a pull request may close this issue.

1 participant