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

Add ability to "Crossdartify" the source code #1165

Merged
merged 1 commit into from Jun 8, 2016

Conversation

astashov
Copy link
Contributor

If there's a crossdart.json file in the input dir (which can be generated by Crossdart), it will use that file to add links to crossdart.info in the source code block.

Since now we should support links in the source code section, I removed <code> wrapping tags from it, and also now I HTML escape the source code, and use triple mustache curly braces ({{{sourceCode}}}) to insert it to the page.

Hopefully, that will simplify developer's life, when they want to look into the method's implementation.

Example: https://www.dartdocs.org/documentation/hop/0.30.4+2/hop/runHop.html

/cc @sethladd

@sethladd
Copy link
Contributor

cc @keertip @kevmoo

@astashov
Copy link
Contributor Author

astashov commented Jun 3, 2016

Ping :)

@astashov
Copy link
Contributor Author

astashov commented Jun 8, 2016

Any updates? :)

@keertip
Copy link
Collaborator

keertip commented Jun 8, 2016

Sorry, missed this PR. Will take a look.

Anton, could you sync to head? The test failure on the dev build has been fixed.

If there's a crossdart.json file in the input dir (which can be
generated by Crossdart), it will use that file to add links to
crossdart.info in the source code block.

Since now we should support links in the source code section, I removed
`<code>` wrapping tags from it, and also now I HTML escape the source
code, and use triple mustache curly braces (`{{{sourceCode}}}`) to
insert it to the page.

Hopefully, that will simplify developer's life, when they want to look
into the method's implementation.
@astashov
Copy link
Contributor Author

astashov commented Jun 8, 2016

Done! All green now!

@keertip
Copy link
Collaborator

keertip commented Jun 8, 2016

Looks good! Looking forward to seeing it on dartdocs.org.

@keertip keertip merged commit 1430d4f into dart-lang:master Jun 8, 2016
@astashov
Copy link
Contributor Author

astashov commented Jun 8, 2016

Thank you! I'll start regenerating docs with crossdart integration this week.

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

3 participants