348830 - text-nodes inside a XUL element in a XHTML document break XBL bindings

Status: RESOLVED DUPLICATE of bug 183641

(Core :: General, defect)

Description

[Asaf Romano (gone)]

While working on bug 346009, I noticed some issues related to text-nodes inside XUL code in a XHTML document.

First, at least with the menupopup binding, once text-nodes are included in the subtree of the dom element, e.g.:

<xul:menupopup>
  <xul:menuitem/>
</xul:menupopup>

the childNodes subtree of the element apparently includes the binding anonymous content and the text nodes.

Second, I've seen semi-random spacing changes between the XUL elements when I tabbed to and out of the menulist.

Comment 1

[Boris Zbarsky [:bzbarsky]]

> the childNodes subtree of the element apparently includes the binding anonymous
> content 

Er... that's really wacky.  Post a testcase?

Comment 2

[Boris Zbarsky [:bzbarsky]]

Still waiting for that testcase...

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Could this just be bindings that depend on the XUL content sink's behavior of stripping whitespace?

Comment 4

[Boris Zbarsky [:bzbarsky]]

The bug as originally filed, absolutely.  I just wanted a testcase that showed anonymous content appearing in the childNodes of a bound element, as described in comment 0.  That should absolutely not be happening.

Comment 5

[Myk Melez [:myk] [@mykmelez]]

I wonder if bug 183641 is a dupe of this.

Comment 6

[Boris Zbarsky [:bzbarsky]]

Probably, yes.  Or rather vice versa...

Comment 8

[Lingfeng.Guan]

Attachment: a short testcase of the bug

Use javascript to import a button Node type=menu with <popupmenmu/>, and no label is displayed

Comment 9

[David Bolter [:davidb] (NeedInfo me for attention)]

Comment on attachment 340493 [details]
a short testcase of the bug

Interesting that you can tab through the widgets once, and then never again.

Comment 10

[Lingfeng.Guan]

Note that this bug does not only affect Mac OS X，but Windows XP also.

Comment 11

[Boris Zbarsky [:bzbarsky]]

If people can't put up a testcase addressing the question in comment 4, this should just be marked a duplicate of bug 183641 so people stop wasting time on this bug.

Comment 12

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

Making it so due to lack of response.