RFR: 8294158: HTML formatting for PassFailJFrame instructions

Tue Sep 12 13:52:43 UTC 2023

On Tue, 12 Sep 2023 13:15:26 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

> Could be… On the other hand, in post XHTML era, the elements are usually in lower-case. Thus we should stick with it and use lowercase HTML elements anyway.

https://www.w3schools.com/html/html_elements.asp

> HTML is Not Case Sensitive
> 
> HTML tags are not case sensitive: `<P>` means the same as `<p>`.
> 
> The HTML standard does not require lowercase tags, but W3C recommends lowercase in HTML, and demands lowercase for stricter document types like XHTML.
> 
> At W3Schools we always use lowercase tag names.

So this is still a recommendation, not a requirement (we don't use XHTML).
But I agree that this can be helpful for a certain amount of consistency across tests(they can still use other tags in uppercase).

> Although neither <html> nor <body> are required, I prefer to always add them. These two elements don't add much overhead, do they?

It is a matter of personal taste. 
Normally, I prefer to indent a content of such tags by extra spaces, andThis can create additional formatting difficulties under the 80(120) character limit per line.
In those cases, I would prefer to omit those tags.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/15660#discussion_r1323077063