Some tool option UI improvements
ClosedPublic
Actions

Authored by scottpetrovic on May 29 2017, 4:33 PM.

Details

Reviewers

Group Reviewers

Summary

This started out just updating the smart patch tool options, but I started tweaking a few other things. All changes are with tool options.

update the move tool options to be flexible and have a smaller width. also simplified the verbiage a bit with labels
Use a grid layout for the contiguous selection tool so the input fields line up
add a horizontal ruler to the multi-hand tool options to separate the assistant options and tool options
Give a label for the magnetism property so it is easier to understand.
update to the smart patch tool options so it is flexible and takes up less space. added a suffix to the number and improved the labels a bit (I think)

For a better break-down of each change, see the petrovic/smartpatch-tooloptions

Test Plan

played around with the tool options, resizing it to make sure it resizes correctly.

Diff Detail

Repository

R37 Krita

Lint

Lint Skipped

Unit

Unit Tests Skipped

scottpetrovic created this revision.May 29 2017, 4:33 PM

scottpetrovic edited the summary of this revision. (Show Details)

For the multihand tool: I think it's kind of weird that there's white horizontal line to and a horizontal line with relief. I guess both should be the same thing.

For the rest, it does look a lot better.

This revision is now accepted and ready to land.May 30 2017, 9:48 AM

I think I got it. On Ubuntu, I can't see much of a difference, but I made the style plain. I can always tweak something if it doesn't look right. pushing for now.

Revision Contents
Changeset List

			Path	Packages
M			plugins/tools/basictools/kis_tool_brush.cc (9 lines)
M			plugins/tools/basictools/wdgmovetool.ui (167 lines)
M			plugins/tools/basictools/wdgmultihandtool.ui (15 lines)
M			plugins/tools/selectiontools/kis_tool_select_contiguous.cc (28 lines)
M			plugins/tools/tool_smart_patch/kis_tool_smart_patch_options_widget.ui (216 lines)