Timur,

I'm sorry but I really don't see why it's confusing as they are not the same.

Anyway, why not having an hover effect on pickers colors (but not background one) other than ones on curve modules.


Le 15/11/2019 à 19:41, Timur Irikovich Davletshin a écrit :
Nic, I'm just saying that this behavior looks a bit confusing from user
standpoint.

I'm sorry for not posting to github. MS decided that I cannot use it —
https://help.github.com/en/github/site-policy/github-and-trade-controls
  and I'm too lazy to use VPN for that purpose.

Timur.

P.S. there is no need to reply both to personal email and mailing list.

On Fri, 2019-11-15 at 19:32 +0100, Nicolas Auffray wrote:
I have never say they are not pickers, just they are not same one...
Just see how they renders and where they are on modules UI. Size is
not
the same, and also color. just see that they are visibly
different...
But remember that it's an open source software, so feel free to
change
that the way you want if you don't like that, for you or you could
even
propose a PR on Github... Hmm....

Color pickers on curve modules are here to apply black, grey and
white
parts of curve (so they are colored in... black, grey, white). If
hover,
that could only be on background. Other color pickers are not related
to
black, grey and white. So if we had a hover effect, it would be
better
to do that just on the picker (not the background but the picker
himself). It's a possibility, but for me it's good as it is actually
(and nobody else seems to have problem with that...). That's my view
here !


Le 15/11/2019 à 18:40, Timur Irikovich Davletshin a écrit :
If they are not pickers why use same icon? And if all other icons
have
hover action, why not these? Hmm...

Timur.

On Fri, 2019-11-15 at 18:25 +0100, Nicolas Auffray wrote:
Hi Timur,

It would really be better to post issues on Github (to let all
know
and
follow that). Anyway, some answers on what I see :

1. Yes, this icon doesn't seems to have CSS editing possibility
(maybe I
make a mistake but on what I see with GTK inspector...). So
posting
on
Github to let Aurélien see that (as it is his module work) would
be
better.

2 and 3. That's not issues. All color pickers like these ones
(see
exposure module for example) are displayed the same way. Color
picker
on
rgb curve are not same color picker (colored one, black, grey and
white
and not on same place like others color picker ; so having a
different
way is not a problem as they are not totally the same). Not
needed
to
have them have the exact same behavior.

Nicolas


Le 15/11/2019 à 18:09, Timur Irikovich Davletshin a écrit :
Hi developers!

1. tone equalizer module → display exposure mask icon has no
hover
background lightening. Compare with display mask icon in drawn
mask
blending.

2. Same problem in tone equalizer module → masking tab → picker
tool
next to mask exposure compensation and mask contrast
compensation.

3. filmic rgb module → scene tab → four picker tools need hover
background lightening. Compare with picker tool in rgb curve or
tone
curve.

Thank you advance,

Timur.

_______________________________________________________________
____
________
darktable developer mailing list
to unsubscribe send a mail to
darktable-dev+unsubscr...@lists.darktable.org

___________________________________________________________________
________
darktable developer mailing list
to unsubscribe send a mail to
darktable-dev+unsubscr...@lists.darktable.org

___________________________________________________________________________
darktable developer mailing list
to unsubscribe send a mail to darktable-dev+unsubscr...@lists.darktable.org


___________________________________________________________________________
darktable developer mailing list
to unsubscribe send a mail to darktable-dev+unsubscr...@lists.darktable.org

Reply via email to