-
Notifications
You must be signed in to change notification settings - Fork 771
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
Implement Per-User Change Tracking #11226
Comments
https://gerrit.libreoffice.org/c/core/+/182076 and https://gerrit.libreoffice.org/c/core/+/182142 extend the sw xml debug dump code to make this easier to understand -- what is the internal state of the doc model and the view. |
https://gerrit.libreoffice.org/c/core/+/182201 adds a per-view flag to track if changes are recorded or not. |
https://gerrit.libreoffice.org/c/core/+/182302 adds an in-model setter for the new flag, the getter still needs doing. |
https://gerrit.libreoffice.org/c/core/+/182345 with this, you can actually have per-view track change recording: toolbar button state, insert recording, delete recording etc is now per-view, not per-document. |
https://gerrit.libreoffice.org/c/core/+/182418 is towards adding UI to have both per-view and per-doc recording. |
https://gerrit.libreoffice.org/c/core/+/182455 adds UI to have both per-view & per-doc recording. |
https://gerrit.libreoffice.org/c/core/+/182517 adds state support for this new UI, so you can see if recording is off/just-you/everyone. |
Is your feature request related to a problem?
Yes. When Alice keeps typing and Bob enables change tracking, then surprisingly the typed characters of Alice will form a tracked insertion, which is surprising, since that was not the case a second ago and Alice didn't do anything other than typing.
Describe the solution you'd like
Move the "is recording" setting of change tracking to the view from the model, so it's per-user. And even better if the UI has a dropdown to choose if the change tracking would be off/just me/everyone.
Additional context
This is for COOL, but seeing e.g. "show non-printable characters" is also per-view in libreoffice desktop, probably it change tracking recording should be also per-view by default on the desktop, too.
The text was updated successfully, but these errors were encountered: