diff options
| author | Eugen Rochko <eugen@zeonfederated.com> | 2018-02-16 04:40:53 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-02-16 04:40:53 +0100 |
| commit | 1122579216a0ba6d081ff9ad14a1b501636f9601 (patch) | |
| tree | 19b9273e8449eb0fea7d9762632b7d48058102b9 /app | |
| parent | 478ca39e5e8463044a259388459da56d2141e104 (diff) | |
Do not hide NSFW media/CW'd text in OpenGraph tags (#6479)
Reasoning: HTML title tag affects everyone. But OpenGraph only affects when somebody is deliberately sharing the content, usually in an environment where such content is expected. Hiding the content in OpenGraph tags results in deceitful previews which inhibit the shareability of the post. Example: Somebody writes a clever post about politics but kindly puts a "uspol" content warning on it. Mastodon users are thankful, but sharing this post on another platform results in non-Mastodon users believing the entire contents of the post is "uspol" and not clicking through/reading and re-sharing.
Diffstat (limited to 'app')
| -rw-r--r-- | app/views/stream_entries/_og_description.html.haml | 5 | ||||
| -rw-r--r-- | app/views/stream_entries/_og_image.html.haml | 2 |
2 files changed, 2 insertions, 5 deletions
diff --git a/app/views/stream_entries/_og_description.html.haml b/app/views/stream_entries/_og_description.html.haml index d2fa99e63..9c24e0a61 100644 --- a/app/views/stream_entries/_og_description.html.haml +++ b/app/views/stream_entries/_og_description.html.haml @@ -1,4 +1 @@ -- if activity.is_a?(Status) && activity.spoiler_text? - = opengraph 'og:description', activity.spoiler_text -- else - = opengraph 'og:description', activity.content += opengraph 'og:description', [activity.spoiler_text, activity.text].reject(&:blank?).join("\n\n") diff --git a/app/views/stream_entries/_og_image.html.haml b/app/views/stream_entries/_og_image.html.haml index 1056c1744..bd14ef701 100644 --- a/app/views/stream_entries/_og_image.html.haml +++ b/app/views/stream_entries/_og_image.html.haml @@ -1,4 +1,4 @@ -- if activity.is_a?(Status) && activity.non_sensitive_with_media? +- if activity.is_a?(Status) && activity.media_attachments.any? - activity.media_attachments.each do |media| - if media.image? = opengraph 'og:image', full_asset_url(media.file.url(:original)) |