[GitHub] [solr] asalamon74 commented on pull request #38: SOLR-14024 Invalid html generated by changes2html.pl

[GitBox <gi...@apache.org>]

asalamon74 commented on pull request #38:
URL: https://github.com/apache/solr/pull/38#issuecomment-804712149


   I'm calling the following to invoke the script:
   
   ```
   ./gradlew changesToHtml
   ```
   It generates the html which can be found in `./solr/documentation/build/site/changes/Changes.html`.
   
   I was using https://validator.w3.org/#validate_by_input as a validator a copied the generated html to it. For the original HTML it gave me a couple hundred errors like:
   
   - Self-closing syntax (/>) used on a non-void HTML element. Ignoring the slash and treating as a start tag.
   - Element ul not allowed as child of element ul in this context. (Suppressing further errors from this subtree.)
   - No li element in scope but a li end tag seen.
   
   The solr/CHANGES.txt change is not unrelated. That empty line caused a problem in the generation (the script was starting a new block because of that).
   
   Visually I tried to keep it as close as possible, there are minor changes, like marginsize.
   
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org