[openstreetmap/openstreetmap-website] Remove html type from changeset feed entry titles (PR #4522)
Anton Khorev
notifications at github.com
Wed Feb 14 16:38:28 UTC 2024
Look at https://github.com/openstreetmap/openstreetmap-website/commit/fc9768f34c803f9a9746d1705af097655d55f3e2. It added `type="html"` to changeset feed entries for the reasons stated in the commit message. But is it actually correct? Let's check.
I created a couple of changesets. This is how their comments are shown on the `/history` page. I expect to see the same in a feed reader.

This is how their titles look in `/history/feed` output. They are always escaped once.
<title type="html">Changeset 58 - is it going to <em>emphasize</em>?</title>
...
<title type="html">Changeset 57 - this is unescaped (<); this is escaped (&lt;)</title>
This is how they look in [Liferea](https://en.wikipedia.org/wiki/Liferea). Not what's expected.

This is how they look after `type="html"` is removed.

You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/4522
-- Commit Summary --
* Remove html type from changeset feed entry titles
-- File Changes --
M app/views/changesets/index.atom.builder (2)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/4522.patch
https://github.com/openstreetmap/openstreetmap-website/pull/4522.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4522
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/4522 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240214/7e7b6898/attachment.htm>
More information about the rails-dev
mailing list