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

Escape search-index item descriptions #31626

Merged
merged 1 commit into from Feb 14, 2016
Merged

Conversation

@mitaa
Copy link
Contributor

@mitaa mitaa commented Feb 13, 2016

Maybe they should be Markdown rendered instead, though that doesn't really seem to work all that well.

fixes #31059

r? @alexcrichton

The item descriptions are included verbatim in search results
causing certain character sequences to misbehave.
@alexcrichton
Copy link
Member

@alexcrichton alexcrichton commented Feb 13, 2016

Manishearth added a commit to Manishearth/rust that referenced this pull request Feb 14, 2016
Maybe they should be Markdown rendered instead, though that doesn't really seem to work all that well.

fixes rust-lang#31059

r? @alexcrichton
bors added a commit that referenced this pull request Feb 14, 2016
@bors
Copy link
Contributor

@bors bors commented Feb 14, 2016

Testing commit dcb3526 with merge f3619ce...

bors added a commit that referenced this pull request Feb 14, 2016
Maybe they should be Markdown rendered instead, though that doesn't really seem to work all that well.

fixes #31059

r? @alexcrichton
@bors bors merged commit dcb3526 into rust-lang:master Feb 14, 2016
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@mitaa mitaa deleted the mitaa:rdoc-srch-escape branch Feb 15, 2016
bors added a commit that referenced this pull request Feb 15, 2016
bors added a commit that referenced this pull request Feb 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants