On Thu, Dec 16, 2021 at 11:46:08AM -0800, Per Bothner wrote: > > > I know nothing about it - appears to have been added by Gavin: > > 2020-11-25 Gavin Smith <[email protected]> > > data-manual attribute > > * tp/Texinfo/Convert/HTML.pm (_convert_xref_commands): > Set data-manual attribute instead of class="texi-manual" > on links to other Texinfo manuals. > > > I propose to add a customization variable like 'NO_CUSTOM_ATTRIBUTE' to > > set to 1 to avoid data-manual and use it too for future custom > > attributes. > > Why bother? What problem does it solve? It avoids some warnings from > an overly-picky validator. I don't see that as a strong reason to change > anything.
I do not like non standard attribute names in HTML output because I like to have as simple as possible HTML, it helps with understanding and checking the resulting HTML files, it solves the problem of having customized attributes. I also understand the use for custom attributes, the data-manual doesn't only replace class="texi-manual" but also provide manual name of target which can probably be useful sometime. > > > * Tables which do their header and footer inside thead and tfoot, but > > > not the content inside tbody (which seems valid in XHTML 1.0 but not > > > in 1.1). > > > > I think that it would make sens to always use tbody. > > Again, it's just a warning - but if it's easy to fix, we might as well. And also I think that it is better to have it not only because it gives a warning but because it is useful to have this structure for css or subsequent parsing of the HTML, and also because it is a similar structure with docbook output, and also consistent with the Texinfo tree structure. -- Pat
