Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
The feed icon on taxonomy term pages has been broken in HEAD awhile, linking to (e.g.)
taxonomy/term/taxonomy/term/1/feed
instead of
taxonomy/term/1/feed
Additionally, #1857256: Convert the taxonomy listing and feed at /taxonomy/term/%term to Views removes the broken feed icon.
Proposed resolution
Re-add the icon once #2251121: Support to add a feed icon on pages with parameters is fixed.
Remaining tasks
TBD
User interface changes
Feed icon returns!
API changes
None.
Postponed until
#2251121: Support to add a feed icon on pages with parameters
Comment | File | Size | Author |
---|---|---|---|
#11 | add_the_feed_icon_back-2251861-11.patch | 2.45 KB | jibran |
Comments
Comment #1
xjmComment #2
jibran#2251121: Support to add a feed icon on pages with parameters got committed and feed url is
/taxonomy/term/1/feed
Comment #3
olli CreditAttribution: olli commented#2251121: Support to add a feed icon on pages with parameters didn't add feed icon back to the default taxonomy term view.
Comment #4
jibranI'm confused. What do you mean? Was I wrong to close this as duplicate? The bug described in IS is not in the HEAD anymore.
Comment #5
olli CreditAttribution: olli commentedNot sure, but I think this issue is about adding the icon back to the page.
Comment #6
olli CreditAttribution: olli commentedHere is the proposed resolution from IS.
Comment #7
olli CreditAttribution: olli commentedNote that this is now in the default display. Not sure if it would be better to add it only for feed_1.
Comment #9
jibranI don't know why it was there when I test it in #2.
Any ways
HEAD
After the Patch
Thanks for the patch and fix @olli.
I don't like this craft but we can live with it.
Comment #11
jibranReroll
Comment #12
alexpottThis issue is a normal bug fix, and doesn't include any disruptive changes, so it is allowed per https://www.drupal.org/core/beta-changes. Committed 3559826 and pushed to 8.0.x. Thanks!