IMG Tag wird nicht geschlossen

[Timbalu]

Nur zur Einordnung:

zumal denjenigen, die mit einem WYSIWYG-Editor bloggen, das Problem gar nicht bewusst sein dürfte.

Die betrifft es auch nicht, denn der macht es automatisch "richtig" als <img />.

[xoxyss]

Also den Vorschlag von onli habe ich getestet musste allerdings noch zwei kleine Anpassungen vornehmen:

Code: Select all

<content type="html">
    <![CDATA[{$entry.feed_body} {$entry.feed_ext}]]>
</content>

Das Ergebnis ist, im IE 11 geht es allerdings nur mit type="html" andernfalss werden HTML-Tags nicht interpretiert. TT-RSS Android App kann es auch. Firefox Desktop zeigt garnichts an und Firefox mobile interbretiert HTML nicht...

Also so richtig sauber ist die Lösung auch nicht, bleibt wohl nur <img /> von Hand anpassen oder xhtmlcleanup benutzen...

[Timbalu]

Was sagt type html5 ?

[xoxyss]

Teste ich sofort.

Aber es gibt noch ein weiteres Problem:

Code: Select all

This feed does not validate.
    line 49, column 214: XML parsing error: <unknown>:49:214: not well-formed (invalid token) [help]
        ... ttp://board.s9y.org/viewtopic.php?f=10&t=20420' title="Serendipity Forum ...

Ich vermute es liegt an dem & in der URL... Also müsste man bei Links auch noch was machen. Ich hab jetzt erstmal auf RSS umgestellt teste aber mögliche Lösungen für Atom weiter

[xoxyss]

@Timbalu
html5 is not a valid MIME type

[Timbalu]

Schade - ich vermeinte irgendwie quer gelesen zu haben, dass "man" es aufnehmen wollte... Ein Versuch war es wert.

[xoxyss]

Kurze Frage. Wie gehts jetzt mit dem Thema weiter?

[onli]

Sieht so aus, als kriegen wir keine 100%ige Lösung hin. Mein Vorschlag wäre, es so sauber wie möglich zu machen:

Code: Select all

<content type="html">
    <![CDATA[{$entry.feed_body} {$entry.feed_ext}]]>
</content>

Nebenbei, vielen Dank fürs Testen.

Wir haben Atom in 2.0 sowieso versteckt, es wird nicht mehr angezeigt wenn man das nicht manuell aktiviert. Zusammen mit dem möglichst sauberen Code kann das dann in meinen Augen dann so drinbleiben. Echte Feedreader müssten damit sowieso zurechtkommen.

Ich lass das Wochenende über Zeit, falls das gevetoet wird, und commite das ansonsten am Montag.

Für dich bliebe dann nur, zu akzeptieren dass der Atom-Feed im Browser nicht ordentlich angezeigt wird. Das sollte aber kein Problem sein, denn dort muss er ja auch nicht angezeigt werden. Er muss in Feedreadern funktionieren, und das sollte er sowohl mit dem derzeitigen Code als auch mit der CDATA-Lösung. Der verbliebene & -> &amp-Hinweis ist kein Fehler, an dem sich auch nur ein einziger Parser verschlucken wird, und meinem Verständnis nach pure Korinthenkackerei vom Validator.

Nachtrag: Falls du Lust hast, noch weiter zu helfen, könntest du schauen, ob die CDATA-Lösung im Feedreader funktioniert.

[Timbalu]

We opened an issue for this
https://github.com/s9y/Serendipity/issues/354