Confirmed users
146
edits
m (→XTF) |
|||
Line 160: | Line 160: | ||
<html:input> already has ten different types of input supported. WF2 adds fourteen more. The DOM for HTMLInputElement as defined for WF2 includes 41 properties and 10 methods, all of which must be physically exist (even if they are meaningless, as attributes such as pattern would be for checkbox inputs). | <html:input> already has ten different types of input supported. WF2 adds fourteen more. The DOM for HTMLInputElement as defined for WF2 includes 41 properties and 10 methods, all of which must be physically exist (even if they are meaningless, as attributes such as pattern would be for checkbox inputs). | ||
nsHTMLInputElement.cpp is already the second-largest HTML element file in content/html/content/src (behind nsGenericHTMLElement.cpp). With the added complexity of WF2, nsHTMLInputElement threatens to become unmanageable. | nsHTMLInputElement.cpp is already the second-largest HTML element file in content/html/content/src (behind nsGenericHTMLElement.cpp). With the added complexity of WF2's new input types, nsHTMLInputElement threatens to become unmanageable. | ||
It may be time to consider breaking up the functionality of nsHTMLInputElement into various separate controls and/or files, extending the base class of nsHTMLInputElement. This would include probably the new input types of WF2 and possibly even the HTML 4 input types. | It may be time to consider breaking up the functionality of nsHTMLInputElement into various separate controls and/or files, extending the base class of nsHTMLInputElement. This would include probably the new input types of WF2 and possibly even the HTML 4 input types. | ||
Line 175: | Line 175: | ||
<roc> I guess it means that per-type state could be stored more easily | <roc> I guess it means that per-type state could be stored more easily | ||
</pre> | </pre> | ||
=== <html:input type='number'> === | === <html:input type='number'> === | ||
* Implementation: [https://bugzilla.mozilla.org/show_bug.cgi?id=344616 bug 344616] | * Implementation: [https://bugzilla.mozilla.org/show_bug.cgi?id=344616 bug 344616] |