Always show image rotation buttons in Image Operations sidebar when other operations are available
ClosedPublic

Authored by ngraham on Sep 8 2017, 9:33 PM.

Details

Summary

BUG: 354149

Always show the Rotate Left and Rotate Right operations in the Image Operations sidebar when other edit operations are visible.

Test Plan

Tested in KDE Neon (dev unstable). Works as expected:

Diff Detail

Repository
R260 Gwenview
Branch
arcpatch-D7743
Lint
No Linters Available
Unit
No Unit Test Coverage
ngraham created this revision.Sep 8 2017, 9:33 PM

I have a feeling that the reason why these were hidden from the Image Operations panel is because by default there are toolbar items for rotation, but various people complained anyway:

https://bugs.kde.org/show_bug.cgi?id=354149
https://bugs.kde.org/show_bug.cgi?id=357771

Also, it seems a little odd that they'd just be *gone*. And if you customize the toolbar to remove them, they'd still be gone from the Image Operations panel. Seems like it makes sense to add them back.

broulik edited edge metadata.Sep 8 2017, 10:36 PM

Hmm, I can see why both solutions would be valid, though I would tend to having the options in the sidebar only if the toolbar does not have them. Perhaps ask VDG/Usability for input.

ngraham added a reviewer: VDG.Sep 8 2017, 11:15 PM

ADDING VDG. I think it still makes sense to keep them in the toolbar as well, since there are many times when you might not have the Operations panel open but still want to rotate an image (it's one of the most basic operations).

Any more opinions?

broulik accepted this revision.Sep 22 2017, 3:46 PM

Let's go for it.

Just remove the lines instead of leaving commented code, we can still retrieve it from git history should the need arise.

This revision is now accepted and ready to land.Sep 22 2017, 3:46 PM
ngraham updated this revision to Diff 19796.Sep 22 2017, 3:58 PM

Removing these lines entirely instead of commenting them out

ngraham closed this revision.Sep 22 2017, 3:59 PM