After installing the theme I noticed view pagers aren't inlined and needed some styling. I used this general styling to fix it all aspects I found:

.pager__items {padding: 10px 0 50px}

.pager__item {display: inline-block; margin: 10px !important; background: #dddddd; font-weight: normal}
.pager__item a {display: inline-block; padding: 5px 10px 5px 10px; min-width: 30px; text-align: center; color: #000 !important}
.pager__item a:hover, .pager__item a:focus {background: #11CBD4}
.pager__item--active {background: #11CBD4}

.pager__item--previous a, .pager__item--next a {display: block; margin: 0 auto; width: 100%}
.pager__item--ellipsis {display: none}
.pager__item--first a, .pager__item--last a {display: block; margin: 0 auto; width: 100%}
CommentFileSizeAuthor
#2 basic_inline-pager-2788365.patch7.32 KBleahtard
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Benia created an issue. See original summary.

leahtard’s picture

Status: Active » Needs review
FileSize
7.32 KB

Thanks Benia,

The lack of styles here are mostly intentional. Basic aims to provide a solid starting point but it doesn't want to provide an opinionated layout.

I took a look at the pager markup and made a few updates with some new templates. I also added some styles that will display the pager inline. I feel this is a fairly standard starting point for most themes.

I have attached a patch for review/testing.

Cheers, Leah

  • leahtard authored 35954c5 on 8.x-1.x
    Issue #2788365 by leahtard: Small view-pager css issue and how I solved...
leahtard’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

Benia’s picture

Issue summary: View changes

Updated with code that worked for me better and more Basic theme dependent.