Bug 343328 - Copy ends up in the undo stack
Summary: Copy ends up in the undo stack
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Tools (show other bugs)
Version: 2.9 Beta
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-01-26 13:39 UTC by Halla Rempt
Modified: 2015-01-26 19:55 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Halla Rempt 2015-01-26 13:39:59 UTC
A Copy action doesn't change the image so it should not be put on the undo stack. Right now it is, because of the code shared with the Cut action.
Comment 1 Halla Rempt 2015-01-26 19:55:11 UTC
Git commit c55fc01853631061df04b05ff65fd0f315c2a0bf by Boudewijn Rempt.
Committed on 26/01/2015 at 14:01.
Pushed by rempt into branch 'calligra/2.9'.

Don't push Copy onto the undo stack

The copy action doesn't affect the document state, so doesn't need
to be undoable (and in fact, undoing a copy didn't undo the change
to the clipboard).

M  +40   -38   krita/ui/actions/kis_selection_action_factories.cpp

http://commits.kde.org/calligra/c55fc01853631061df04b05ff65fd0f315c2a0bf