HTML-Schließungs-Tags in Push-Benachrichtigungen anzeigen

Don · 21. Dezember 2022 um 18:57

Hallo,

Die neuen HTML-Schlusstags für das Hashtag-Symbol </use></svg> erscheinen in den Push-Benachrichtigungen.

Android 10
Chrome 108.0.5359.128

MacOS 12.6.1
Chrome 108.0.5359.124

Auf den Bildern ist die Testseite das Hashtag.

Danke

martin · 24. Januar 2023 um 04:41

Ich habe gerade einen Fix dafür zusammengeführt:

github.com/discourse/discourse

FIX: Do not add empty use/svg tags in ExcerptParser

main ← issue/use-svg-empty-tags-excerpt-hashtag

opened 03:07AM - 24 Jan 23 UTC

martin-brennan

+39 -5

There was an issue where if hashtag-cooked HTML was sent to the ExcerptParser w…ithout the keep_svg option, we would end up with empty </use> and </svg> tags on the parts of the excerpt where the hashtag was, in this case when a post push notification was sent. Fixed this, and also added a way to only display a plaintext version of the hashtag for cases like this via PrettyText#excerpt. Followup to 9d50790530459535164870bd49e712c30d2d5e8e **Example**: ![image](https://user-images.githubusercontent.com/920448/214204584-96c70b61-b947-449e-b563-d999383c3938.png) c.f. https://meta.discourse.org/t/icon-html-closing-tags-in-push-notification/249607

Thema		Antworten	Aufrufe
Tag icon doesn't appear in the hashtag autocomplete Support tag-icons	0	29	27. Februar 2025
Wrong icon referenced for "Tracked" notification level in Tag preferences UX	6	1407	22. Mai 2018
Android app notifications via red number icon on home screen Feature	14	7080	12. Januar 2017
Add the icons as well in the hashtag #category Feature category-icons	0	221	24. August 2023
Category and tag badges are misaligned and lose their colour in quotes UX	5	155	22. Juli 2025

HTML-Schließungs-Tags in Push-Benachrichtigungen anzeigen

Verwandte Themen