Merge pull request #21 from benbalter/entities
Do not escape smart quotes
This commit is contained in:
commit
9f53718796
|
@ -3,19 +3,10 @@ module Jekyll
|
|||
|
||||
attr_accessor :context
|
||||
|
||||
HTML_ESCAPE = {
|
||||
"\u201c".freeze => '“'.freeze,
|
||||
"\u201d".freeze => '”'.freeze
|
||||
}
|
||||
HTML_ESCAPE_REGEX = Regexp.union(HTML_ESCAPE.keys).freeze
|
||||
|
||||
def render(context)
|
||||
@context = context
|
||||
output = template.render!(payload, info)
|
||||
|
||||
# Encode smart quotes. See https://github.com/benbalter/jekyll-seo-tag/pull/6
|
||||
output.gsub!(HTML_ESCAPE_REGEX, HTML_ESCAPE)
|
||||
|
||||
output
|
||||
end
|
||||
|
||||
|
|
|
@ -32,19 +32,6 @@ describe Jekyll::SeoTag do
|
|||
expect(subject.render(context)).to match(/<title>foo<\/title>/)
|
||||
end
|
||||
|
||||
it "escapes titles" do
|
||||
site = site({"title" => 'Jekyll & "Hyde"'})
|
||||
context = context({ :site => site })
|
||||
expect(subject.render(context)).to match(/<title>Jekyll & “Hyde”<\/title>/)
|
||||
end
|
||||
|
||||
it "escapes descriptions" do
|
||||
site = site({"description" => 'Jekyll & "Hyde"'})
|
||||
context = context({ :site => site })
|
||||
expected = /<meta name="description" content="Jekyll & “Hyde”" \/>/
|
||||
expect(subject.render(context)).to match(expected)
|
||||
end
|
||||
|
||||
it "uses the page description" do
|
||||
page = page({"description" => "foo"})
|
||||
context = context({ :page => page })
|
||||
|
|
Loading…
Reference in New Issue