Insufficient whitespace between "<previous" and "..." [#1864708]

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

Comment	File	Size	Author
#6	Home___Site-Install.png	68.44 KB	joelpittet
#2	1864708-pager-whitespace.patch	627 bytes	lyricnz
#2
#1	Screenshot 13:12:12 12:56 AM.png	4.99 KB	lyricnz
#1	1864708-pager-previous-whitespace.patch	534 bytes	lyricnz
#1
	Screenshot 13:12:12 12:40 AM.png	6.47 KB	lyricnz

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

lyricnz CreditAttribution: lyricnz commented 12 December 2012 at 13:56

Status:

Active

» Needs review

File	Size
1864708-pager-previous-whitespace.patch	534 bytes

Screenshot 13:12:12 12:56 AM.png	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.