Make WordPress Core

Changes between Initial Version and Version 1 of Ticket #47670, comment 16


Ignore:
Timestamp:
10/03/2019 04:47:48 AM (7 years ago)
Author:
sabernhardt
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #47670, comment 16

    initial v1  
    1 I still think there are cases where the feed URL is valuable to include; however, I do realize that it makes little sense to have both links within the same heading tag. Besides, anyone can add a separate link to the feed URL if they intend to make it available.
     1While I think there are cases where the feed URL is valuable to include, I realize that it makes little sense to have both links within the same heading tag. Also, anyone can add a separate link to the feed URL if they intend to make it available.
    22
    33So if it's not too late for 5.3, I'd like to suggest a temporary option that:
    441. Changes the URL on the icon to the same URL as the text link (unifying them in preparation for an actual solution in the next release cycle),
    552. Hides the icon link from screen readers and keyboard navigation with both `aria-hidden="true"` and `tabindex="-1"` since that link would be redundant,
    6 3. Still allows people to click (or touch) the icon with the new URL, and
     63. Allows people to click (or tap) the icon for the preferred URL, and
    774. Replaces the hard-coded `alt` text with an escaped text string that has already been translated.
    88
    99This would probably warrant a dev note, something like:
    10  "RSS feed URL removed from the RSS Widget heading so there is only one link there. A link to the RSS feed itself can still be added below the RSS Widget in a separate widget (or elsewhere on the page)."
     10 "The RSS feed URL has been removed from the RSS Widget heading so there is only one link there. A link to the RSS feed itself can still be added below the RSS Widget in a separate widget (or elsewhere on the page)."
    1111
    1212Then we can revisit how to adjust to a single link in various themes for 5.4 (open a new ticket?).