Skip to content

<image><link> in RSS should link to the permalink#1545

Closed
xuhdev wants to merge 1 commit intoadityatelange:masterfrom
xuhdev:image-link
Closed

<image><link> in RSS should link to the permalink#1545
xuhdev wants to merge 1 commit intoadityatelange:masterfrom
xuhdev:image-link

Conversation

@xuhdev
Copy link
Contributor

@xuhdev xuhdev commented Jun 26, 2024

RSS specification says: https://www.rssboard.org/rss-specification#ltimagegtSubelementOfLtchannelgt

is the URL of the site, when the channel is rendered, the image is a link to the site. (Note, in practice the image <title> and should have the same value as the channel's <title> and .

Practically, some RSS readers may produce clicable images that leads to homepage. Linking an image to the image itself doesn't make much sense.

What does this PR change? What problem does it solve?

Was the change discussed in an issue or in the Discussions before?

PR Checklist

  • This change adds/updates translations and I have used the template present here.
  • I have enabled maintainer edits for this PR.
  • I have verified that the code works as described/as intended.
  • This change adds a Social Icon which has a permissive license to use it.
  • This change does not include any CDN resources/links.
  • This change does not include any unrelated scripts such as bash and python scripts.
  • This change updates the overridden internal templates from HUGO's repository.

RSS specification says: <https://www.rssboard.org/rss-specification#ltimagegtSubelementOfLtchannelgt>

> <link> is the URL of the site, when the channel is rendered, the image is a link to the site. (Note, in practice the image <title> and <link> should have the same value as the channel's <title> and <link>.

Practically, some RSS readers may produce clicable images that leads to
homepage. Linking an image to the image itself doesn't make much sense.
xuhdev added a commit to 8hobbies/KamiMod that referenced this pull request Jun 26, 2024
xuhdev added a commit to 8hobbies/KamiMod that referenced this pull request Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants