Bug 16250 – Implement Element.all (http://news.vnet.cn : Mouse hover on tabs not working)

• Component: DOM
• Status: RESOLVED WONTFIX

[Anantha Keesara]

Steps: Mouse over on tabs below flash image, with 1 2 3, on left side of the page.

Issue: The tab content doesn't change.

IE and Opera gets it right.

Tested with: WebKit-SVN-r28278

[Anantha Keesara]

Created attachment 17647 [details]
reduction

[Alexey Proskuryakov]

Confirmed. We only have document.all, HTML elements do not implement all() in WebKit.

[Alexey Proskuryakov]

http://msdn2.microsoft.com/en-us/library/ms537434.aspx

[Maciej Stachowiak]

Should Element.all be hidden, like Document.all?

[Eric Seidel (no email)]

I'm no expert in the area.  But I would suggest we hide it.  There is no sense in advertising our being more like IE.  Element.all only needs to exist for compat with sites which only assume IE (and are currently completely broken on all non-IE browsers).

We already have the code to make a hidden property, we can just reuse the stuff for Document.all.

[Jeff Walden (remove +bwo to email)]

Catfish_Man asks me to note this here:

While I'm thinking about it, I remember hearing that Mozilla got asked to implement Element.all but didn't see the need and were attempting to hold the line on that particular IEism; webkit implementing doesn't help that.  :-\

[David Smith]

Created attachment 18557 [details]
Patch

The full .all feature is pretty weird, according to Microsoft's documentation. This reuses what we have for document.all, which is enough to make the tabs on news.vnet.cn work.

[Eric Seidel (no email)]

Comment on attachment 18557 [details]
Patch

So I think this patch is fine.  The only thing missing is test cases. :)

r- until we add at least one element.all test case! :)  The test that it's undetectable is nice, but doesn't really test the feature. :)

[Anne van Kesteren]

At this point this is no longer needed.