Site template: Escape title and description where it is used in HTML #4606

Merged
merged 1 commit into from Feb 29, 2016

Conversation

Projects
None yet
3 participants
@yous
Contributor

yous commented Feb 28, 2016

Follows after #4307. site.title, page.title, and site.description are usually written in plaintext, so escaping is needed.

@parkr parkr changed the title from Escape title and description to Site template: Escape title and description where it is used in HTML Feb 29, 2016

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Feb 29, 2016

Member

Agreed! 👍

@jekyllbot: merge +bug

Member

parkr commented Feb 29, 2016

Agreed! 👍

@jekyllbot: merge +bug

jekyllbot added a commit that referenced this pull request Feb 29, 2016

@jekyllbot jekyllbot merged commit 0618ae8 into jekyll:master Feb 29, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@parkr parkr added the bug label Feb 29, 2016

jekyllbot added a commit that referenced this pull request Feb 29, 2016

@yous yous deleted the yous:escape-template branch Mar 1, 2016

@yous

This comment has been minimized.

Show comment
Hide comment
@yous

yous Mar 1, 2016

Contributor

Thanks!

Contributor

yous commented Mar 1, 2016

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment