687422 - Remove isElementContentWhitespace

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Anne (:annevk)]

Text.isElementContentWhitespace is no longer part of the DOM standard.

Comment 1

[Anne (:annevk)]

WebKit does not support this attribute. I filed a bug on Opera for removing it there (along with replaceWholeText()).

Comment 2

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Attachment: Patch v1

Comment 3

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Jonas?

Comment 4

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

Comment on attachment 560927 [details] [diff] [review]
Patch v1

Hrm.. this actually seemed like a useful function. It always feels silly to me when all browsers have to have state or functionality internally that isn't exposed to the webpage.

So I'd be in favor of keeping this in spec/implementations. But if people prefer to remove it then I won't stand in the way.

Comment 5

[Anne (:annevk)]

The current name does not make any sense. I would be open to renaming it. E.g. Text.isWhiteSpace. I used an uppercase S for consistency with white-space in CSS.

Comment 6

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

I'd be fine with that.

Comment 7

[Anne (:annevk)]

I filed http://www.w3.org/Bugs/Public/show_bug.cgi?id=14443 on adding Text.isWhiteSpace to DOM4, but Simon rightly asks what the use case is and I have no idea.

Comment 8

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

https://hg.mozilla.org/mozilla-central/rev/292803c8c406

Comment 9

[Jean-Yves Perrier [:teoli]]

I updated:
https://developer.mozilla.org/en/Firefox_10_for_developers
https://developer.mozilla.org/en/DOM/document.xmlEncoding
https://developer.mozilla.org/en/DOM/document

Comment 10

[Jean-Yves Perrier [:teoli]]

Oops, wrong bug (sorry for the spam).

Comment 11

[Jean-Yves Perrier [:teoli]]

This time, I updated:
https://developer.mozilla.org/en/Firefox_10_for_developer
https://developer.mozilla.org/En/DOM/Text.isElementContentWhitespace
https://developer.mozilla.org/En/DOM/Text