Hi Sergey,

Thanks for the review. 
I added the closing "</li>" tag, since it wouldn't hurt and the tool was 
reporting it.
As for the style spec, thanks for your suggestion, I have moved it to the style 
block in the head section.
Here is the updated webrev: 
http://cr.openjdk.java.net/~kaddepalli/8213048/webrev01/

Thanks,
Krishna

-----Original Message-----
From: Sergey Bylokhov 
Sent: Saturday, November 10, 2018 2:19 AM
To: Krishna Addepalli <krishna.addepa...@oracle.com>; awt-dev@openjdk.java.net
Subject: Re: <AWT Dev> [12]RFR: JDK-8213048: Invalid use of HTML5 in java.awt 
files

Hi, Krishna.

The closed tag "</li>" is optional and might be omitted, it was dropped in most 
of our specifications.

BTW I think that in most cases in this specs the style "align=" for the table 
can be dropped as well and some default could be used.
  
On 09/11/2018 04:34, Krishna Addepalli wrote:
> Hi All,
> 
> Please review a simple fix for updating html attributes to use CSS style 
> options to comply with HTML5.
> 
> JDK-8213048: https://bugs.openjdk.java.net/browse/JDK-8213048
> 
> Webrev: http://cr.openjdk.java.net/~kaddepalli/8213048/webrev00/
> 
> Thanks,
> 
> Krishna
> 


--
Best regards, Sergey.

Reply via email to