AXUL/ananke
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

theme(fix): add transform.Plainify to the summary body content of the share links (#758)

Browse source 
You definitely want to **add Plainify transformer first**, because the
body content will become a HTML encoding mess. I left `HTMLEscape` there
as well, just be sure for security reasons. But if you want, I can
remove the second transformer.

Like a tweet on X will look like this:


![image](https://github.com/user-attachments/assets/73ab37b0-decc-47a4-8bbd-cf43cbef1641)


After implementing `transform.Plainify`, it will look much better:


![image](https://github.com/user-attachments/assets/962ba515-72ae-4b46-827f-2c477358d065)

(ignore the red, I don't have X premium pro blabla, it's red because I
hit the max limit of chars)

Signed-off-by: Melroy van den Berg <melroy@melroy.org>
This commit is contained in:
Melroy van den Berg 2024-10-24 00:56:04 +02:00 committed by GitHub
parent 43176864bc
commit 2cfb09e9a3
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
layouts/partials/social/share.html
View file

@ -66,7 +66,7 @@
{{- $setup := .setup -}}
{{- $separator := "&" -}}
{{- $title := $context.Title | transform.HTMLEscape -}}
{{- $description := $context.Summary | transform.HTMLEscape -}}
{{- $description := $context.Summary | transform.Plainify | transform.HTMLEscape -}}
{{- $permalink := $context.Permalink | transform.HTMLEscape -}}
{{- with $setup.separator -}}
{{- $separator = . -}}