45092 - [CBX]SELECT control breaks when semi-transparent

Status: RESOLVED FIXED

(Core :: Layout: Form Controls, defect)

Description

[Taras Tielkes]

When a SELECT control has inherited opacity, the "pull-down" arrow is drawn
apart from the controls box, and not inside at the right side.

Button clicks from the original area (where the down-arrow should be) *are* sent
to the control.

Comment 1

[Taras Tielkes]

Attachment: Testcase: Simple SELECT elements is rendered wrong when semi-transparent

Comment 2

[Taras Tielkes]

Seeing the same behaviour on Linux, changing plaf/OS to All/All

Comment 3

[Marc Attinasi]

I see it too - over to the HTML control wizard to check it out... problem with 
the view being setup incorrectly?

Comment 4

[rods (gone)]

futuring

Comment 5

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

This works for me on Win98SE Build 2000-09-06-08

Comment 6

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Sorry, forgot that the CSS syntax has changed to -moz-opacity. This dosn't work
for me...

Comment 7

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Attachment: updated testcase with -moz-opacity

Comment 8

[Taras Tielkes]

Thanks for the updated testcase, Jonas. This also still fails for me on WinNT.

Comment 9

[Hixie (not reading bugmail)]

Robert - this is another opacity related bug, does your view manager solve it?
I would guess no; seems like a problem in the control code, but we never know...

Comment 10

[Taras Tielkes]

Related to: http://bugzilla.mozilla.org/show_bug.cgi?id=45095

Comment 11

[Robert O'Callahan (:roc) (email my personal email if necessary)]

I have a feeling I might have fixed this... However, my build is broken (XSL 
stuff) so I can't tell you yet :-).

Comment 12

[Robert O'Callahan (:roc) (email my personal email if necessary)]

My view manager didn't fix it. But now I see this bug on ALL SELECT elements,
not just the transparent ones! Not sure if this is something I did, or something
that recently hit the tree. Anyway, this should be easier to track down now :-).

Comment 13

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Found it. This is Yet Another Bug caused by people assuming that certain kinds 
of frames will never have views attached. The fix should be easy.

Comment 14

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Attachment: Fix

Comment 15

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Fixed. Rod, can you get this in? You can assign the bug to me if you prefer.

Comment 16

[Robert O'Callahan (:roc) (email my personal email if necessary)]

See bug 34887 for a discussion of why you should NEVER assume that a frame does 
not have a view; wherever you call nsIFrame::SetRect, you have to check for a 
view and position it if necessary.

Comment 17

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Wait wait wait. This breaks XUL. Hmm.

Comment 18

[Robert O'Callahan (:roc) (email my personal email if necessary)]

I take it back. XUL popups are broken in my tree for some other reason; this 
particular code is not even used by XUL. So the fix looks OK to me.

Comment 19

[bsharma]

Updating QA contact.

Comment 20

[Vladimir Ermakov]

QA Contact Update

Comment 21

[Jesse Ruderman]

Another problem that occurs when viewing the testcase on this bug:
bug 92748 semi-transparent <select> drop-down thinks background is black.

Comment 22

[Simon Fraser [no longer active]]

Does the fix in attachment 14277 [details] [diff] [review] still apply?

Comment 23

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Your guess is as good as mine ... my guess is "sure, why not?" Try it :-).

Comment 24

[Taras Tielkes]

Was the fix for this ever checked in?

The updated testcase appears to work correctly using 1.0rc1/Win32.
Still seeing the problem with
(http://bugzilla.mozilla.org/show_bug.cgi?id=92748) though...

So can this one be closed?

Comment 25

[Robert O'Callahan (:roc) (email my personal email if necessary)]

This was fixed by the support for group opacity, bug 212366