Closed (fixed)
Project:
Date
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
21 Apr 2013 at 18:25 UTC
Updated:
5 Jul 2014 at 16:10 UTC
Jump to comment: Most recent
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">« Previous</a>
</li>
<li class="date-next">
<a href="..." rel="nofollow">Next »</a>
</li>
</ul>I attached a patch to fix this.
| Comment | File | Size | Author |
|---|---|---|---|
| date-views-pager-nbsp.diff | 884 bytes | Harmageddon |
Comments
Comment #1
rv0 commentedreminds me of #1301146: «Prev should be « Prev in date-views-pager.tpl.php
edit:but its something different, so ignore this post ;)
Comment #2
podarokdate-views-pager-nbsp.diff queued for re-testing.
Comment #3
kevin morse commentedI would appreciate if this could happen as well. I'm having display issues that go away if the nbsp is remove.
Comment #4
kevin morse commenteddate-views-pager-nbsp.diff queued for re-testing.
Comment #5
kevin morse commentedPatch 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.
Comment #7
podarokcommited
thanks!