Follow-up to #1912608: Update pagination markup for new CSS standards and improved accessibility.

Problem/Motivation

#1912608: Update pagination markup for new CSS standards and improved accessibility pulls out the title and href from links, bypassing a lot of code like hook_link_alter, render array titles, hreflang handling, and all active link code.

We now have the link generator available in Twig thanks to #1777332: Replace theme_menu_link() and menu-tree.html.twig with a single Twig template so let's try and use it.

Proposed resolution

Use the link generator. We may be blocked on #2342745: Allow Twig link function to pass in HTML attributes.

Remaining tasks

Patch

User interface changes

n/a

API changes

n/a

Comments

chr.fritsch’s picture

It's not possible to replace the current pager a tags with the new link function, because the current a tags all have span's inside.

joelpittet’s picture

Status: Active » Postponed
mgifford’s picture

Status: Postponed » Active

Ok, that's fixed.

frantisekivanko’s picture

Status: Active » Closed (fixed)

Wrong status.

Cottser’s picture

Version: 8.0.x-dev » 8.4.x-dev
Status: Closed (fixed) » Active

This should not have been closed, re-opening.