doxygen-awesome-css icon indicating copy to clipboard operation
doxygen-awesome-css copied to clipboard

Invalid XHTML

Open dsieger opened this issue 4 years ago • 2 comments

Hi,

First of all: Thanks for your work, this is indeed awesome. I did something similar for our project documentation, but your solution is much cleaner and more comprehensive. I immediately adopted it, see our user guide for an example. I did some minor changes, mostly to the sidebar, maybe you are interested in some of those.

Anyway, the main issue I had was invalid XHTML markup: If you run your site through the W3C validator you'll notice a couple of issues you might want to fix.

Thanks, Daniel

dsieger avatar Jul 01 '21 14:07 dsieger

Thanks for pointing that out, it seems like I have mixed in some html5 by accident...

Thanks for sending me a link of your documentation, it's always helpful to see what others do with the theme! I like the style of the arrows in the sidebar! It seems like the adjustments that you made don't work for the dark theme, though:

Bildschirmfoto 2021-07-01 um 18 04 14

If you don't want / don't care about dark mode you will be able to disable it in the upcoming release by setting

<html class="light-mode">

jothepro avatar Jul 01 '21 16:07 jothepro

Thanks for spotting, fixed. Only tested dark mode on mobile.

For the arrows I simply override the icon used in navtree.js by adding a snippet to my header.html:

<script type="text/javascript">
  var arrowRight = '&#9655;';
</script>

dsieger avatar Jul 01 '21 16:07 dsieger