Problem/Motivation

The pager is currently un-styled as it has no pager class on the nav element.

As you can see in ./core/themes/stable/templates/navigation/pager.html.twig the nav element should have this class.

Proposed resolution

Add the class and provide patch.

Remaining tasks

Review patch

User interface changes

N/A

API changes

N/A

Data model changes

N/A

CommentFileSizeAuthor
#2 calendar-add-pager-class-2631620-2-d8.patch565 bytesmccrodp
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mccrodp created an issue. See original summary.

mccrodp’s picture

mccrodp CreditAttribution: mccrodp as a volunteer commented
Issue summary: View changes
FileSize
calendar-add-pager-class-2631620-2-d8.patch565 bytes
mccrodp’s picture

mccrodp CreditAttribution: mccrodp as a volunteer commented
Status: Active » Needs review

  • tedbow committed 3ab9f52 on 8.x-1.x authored by mccrodp 
    Issue #2631620 by mccrodp: Add 'pager' class to nav element in calendar-...
tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
CreditAttribution: tedbow at Acquia commented
Status: Needs review » Fixed

Thanks for the patch! Committted!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.