• mirror axis: fix paintop box button highlighting

There is one issue left to be solved: the highlight on the paintop box buttons. When the first document is open, the buttons are checked, but not highlighted. After switching around documents it starts working.

• Do not save mirror axis config if default
• code cleanup
• Mirror axis: apply actions to current view; set action state on load

For me it would certainly work in the tool options. I wonder if it would be a hurdle for users coming from other software, though. I think we discussed it a while back with @scottpetrovic, but I do not recall the details.

I would like to say in advance that I do not oppose the change (though must say I quite like the docker, since I worked on it some time ago; that can easily put me into a conflict of interest :). I just want to add some thoughts about a possible accessibility issue I can see as a dyslexic (also I'm not an expert on the matter, just what I know from myself and my relatives).

The redesign is done and merged in master.

I briefly looked into KDE's Breeze theme, it seems to do something to the QDial while painting the widget (it's not necessarily wrong, but it yields different results to the fusion theme under some circumstances). To work around that, I changed the min/max value of the dial and, subsequently, the conversion between the values of the dial and the spinbox. I tested with both Breeze and Fusion Qt themes, under Gnome, and it seems to work fine now.

• KisLayerStyleAngleSelector: make the dial work in KDE's Breeze theme

Thanks for testing.

I fixed a bug that caused the dial to skip the quadrant (90°,180°). That could have caused the moves to other side of the widget.

• KisLayerStyleAngleSelector: change QDial's min/max values

• add license to new files

In D16902#360199, @rempt wrote:

That looks fine. The only thing I wonder about is whether there isn't a way to bunch all the buttons up at the top of the docker when making it tall -- now there's some spacing appearing between the rows if you stretch the docker. But that's not new.

• spec.color.sel: UI cleanup

• unit suffix for KisIntegerColorInput
• right-align Kis*ColorInput/Kis*ParseSpinbox

• spec.color.sel: add colorspace name as button label

From a discussion with @scottpetrovic on the IRC:

I'm trying something different, to address non-trivial UX issues that arose from having masks both in document and on disk.

• ArtColorSel: reinitialize colors after canvas change
• Revert "ArtColorSel: pass KisCanvas2 through to KisColor"
• ArtColorSel: fixes dirty states on toggling gamut mask
• ArtColorSel: limit font size searches for value scale numbers

In D16288#348514, @dkazakov wrote:

Hi, @amedonosova!

The initial idea of implementing KisDisplayColorConverter was to avoid other classes depending on big fat KisCanvas class. Why exactly this dependency was needed?

• ArtColorSel: reinitialize colors after canvas change

@scottpetrovic Thanks for your feedback. I will go with the most automatic way with the smallest amount of buttons and other controls I will find, starting with solution 2. I will proceed to writing code, so we can evaluate it in action.

I would personally prefer solution 2, with storing enabled by default. I will explain why I added those controls, maybe we can come up with better workflow which would not need them. Solution 1 does not have those problems, but it is rather manual and therefore error-prone.

In D15860#334771, @scottpetrovic wrote:

I think what you have is great!