RFR: JDK-8326694: Defer setting of autocapitalize attribute
Hannes Wallnöfer
hannesw at openjdk.org
Wed Feb 28 18:02:53 UTC 2024
On Tue, 27 Feb 2024 18:48:22 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:
> We could set "standard" attributes in the `HtmlTree.INPUT` factory method, optionally conditional on the input type. This would allow the code to set `autocomplete` just once, in that method, instead of 3 places, as here. It would also give a single place to explain why we do _not_ set `autocapitalize` in the HTML tree.
This sounds like a good idea, but there are a number of problems when trying to put it into practice. To start with, there are two `HtmlTree.INPUT` factory methods, both are used for text inputs as well as other kinds of controls such as checkboxes and reset buttons where the `autocomplete` attribute does not make sense. Both of these methods consist of a single `return` statement with multiple chained "fluent" setters, so setting an attribute depending on the kind of input would complicate the code quite a bit. With these issues considered, I would prefer to keep the code in its current form.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/18029#issuecomment-1969543178
More information about the javadoc-dev
mailing list