Bug 387997 - Zooming results in "Edit guides" history entry
Summary: Zooming results in "Edit guides" history entry
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: General (show other bugs)
Version: git master (please specify the git hash!)
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Thorsten Zachmann
URL:
Keywords: release_blocker
Depends on:
Blocks:
 
Reported: 2017-12-18 05:41 UTC by Storm Engineer
Modified: 2018-01-04 14:48 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Storm Engineer 2017-12-18 05:41:54 UTC
Zooming after making brushstrokes results in an "Edit guides" entry in undo history, even for documents that do not have any guides.

To reproduce:
1; make a brushstroke
2; Zoom canvas in or out.

Just built from Master.
Comment 1 Quiralta 2017-12-20 20:41:20 UTC
I can reproduce this too, same git master build.

Also as the zooming is seen as a "step", it erases any -redo- in front of it, which has a potential of removing entirely some strokes, a definitely unwanted behavior.
Comment 2 Storm Engineer 2017-12-25 19:01:09 UTC
Some other actions also cause the Edit Guides event, such as transforming.

Since these events create a new history entry it makes Undo-ing very dangerous because this can destroy the steps preventing Redo.
Comment 3 Nicholas LaPointe 2018-01-01 04:44:54 UTC
This broke with commit 2a1329ec20a3f16a9c2d29bf057da76786cd0e50 ("Add an option to draw ruler tips as a power of 2") and is also the cause of bug 388318.
Comment 4 T Zachmann 2018-01-04 05:32:58 UTC
Git commit f91fa3c18a0cc8d72f3dab8285fc32993c6f6b7d by Thorsten Zachmann.
Committed on 04/01/2018 at 05:32.
Pushed by zachmann into branch 'master'.

Fix command generated when document is created

Don't create a command when the unit of the document is changed.
Related: bug 388318

M  +1    -1    libs/ui/canvas/kis_guides_manager.cpp

https://commits.kde.org/krita/f91fa3c18a0cc8d72f3dab8285fc32993c6f6b7d
Comment 5 T Zachmann 2018-01-04 05:34:12 UTC
Git commit 008d4977309776b496a7f38b0764a97bfddc5a8a by Thorsten Zachmann.
Committed on 04/01/2018 at 05:33.
Pushed by zachmann into branch 'krita/3.3'.

Fix command generated when document is created

Don't create a command when the unit of the document is changed.
Related: bug 388318
(cherry picked from commit f91fa3c18a0cc8d72f3dab8285fc32993c6f6b7d)

M  +1    -1    libs/ui/canvas/kis_guides_manager.cpp

https://commits.kde.org/krita/008d4977309776b496a7f38b0764a97bfddc5a8a
Comment 6 Storm Engineer 2018-01-04 14:48:03 UTC
I can confirm, the issue is fixed.