See attached screenshot (default theme, new install). The markup is as follows, newlines added for clarity, but I think the problem is css padding.

<ul class="pager">
<li class="pager-first odd first"><a href="/frontpage" rel="first" title="Go to first page">« first</a></li>
<li class="pager-previous even"><a href="/frontpage?page=12" rel="prev" title="Go to previous page">‹ previous</a></li>
<li class="pager-ellipsis odd">…</li>
<li class="pager-item even"><a href="/frontpage?page=9" title="Go to page 10">10</a></li>...

Possibly caused by #1598886: Clean up pager theme functions

Files: 
CommentFileSizeAuthor
#6 Home___Site-Install.png68.44 KBjoelpittet
#2 1864708-pager-whitespace.patch627 byteslyricnz
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] Unable to apply patch 1864708-pager-whitespace.patch. Unable to apply patch. See the log in the details link for more information. View
#1 Screenshot 13:12:12 12:56 AM.png4.99 KBlyricnz
#1 1864708-pager-previous-whitespace.patch534 byteslyricnz
PASSED: [[SimpleTest]]: [MySQL] 49,361 pass(es). View
Screenshot 13:12:12 12:40 AM.png6.47 KBlyricnz

Comments

lyricnz’s picture

Status: Active » Needs review
FileSize
534 bytes
PASSED: [[SimpleTest]]: [MySQL] 49,361 pass(es). View
4.99 KB

The CSS looks like this:

.item-list .pager li {
  padding: 0;
}
.item-list .pager a {
  display: inline-block;
  padding: 10px 15px;
}
.item-list .pager .pager-first a {
  padding: 10px 10px 10px 0;
}
.item-list .pager .pager-previous a {
  padding: 10px 0;
}
.item-list .pager .pager-current {
  padding: 0 10px;
}
.item-list .pager .pager-next a,
.item-list .pager .pager-last a {
  padding: 10px 0 10px 10px;
}

It looks like .pager-previous should be "padding: 10px 10px 10px 0;" same as .pager-first

Attached patch for that, and updated screenshot with the change in place.

lyricnz’s picture

FileSize
627 bytes
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] Unable to apply patch 1864708-pager-whitespace.patch. Unable to apply patch. See the log in the details link for more information. View

Status: Needs review » Needs work

The last submitted patch, 2: 1864708-pager-whitespace.patch, failed testing.

joaogarin’s picture

Issue summary: View changes

Cant replicate this in the latet beta.Should be closed.

joelpittet’s picture

Status: Needs work » Closed (cannot reproduce)
FileSize
68.44 KB