Skip to content
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

Html5 #72

Open
wants to merge 29 commits into
base: master
Choose a base branch
from
Open

Html5 #72

wants to merge 29 commits into from

Conversation

Iain-S
Copy link

@Iain-S Iain-S commented Apr 3, 2019

Hello

Hope you don't mind but I have made a couple of additions to isicp:

  • Using validate.py and the W3C validator, I have changed the invalid xhtml into valid html5. This should help with development because a pre-commit hook can be added to check that the rendered files are always valid html5 from now on.

  • I added render_all.py. This does the same as render_all.sh but can be run more easily on Windows. Keeping render_all.sh is probably unnecessary but I have left it in for now.

  • A few other fixes too small to mention here.

…@ref tags but that doesn't seem to be happening, at least not consistently.  I have inserted/corrected some section links manually but there are still some to do.
…_all.py does the same as render_all.sh but is in python, making it convenient for Windows users.
…ers to .gitignore. Copying xhtml changes to index.html.
@gbezyuk
Copy link
Collaborator

gbezyuk commented Apr 5, 2019

@IIIIllllIIIIllllIIIIllllIIIIllllIIIIll, would you take a look?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants