Bug 356664 - Fix colour picker cursor form
Summary: Fix colour picker cursor form
Status: RESOLVED MOVED
Alias: None
Product: kdenlive
Classification: Applications
Component: User Interface & Miscellaneous (show other bugs)
Version: unspecified
Platform: Arch Linux Linux
: NOR wishlist
Target Milestone: ---
Assignee: Jean-Baptiste Mardelle
URL: https://dl.dropboxusercontent.com/u/5...
Keywords:
Depends on:
Blocks:
 
Reported: 2015-12-14 09:53 UTC by Egor
Modified: 2018-10-14 14:03 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:
snd.noise: low_hanging+

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Egor 2015-12-14 09:53:50 UTC 
Colour picker cursor form not optimal. Which point at cursor image actually choose colour pixel?
Visual explanation: https://dl.dropboxusercontent.com/u/59794362/kdenlive-bugs/color-picker/color-picker.jpg

Mate DE

Reproducible: Always
Comment 1 qubodup 2015-12-14 20:01:41 UTC 
I agree that that icon looks confusing at first sight and that icons that are pointed towards a corner are more familiar.

Breeze has a colorpicker cursor, which could be used
http://i.imgur.com/X0Dg5gG.png
Comment 2 farid 2016-10-26 16:38:35 UTC 
I prefer something like this:

https://github.com/KDE/krita/blob/46c8b4048fc7f5b81a4128bc4dcaf1568d3cf316/krita/data/cursors/color-picker_image_foreground.xpm

(You must download it and open it with an image viewer to see it properly)
Comment 3 Vincent PINON 2017-04-16 02:20:48 UTC 
Any artist would prepare an SVG with the proper color codes allowing KF5 color theme switching?
Comment 4 farid 2017-04-16 03:33:49 UTC 
(In reply to Vincent PINON from comment #3)
> Any artist would prepare an SVG with the proper color codes allowing KF5
> color theme switching?

Sorry but I didn't understand. What does the proper color codes mean? Obviously me asking means I don't know how to help but I can take this to the VDG people or if you show me the way I'll try to do it. You're talking about the cursor I proposed right? 

I somehow remember that someone tackled this issue in the bug squashing day I just can't find the log to be sure...
Comment 5 farid 2018-10-14 14:03:24 UTC 
Moved to phabricator:

https://phabricator.kde.org/T9863