77847 - XBL demo broken.

Status: RESOLVED INCOMPLETE

(Core :: XBL, defect)

Description

[/\/\arcio Galli]

build: 2001042604
win98

go to the demo URL. In Older Mozilla it was animating the defined LAYER object.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Um... looks like possible xpcdom issues...
note that the <layer> and document.layers _are_ defined through XBL, so it
should be working...

Comment 2

[David Hyatt]

Bindingattached has been replaced with <constructor> in the implementation.  
Could someone try patching the demo and see if that fixes the problem?

Comment 3

[Brendan Eich [:brendan]]

Cool -- layers emulation via xbl!

/be

Comment 4

[Mike Shaver (:shaver emeritus)]

I changed
  <handler event="bindingattached">
to
  <constructor>
and the matching
  </handler>
to
  </constructor>
but still no joy.

Comment 5

[David Hyatt]

Did you move it under <implementation> and not under <handlers>?

Comment 6

[Mike Shaver (:shaver emeritus)]

That helps, but not enough.  I now get 3 Hello World things, with some cool
selection-at-a-distance behaviour between two of them.

And no animation.

Comment 7

[David Hyatt]

I'm not convinced that there's anything wrong here.

Comment 8

[/\/\arcio Galli]

Sorry for my long delay,..

David, 

The demo moves the layer with N6.0+....With latest seems like the
document.layers[""] array is not prepared by the XBL code (JS code in the XBL file).

Basically the demo try to set the document.layers[""] array. 

-- 

I will try to refine the testcase.. soon..

Comment 9

[/\/\arcio Galli]

All right, 

seems like event="bindingattached" approach now is <constructor> inside the
<implementation> element (please correct me if I am wrong).. I am trying to fix
/ update the demo.....

Comment 10

[/\/\arcio Galli]

Ok. I updated the xbl file to have the old "bindindattached" JavaScript code in
the implementation->constructor area. The demo is now happy. However I can see
multiple instances of the 'layer' element in the page. It's supposed to be just
one. I will review the demo again to verify if I did something crazy. 

New updated version:

http://geckonnection.com/layer/x/layer.html

Comment 11

[/\/\arcio Galli]

Version (document.layers) for DIV elements. This version is fine, basically no
transformations (LAYER -> DIV) involved:

http://www.dhtmlplanet.com/layer/x/layer2.html

Comment 12

[José Jeria]

Is this still an issue?

Comment 13

[Michael Kohler [:mkohler]]

This is a mass change. Every comment has "assigned-to-new" in it.

I didn't look through the bugs, so I'm sorry if I change a bug which shouldn't be changed. But I guess these bugs are just bugs that were once assigned and people forgot to change the Status back when unassigning.

Comment 14

[Boris Zbarsky [:bzbarsky]]

Demo is gone, and XBL is going away anyway.