Add rotate and rotation-reset buttons to overview docker.
Needs ReviewPublic

Authored by woltherav on Aug 21 2016, 2:13 PM.

Details

Reviewers
dkazakov
rempt
Summary

I... vaguely recall there was a wishbug that assigned this to me... But I am pretty sure that there's a bug out there that needed this.

Anyway, adds buttons for rotate-right/reset rotation/rotate-left to the overview docker, to increase discoverability.

Test Plan

For some reason the icons don't show up.
Also we have no icon for resetRotation.

I don't know why the first happens, so hence the review request.

Diff Detail

Repository
R37 Krita
Lint
Lint Skipped
Unit
Unit Tests Skipped
woltherav updated this revision to Diff 6117.Aug 21 2016, 2:13 PM
woltherav retitled this revision from to Add rotate and rotation-reset buttons to overview docker..
woltherav updated this object.
woltherav edited the test plan for this revision. (Show Details)
woltherav added reviewers: rempt, dkazakov.
woltherav set the repository for this revision to R37 Krita.
woltherav added a project: Krita.
woltherav added a subscriber: timotheegiet.
dkazakov edited edge metadata.Aug 22 2016, 6:30 AM

Hi, Wolthera!

For some reason I don't see any icons on the new buttons :( Probably, the icons should be set on the actions instead of the buttons (just a guess)?

Yeah, that was my problem as well. But yes, perhaps using the action instead might be a good idea to try...

rempt added a comment.Nov 2 2017, 8:55 AM

Um... Did we completely miss this patch, or did it land?