Skip to content

Commit

Permalink
Fix apostrophe in news title for search (#2879)
Browse files Browse the repository at this point in the history
Fix #2876 

Weirdly enough, we already escape the title a few lines above the
changed one, but had to escape it again here.

Signed-off-by: George Araújo <[email protected]>
  • Loading branch information
george-gca authored Dec 6, 2024
1 parent 957dc2c commit 8121ad0
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion _includes/scripts/search.liquid
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@
{%- assign title = item.title | newline_to_br | replace: "<br />", " " | replace: "<br/>", " " | strip_html | strip_newlines | escape | strip -%}
{%- endif -%}
id: "{{ collection.label }}-{{ title | slugify }}",
title: '{{ title | emojify | truncatewords: 13 }}',
title: '{{ title | escape | emojify | truncatewords: 13 }}',
description: "{{ item.description | strip_html | strip_newlines | escape | strip }}",
section: "{{ collection.label | capitalize }}",
{%- unless item.inline -%}
Expand Down

0 comments on commit 8121ad0

Please sign in to comment.