Issue 121251

Summary:	Color Picker: Edit will not pick up the hex color correctly.
Product:	ui	Reporter:	Pedro Giffuni <pfg>
Component:	code	Assignee:	Armin Le Grand <Armin.Le.Grand>
Status:	CLOSED FIXED	QA Contact:
Severity:	Normal
Priority:	P3	CC:	Armin.Le.Grand, issues
Version:	AOO 3.4.1
Target Milestone:	4.0.0
Hardware:	All
OS:	All
See Also:	https://bugs.freedesktop.org/show_bug.cgi?id=55800
Issue Type:	DEFECT	Latest Confirmation in:	---
Developer Difficulty:	---

Description Pedro Giffuni 2012-10-22 23:55:25 UTC

In the new color picker (that we are sharing with LibreOffice), attempting to set a new hex color will end up in a grey-ish color always.

Looking at the new color picker in file:
main/cui/source/dialogs/colorpicker.cxx

towards the end of IMPL_LINK( ColorPickerDialog, ColorModifyHdl, void *, p )

it appears that all of mdRed, mdGreen and mdBlue are being assigned a scaled version of aColor.GetRed: in other words GetBlue and GetGreen are being ignored.

this seems wrong.

Comment 1 Pedro Giffuni 2012-10-22 23:58:55 UTC

Ivan Timofeev seems to have fixed this same problem on LibreOffice.

Comment 2 Armin Le Grand 2012-10-23 08:58:24 UTC

ALG: Seems reasonable, I'll take a look...

Comment 3 SVN Robot 2012-10-23 09:44:06 UTC

"alg" committed SVN revision 1401221 into trunk:
#121251# corrected processing on hex value change

Comment 4 Armin Le Grand 2012-10-23 09:44:26 UTC

ALG: Okay, adapted. Thanks Pedro!

Comment 5 Marcus 2016-04-09 10:47:21 UTC

fixed in 4.0.0