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

Xhtml fixes #1

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

Xhtml fixes #1

wants to merge 21 commits into from

Conversation

capncodewash
Copy link

Hi there,
This branch contains a fixes to get the output from wsdl-viewer to validate as XHTML, and to fix some recursion issues I was having with the output.

@tohvu
Copy link
Contributor

tohvu commented Feb 25, 2013

Hey, first of all, thanks for the input!

The decision for using inline JQuery was made because we wanted the document formatting functioning even without an active internet connection. Do you have any suggestions how we could improve based on that principle?

@capncodewash
Copy link
Author

Yeah, that makes sense.

I did try a variety of things to get it to validate using inline JS (including wrapping the CDATA sections in JS comments, see here: capncodewash@f05e9a8 ), however nothing seemed to allow it to validate.

It seems that inline JavaScript is inherently breaky in XHTML. Is there any particular requirement for the stylesheet to generate XHTML, or would the HTML5 doctype be a possibility?

@tohvu
Copy link
Contributor

tohvu commented Feb 25, 2013

Right now I can't think of anything against switching to HTML5 doctype.

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.

2 participants