Request for review: 7124530 What is background color of AWT component? (And foreground, for that matter) (original) (raw)
Sergey Bylokhov sergey.bylokhov at oracle.com
Tue Jan 10 07:33:14 PST 2012
- Previous message: hg: jdk7u/jdk7u-osx/jdk: 7127199: [macosx] test/com/sun/jdi/ShellScaffold.sh needs to include Darwin as a recognized platform
- Next message: Request for review: 7124530 What is background color of AWT component? (And foreground, for that matter)
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Hi Everyone,
Does anybody have a comments?
29.12.2011 17:34, Sergey Bylokhov wrote:
Hi Everyone, This is a fix for some glitches in the code for background/foreground/font properties. 1. SystemColor.window was changed to color which is used by default in swing l&f(Aqua). Changes in AquaImageFactory.java and CSystemColors.m. Now most of the awt components use this color as default. 2. set** methods were moved from LWWindowPeer to LWComponentPeer, because there is an issues when these methods has no effect, because repaint of the component does not happen.For example: - call Label.setbackground which set component background property - call Peer.setBackground - call Delegate.setBackgound - compare passed color with color from Label property, which was set in the first step: changes in LWWindowPeer,LWContainerPeer,LWComponentPeer. 3. List default color was changed to SystemColor.text: changes in LWListPeer.java. 4. LWWIndow target color initialization was moved from CPlatformIndow to LWWindowPeer, so it can be reused later by CPlatformEmbeddedFrame . 5. unnecessary peers set** methods and unnecessary delegate in LWCanvasPeer were removed: changes in LWCanvasPeer.java, LWListPeer.java, LWPanelPeer.java.
Bug: http://bugs.sun.com/bugdatabase/viewbug.do?bugid=7124530 Webrev can be found at: http://cr.openjdk.java.net/~serb/7124530/webrev.00/
-- Best regards, Sergey.
- Previous message: hg: jdk7u/jdk7u-osx/jdk: 7127199: [macosx] test/com/sun/jdi/ShellScaffold.sh needs to include Darwin as a recognized platform
- Next message: Request for review: 7124530 What is background color of AWT component? (And foreground, for that matter)
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]