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

Handle spaces in SCDoc internal links #2336

Conversation

crucialfelix
Copy link
Member

Conflict fixed from #2135 and reposted here

James Harkins and others added 3 commits May 28, 2016 10:55
…c-internal-links

* snappizz-scdoc-internal-links:
  Add encodeURIComponent to toc links
  SCDoc: Escape spaces in named anchors (use %20 instead)

# Conflicts:
#	SCClassLibrary/SCDoc/SCDocRenderer.sc
@crucialfelix crucialfelix added the comp: class library SC class library label Sep 14, 2016
@crucialfelix crucialfelix added this to the 3.8 milestone Sep 14, 2016
@crucialfelix
Copy link
Member Author

This is still a bug:

(See: link::#NodeProxy elastic behavior setting::)

renders as:

(See: NodeProxy%20elastic%20behavior%20setting)

But the link is correctly escaped and is clickable but the text should not have been.

This is the case where there is only an anchor.

See:
HelpSource/Other/JITLibChanges3.7.schelp

I think that should be a new bug to fix. Let's merge this and move on.

@crucialfelix crucialfelix merged commit 772bdd6 into supercollider:master Sep 14, 2016
@crucialfelix crucialfelix deleted the topic/snappizz-scdoc-internal-links branch November 4, 2016 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp: class library SC class library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant