66192 - mousethrough attribute on image widget in bulletinboard does not work if set by script

Status: RESOLVED WORKSFORME

(Core :: XUL, defect)

Description

[ugo]

From Bugzilla Helper:
User-Agent: Mozilla/4.0 (compatible; MSIE 5.01; Windows NT)
BuildID:    2001011804

Setting the mousethrough attribute directly on the widget declaration works 
fine.

Setting the "mousethrough" attribute on an "image" widget using one or both of 
the following script lines seems, instead, to not work:

1] foo.setAttribute('mousethrough','always')
2] foo.mousethrough = 'always'" 

Using the scripts above the attributes appear to be modified successfully by 
subsequent calls to "getAttribute(mousethrough)" and ".mousethrough" but they 
actually behave as if they retained the value set in the widget declaration.

Reproducible: Always
Steps to Reproduce:
1.Load the attached XUL file
2.Click on the "Set mousethrough" button.
3.Click on the topmost image (the darker one).

This test is designed by placing two overlapping images into a bulletinboard. 
Each image when clicked pops up a dialog with an identifiying text . Pressing 
the "Set mousethrough" button tries to set the "mousethrough" attribute by 
script as specified before.

Actual Results:  A dialog box appears, with the words 'topmost image' in it.
This means that the topmost image was clicked.

Expected Results:  Instead, a dialog with the words 'background image' should 
appear. This is because the topmost image should have the 
attribute "mousethrough" set and thus should not catch the mouse clicks, 
letting them hit the underlying image.

Comment 1

[ugo]

Attachment: a testcase showing the bug

Comment 2

[Blake Ross]

I'm confused.  What is mousethrough?  I've never heard of it.  Also, 
x.mousethrough = how shouldn't work, because x isn't an xbl element and 
mousethrough isn't an xbl property of it; you need to use setAttribute (but you 
do that anyways, right above).  You also don't have a binding for bulletinboard.

Comment 3

[John Morrison]

mousethrough is an attribute for stacks and bulletinboards, controls the 
capturing, the example does show that setting the attribute in script does 
not get reflected in the bulletinboard's behaviour.

Comment 4

[ugo]

It is exactly as Morrison explained (in a more compact format than i did ;) : 
setting the mousethrough attribute on the bb does not alter the bb behavior.

In response to Ross comment: 
why do i perform a double "set" on "mousethrough", once with x.setAttribute() 
and once with x.value ? Read the comments in bug 33412 (from date 2000-11-18 
14:16 onwards). From that comments it seems that the two ways of setting an 
attribute are NOT necessarily the same thing (defying common logic and user 
expectations, IMHO).

Comment 5

[Peter Trudelle]

->evaughan?

Comment 6

[Peter Trudelle]

->moz1.1

Comment 7

[Eric Vaughan]

Attachment: Fix

Comment 8

[Eric Vaughan]

Attachment: Opps forgot the content directory

Comment 9

[David Hyatt]

sr=hyatt

Comment 10

[Vedran Miletic]

retargeting

Comment 11

[Martijn Wargers (dead)]

This is WORKSFORME with current trunk build. Marking WFM, please reopen if you can still see the bug.