-
Notifications
You must be signed in to change notification settings - Fork 107
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Invalid XHTML #29
Comments
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! If you don't want / don't care about dark mode you will be able to disable it in the upcoming release by setting
|
Thanks for spotting, fixed. Only tested dark mode on mobile. For the arrows I simply override the icon used in <script type="text/javascript">
var arrowRight = '▷';
</script> |
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
The text was updated successfully, but these errors were encountered: