- From: Marat Tanalin | tanalin.com <mtanalin@yandex.ru>
- Date: Tue, 03 Dec 2013 04:37:26 +0400
- To: Tab Atkins Jr. <jackalmage@gmail.com>
- Cc: Elliott Sprehn <esprehn@gmail.com>, "www-dom@w3.org" <www-dom@w3.org>
03.12.2013, 04:04, "Tab Atkins Jr." <jackalmage@gmail.com>: > I find "new Element('h1',...)" less good to read than something like > "HTML.h1(...)". š(If you have a tagname in a string, you can always do > HTML[tag](...).) As for `HTML.h1(...)` specifically, this is a potentially bad idea to have a tag name as a method name. Tag name may typically be arbitrary and even unsupported by browser while method names are usually predefined and cannot be arbitrary. Also, there is a risk that some tag name will eventually interfere with some sensible method or property name. For example, if future HTML version will add `length` element, `HTML.length` as an object property and `HTML.length()` as a "constructor" method would have totally different purpose/functionality and therefore be ambiguous/confusing. Actually, using tag names as method names is probably a _worst_ syntactic idea I've heard of lately. >> šIt would also be consistent with already existing web-developer-friendly constructors like `new Text` and `new Comment` (implemented in Firefox 24+, Chrome 28+, and Opera 15+). > > Not really. šThere's a single type of Text node and Comment node. > There are many significant types of Element nodes. šThis isn't a good > way to slice up the functionality. If someone would like to use constructors like `new HTMLDivElement`, they are free to do that. However, if we are talking about a really usable and non-error-prone syntax, this probably shouldn't be the _only_ option. For me as a 10-years+ experienced web-developer, `new Element(arbitraryTagName)` looks like a more suitable option. Just in case, please note that my original proposal is not about constructor-or-factory-method topic at all. My proposal is about creating element and setting its multiple attributes at once. Thanks.
Received on Tuesday, 3 December 2013 00:37:56 UTC