[Libreoffice-ux-advise] [Bug 153933] bg color for area fill of cell and dialog fields for negative find results against calc cells -- #ff6563 is a bit harsh

[bugzilla-daemon at bugs.documentfoundation.org]

https://bugs.documentfoundation.org/show_bug.cgi?id=153933

V Stuart Foote <vsfoote at libreoffice.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |UNCONFIRMED
     Ever confirmed|1                           |0
            Summary|bg color for area fill of   |bg color for area fill of
                   |cell and dialog fields for  |cell and dialog fields for
                   |negative find results       |negative find results
                   |against calc cells --       |against calc cells --
                   |#ff6d6d  is a bit harsh     |#ff6563  is a bit harsh

--- Comment #6 from V Stuart Foote <vsfoote at libreoffice.org> ---
(In reply to Heiko Tietze from comment #5)
> In case of weld::EntryMessageType::Error the background becomes #ff6563, see
> vcl/source/app/salvtables.cxx set_message_type().
> 
> This red has not changed (neither it follows dark/light color schemes) and I
> don't get the "pink". Since we do not have a warning/error color from the OS
> this looks like a proper solution.
> 

Thanks Heiko for finding that one off color ref (OOo from i75179) for the
ERROR--could we adjust it from 0xff6563 to 0xff6d6d so picker can match a color
on the Standard palette "Light Red 2"--but have to say it does trend "pink" .

> Although some adjustments make sense, ie. the yellow color is over-saturated
> and should become #ffff63.

I guess, for Warning the COL_YELLOW is a bit saturated.

So, how about tweaking to 

ERROR -- 0xff3838 
WARNING -- 0xffff38 

which would correspond to the  "Light Red 1" and "Light Yellow 1" of the
standard palette?

As long as there is no unit test conflict, that change would improve UI for
both.

-- 
You are receiving this mail because:
You are on the CC list for the bug.