Bug 332832

Summary: Popup Palette broken with unused gray space and hover areas for selecting presets.
Product: [Applications] krita Reporter: vanyossi <ghevan>
Component: UsabilityAssignee: Krita Bugs <krita-bugs-null>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version First Reported In: git master (please specify the git hash!)   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: http://commits.kde.org/calligra/b6b993801631bb27ada4c8d139095e637a784933 Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: Popup palette, now and patched.
Proposed fix patch

Description vanyossi 2014-03-31 05:05:44 UTC 
Reduced hover area than visible area in presets. (see attached image)
Big gray unused area in the between the color history and presets. (it seems unused on my build at least)

Made a patch for master that does the following
Reduce palette size to 280 diameter
eliminate gap between color history and presets
Reduce a few pixels preset area but readjust hover area to match visible area.
Remove selected color backdrop from the triangle area as this makes color selection more difficult because of surround color. And because we now have the "ear" color that updates as the color changes.


Reproducible: Always
Comment 1 vanyossi 2014-03-31 05:06:35 UTC 
Created attachment 85856 [details]
Popup palette, now and patched.
Comment 2 vanyossi 2014-03-31 05:07:51 UTC 
Created attachment 85857 [details]
Proposed fix patch
Comment 3 Halla Rempt 2014-03-31 07:55:55 UTC 
Git commit 486ee6dbdad4880d1cb3627f9a19369790f51fb6 by Boudewijn Rempt.
Committed on 31/03/2014 at 07:53.
Pushed by rempt into branch 'master'.

Patch by Ghevan
CCMAIL:ghevan@gmail.com

M  +4    -15   krita/ui/kis_popup_palette.cpp

http://commits.kde.org/calligra/486ee6dbdad4880d1cb3627f9a19369790f51fb6
Comment 4 Halla Rempt 2014-05-30 15:49:15 UTC 
Git commit b6b993801631bb27ada4c8d139095e637a784933 by Boudewijn Rempt.
Committed on 31/03/2014 at 07:53.
Pushed by dickson into tag 'steam-247500_247566'.

Patch by Ghevan
CCMAIL:ghevan@gmail.com

M  +4    -15   krita/ui/kis_popup_palette.cpp

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