Community
Participate
Working Groups
+++ This bug was initially created as a clone of Bug #290212 +++ User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.3) Gecko/20090824 Firefox/3.5.3 (.NET CLR 3.5.30729) Build Identifier: I20090917-0100 SWT controls that are the children of a Group or TabFolder do not have their backgrounds filled correctly. Essentially, the control backgrounds look wrong. This problem does not exhibit in Carbon - only Cocoa. Bug #234240 dealt with a similar problem. It's probably not related. Bug #283045 looks like it could be the problem that was fixed by the resolution of the above Bug #234240. Screen shot is attached. Test code will be uploaded in the next few days, but it's a simple thing to reproduce -- not so simple (for me) to solve at this point. I suspect that the alpha value passed to the underlying Cocoa routine (in Control.fillBackground) is not having the desired effect. Reproducible: Always See attached screenshot
Cloned for 3.6.2. No rush, but it would be good to consider this for 3.6.2.
This is most likely the same bug as Bug 297633. The fix for this was in combination with Bug 290212 that Scott fixed. I haven't seen the *actual* patch but if it's the same as the one I described in these above two bugs, it should be good. I'm running with a patched system here, so when you upload the test case, I'll check the changes in the HEAD and confirm that it's fixed by the fixes for these other bugs. I've spent so much time getting widgets to work correctly in Groups and TabFolders that I'd be amazed if this is *not* the case. But it's always possible! By the way, you forgot the screenshot.
Created attachment 186864 [details] Fix for 3.6.2 Patch from original bug applied to 3.6.2.
Ready for review.
Created attachment 186865 [details] Fix for 3.6.2 Wrong patch first time.
Should Button.drawsBackground() (label too) check for "backgroundImage != null" as well? I have not tried but I believe if someone sets a background image in a button or label, it is not going to work? return background != null || backgroundImage != null;
(In reply to comment #6) > Should Button.drawsBackground() (label too) check for "backgroundImage != null" > as well? I have not tried but I believe if someone sets a background image in a > button or label, it is not going to work? > > > return background != null || backgroundImage != null; Good point. Yes. Your code above is correct and we need to alter the patch accordingly.
Created attachment 186962 [details] Updated 3.6.2 patch Updated with the suggestion in comment #6
Need to fix HEAD as well.
Fixed in HEAD and on the 3_6_maintenance branch > 20110118.
