You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@openoffice.apache.org by bu...@apache.org on 2012/10/23 01:55:25 UTC

[Bug 121251] New: Color Picker: Edit will not pick up the hex color correctly.

https://issues.apache.org/ooo/show_bug.cgi?id=121251

          Priority: P3
            Bug ID: 121251
                CC: ooo-issues@incubator.apache.org
          Assignee: uineedsconfirm@ui.openoffice.org
           Summary: Color Picker: Edit will not pick up the hex color
                    correctly.
          Severity: normal
        Issue Type: DEFECT
    Classification: Code
                OS: All
          Reporter: pfg@apache.org
          Hardware: All
            Status: CONFIRMED
           Version: AOO 3.4.1
         Component: code
           Product: ui

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.

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

[Bug 121251] Color Picker: Edit will not pick up the hex color correctly.

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121251

--- Comment #3 from SVN Robot <sv...@dev.null.org> ---
"alg" committed SVN revision 1401221 into trunk:
#121251# corrected processing on hex value change

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

[Bug 121251] Color Picker: Edit will not pick up the hex color correctly.

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121251

Armin Le Grand <Ar...@me.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |Armin.Le.Grand@me.com
           Assignee|uineedsconfirm@ui.openoffic |Armin.Le.Grand@me.com
                   |e.org                       |

--- Comment #2 from Armin Le Grand <Ar...@me.com> ---
ALG: Seems reasonable, I'll take a look...

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

[Bug 121251] Color Picker: Edit will not pick up the hex color correctly.

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121251

Pedro Giffuni <pf...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           See Also|                            |https://bugs.freedesktop.or
                   |                            |g/show_bug.cgi?id=55800

--- Comment #1 from Pedro Giffuni <pf...@apache.org> ---
Ivan Timofeev seems to have fixed this same problem on LibreOffice.

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

[Bug 121251] Color Picker: Edit will not pick up the hex color correctly.

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121251

Armin Le Grand <Ar...@me.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |AOO 4.0

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

[Bug 121251] Color Picker: Edit will not pick up the hex color correctly.

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121251

Armin Le Grand <Ar...@me.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CONFIRMED                   |RESOLVED
         Resolution|---                         |FIXED

--- Comment #4 from Armin Le Grand <Ar...@me.com> ---
ALG: Okay, adapted. Thanks Pedro!

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