Hi Alan,

Thank you for reviewing this!


On 11/30/17 00:09, Alan Bateman wrote:


On 30/11/2017 07:45, [email protected] wrote:
:
 
     void document(PrintWriter writer) {
+ writer.println("<!DOCTYPE html>");
         writer.println("<html><head><title>" + comment() + "</title></head>");
         writer.println("<body bgcolor=\"white\">");
         for (Node node : components) {

This looks okay to me. Anything else missing while you are there?

The Copyright element is not present:
 <p><em><a href="">Copyright</a> © 1993, 2017, Oracle and/or its affiliates. All rights reserved.</em></p>

Not sure if we need the charset element:
  <meta charset="utf-8">

Thanks,
Serguei

-Alan

Reply via email to