- Issue created by @GreenSkunk
- 🇮🇳India swatidhurandhar
I have added a patch to add title attribute in anchor with 'main-content' id.
Thanks. - Status changed to Needs review
over 1 year ago 9:42am 28 July 2023 - Open on Drupal.org →Core: 10.1.0-alpha1 + Environment: PHP 7.4 & MySQL 5.7last update
over 1 year ago Waiting for branch to pass - 🇮🇳India ravi kant Jaipur
The patch is not applying for me.
Also path name is not according Drupal instructions. - 🇺🇸United States andystitt829
The patch applies for me. The only feedback I have is that the anchor element that the skip link is targeting within the tags:
<a href="#main-content" id="main-content" tabindex="-1" title="Main Content"></a>
doesn't need the href attribute in it since that's already contained in the skip link that links to it at the top of the page. It can just be:
<a id="main-content" tabindex="-1" title="Main Content"></a>
- Open on Drupal.org →Core: 10.1.0-alpha1 + Environment: PHP 7.4 & MySQL 5.7last update
over 1 year ago Waiting for branch to pass - 🇮🇳India swatidhurandhar
- Status changed to RTBC
over 1 year ago 5:49am 11 August 2023