allow XHTML inside wadl:doc#4
Open
TheConstructor wants to merge 3 commits intoipcsystems:masterfrom
TheConstructor:master
Open
allow XHTML inside wadl:doc#4TheConstructor wants to merge 3 commits intoipcsystems:masterfrom TheConstructor:master
TheConstructor wants to merge 3 commits intoipcsystems:masterfrom
TheConstructor:master
Conversation
… instead of value-of. - Removed adding of pre-tag to wadl:doc containing "Example" - you can now use e.g. code-tags to have monospaced examples
…st some crawlers from following it. This is most useful for statically rendered pages according to this stylesheet which are indexed by search engines.
|
Matthias, Thanks for trying to help make this better! Unfortunately it makes it better for Chrome and Firefox but worse for IE9 and Xalan 2.7.1. The example block is formatted as regular text when running example_wadl.xml through IE9 or transforming it with Xalan. Enclosing it in pre seems to be the only cross-tool approach that carries through formatting. I don't like the way the stylesheet decides if the block should be pre'd. Looking for the text 'Example' in the content is goofy. It requires users to understand to change their content to get it to work. Maybe there's another way to check if the content includes XML? At the very least we can add the rel nofollow. Please let me know what you'd like to do with the example formatting. Mark |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi,
with the current version of your wadl.xsl any tag inside a wadl:doc-tag is discarded upon displaying.
To me a nice documentation should contain e.g. new-lines to be easy to read. To achieve this XHTML-tags like
or
are required.
I modified the wadl.xsl to copy tags contained in wadl:doc-tags for wadl:application and presumably most other wadl:doc-tags.
Please have a look into this pull-request.
Regards
Matthias