Testing instructions:
1. Create a date view that has a pager (e.g. a month view with Calendar).
2. Look at the pager output.

Inside the li elements of the pager, a whitespace (& nbsp;) is added, which doesn't seem to have any purpose (as far as I see):

<ul class="pager">
  <li class="date-prev">
      <a href="..." rel="nofollow">&laquo; Previous</a>
      &nbsp;
  </li>
  <li class="date-next">
      <a href="..." rel="nofollow">Next &raquo;</a>
      &nbsp;
  </li>
</ul>

I attached a patch to fix this.

CommentFileSizeAuthor
date-views-pager-nbsp.diff884 bytesHarmageddon
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rv0’s picture

reminds me of #1301146: «Prev should be « Prev in date-views-pager.tpl.php
edit:but its something different, so ignore this post ;)

podarok’s picture

date-views-pager-nbsp.diff queued for re-testing.

Kevin Morse’s picture

Issue summary: View changes

I would appreciate if this could happen as well. I'm having display issues that go away if the nbsp is remove.

Kevin Morse’s picture

date-views-pager-nbsp.diff queued for re-testing.

Kevin Morse’s picture

Status: Needs review » Reviewed & tested by the community

Patch passed testing, worked correctly on my test site, and is VERY straightforward. I really can't see what problems this would cause so I think it is RTBC.

  • Commit 2b40d8f on 7.x-2.x authored by Harmageddon, committed by podarok:
    Issue #1976014 by Harmageddon: Remove unnecessary &nbsp; in date views...
podarok’s picture

Status: Reviewed & tested by the community » Fixed

commited
thanks!

Status: Fixed » Closed (fixed)

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