[7u4] Request for approval for 7124530 (original) (raw)

[7u4] Request for approval for 7124530 - What is background color of AWT component? (And foreground, for that matter)

Sergey Bylokhov sergey.bylokhov at oracle.com
Fri Jan 13 06:27:52 PST 2012

13.01.2012 6:30, Mike Swingler wrote:

On Jan 12, 2012, at 4:24 AM, Sergey Bylokhov wrote:

Hello, This is a request to push the following changes to jdk7u-osx. The fix has been reviewed on macosx-port-dev mailing list by Alexander Potochkin.

Bug: http://bugs.sun.com/bugdatabase/viewbug.do?bugid=7124530 Webrev can be found at: http://cr.openjdk.java.net/~serb/7124530/webrev.00/ Technical review: http://mail.openjdk.java.net/pipermail/macosx-port-dev/2012-January/002143.html I don't understand this part: --- old/src/macosx/native/sun/awt/CSystemColors.m 2011-12-28 19:02:19.913935900 +0400 +++ new/src/macosx/native/sun/awt/CSystemColors.m 2011-12-28 19:02:19.557915600 +0400 @@ -81,7 +81,8 @@ sColors[javaawtSystemColorINACTIVECAPTION] = [NSColor grayColor]; sColors[javaawtSystemColorINACTIVECAPTIONTEXT] = [NSColor grayColor]; sColors[javaawtSystemColorINACTIVECAPTIONBORDER] = [NSColor grayColor]; - sColors[javaawtSystemColorWINDOW] = [NSColor grayColor]; + const CGFloat color = (CGFloat)0xEE/(CGFloat)0xFF; + sColors[javaawtSystemColorWINDOW] = [NSColor colorWithCalibratedRed:color green:color blue:color alpha:1.0f]; sColors[javaawtSystemColorWINDOWBORDER] = [NSColor windowFrameColor]; sColors[javaawtSystemColorWINDOWTEXT] = [NSColor windowFrameTextColor]; sColors[javaawtSystemColorMENU] = [NSColor controlBackgroundColor]; Why aren't you just using [NSColor windowBackgroundColor]? Only selectedControlColor and selectedTextBackgroundColor are supported. http://developer.apple.com/library/mac/#documentation/Cocoa/Conceptual/DrawColor/Tasks/SystemColors.html#//apple_ref/doc/uid/20000790 As I understand thats why swing didnt use this color. After the fix awt and swing use one color (before the fix this color was used by swing). If this color is wrong, now it`s possible to change it in one place. Regards, Mike Swingler Apple Inc.

-- Best regards, Sergey.

More information about the macosx-port-dev mailing list