Re: [PATCH 0/9] Fix broken HTML in API docs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Feb 29, 2024 at 03:58:14PM +0100, Peter Krempa wrote:
> While checking API docs after recent migration to gitlab-pages I've
> noticed that the footer is not properly rendered.
> 
> A deeper dig showed that the issue is that empty <div> elements, while
> formatted by the XML output version of XSLT conversion is turned into
> the non-pair empty variant. This contradicts the HTML standard which we
> declare to use for our pages and thus is mis-parsed into dom, where
> every empty div is treated as a start of a div, thus nesting everything.
> 
> This series fixes the above and also many other errors pointed out by
> the HTML validator at:
> 
> https://validator.w3.org/
> 
> Compare:
> 
>  https://libvirt.org/html/libvirt-libvirt-domain.html
> 
>  vs
> 
>  https://pipo.sk.gitlab.io/-/libvirt/-/jobs/6288946946/artifacts/website/html/libvirt-libvirt-domain.html
> 
> and the validation:
> 
>   https://validator.w3.org/nu/?doc=https%3A%2F%2Flibvirt.org%2Fhtml%2Flibvirt-libvirt-domain.html
> 
>   vs
> 
>   https://validator.w3.org/nu/?doc=https%3A%2F%2Fpipo.sk.gitlab.io%2F-%2Flibvirt%2F-%2Fjobs%2F6288946946%2Fartifacts%2Fwebsite%2Fhtml%2Flibvirt-libvirt-domain.html
> 
> Also non-API pages validation:
> 
>  https://validator.w3.org/nu/?doc=https%3A%2F%2Flibvirt.org%2Fdocs.html
> 
>  vs
> 
>  https://validator.w3.org/nu/?doc=https%3A%2F%2Fpipo.sk.gitlab.io%2F-%2Flibvirt%2F-%2Fjobs%2F6288946946%2Fartifacts%2Fwebsite%2Fdocs.html
> 
> Peter Krempa (9):
>   docs: site: Don't generate '<?xml' header for HTML documents
>   docs: page: Add 'lang="en"' for all HTML output documents
>   docs: page: Fix declaration of main javascript source
>   docs: index: Fix import of blog planet javascript
>   docs: newapi: Don't generate empty <div> in template for ACL
>     permissions
>   docs: newapi: Avoid empty <div>s when there is no description
>   docs: newapi: Avoid table where every row has an cell with 'colspan'
>   docs: newapi: Properly skip ACL entries if empty
>   docs: newapi: Fix generation of type definition tables
> 
>  docs/index.rst  |   2 +-
>  docs/newapi.xsl | 158 ++++++++++++++++++++++++++----------------------
>  docs/page.xsl   |   7 ++-
>  docs/site.xsl   |   5 +-
>  4 files changed, 93 insertions(+), 79 deletions(-)

Reviewed-by: Pavel Hrdina <phrdina@xxxxxxxxxx>

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Devel mailing list -- devel@xxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxx

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux