Opened 11 years ago

Last modified 11 years ago

#225 closed defect (FIXED)

extra character shows up in pagination

Reported by: Caleb Davis Owned by: Chris Wilson
Priority: minor Milestone: 0.1.0
Component: ui Keywords:
Cc: Parent Tickets:

Description

.. figure:: http://i.imgur.com/ewXe7.png
   :align: center
   :alt: 
   

See that dash after the arrow? I think I remember how Christopher
fixed this in the last bug.



Subtickets

Change History (5)

comment:1 Changed 11 years ago by Jef van Schendel

Confirmed good sir! I was actually just about to report this. :)

Actually, at my side the whole thing is still underlined, including
the arrow. No idea how that happened.

My question is this though... If we remove the little green/blue
stripe that's in your screenshot, it will look like two floating
elements, the "Older" link and the arrow image. You can click
either of those, but not in between them. It would be preferable to
have a hit target that's a single, large object that includes both
of these elements. And if that's the case then we should also show
it graphically (i.e. it looks the way it acts) for affordance
reasons.

So I'm considering a bigger change here, but I am not sure what it
should be.



comment:2 Changed 11 years ago by Caleb Davis

I'm trying to think of the bug where Christopher fixed this in
another case. Actually, IIRC all that will be needed is a change to
the whitespace in the template code itself. I think that would fix
both our issues.



comment:3 Changed 11 years ago by Caleb Davis

ah, the git repository is easier to search -
132aa9d97a32abfe86bc5de3ea4e2fa62ddc3793



comment:4 Changed 11 years ago by Christopher Allan Webber

Milestone: 0.1.0
Owner: set to Chris Wilson
Status: NewClosed
Easy fix, pushed!

`https://gitorious.org/mediagoblin/mediagoblin/commit/8775f68d33279347f7bc72258d9133f20b192149 <https://gitorious.org/mediagoblin/mediagoblin/commit/8775f68d33279347f7bc72258d9133f20b192149>`_

What I do is set a variable to the generated url and then call it
in two separate anchors. Jef is right... a larger hitbox that
encompasses the whole thing would be better because then it would
skip a "gap". For now, this is at least a fix from an underline
mess.



comment:5 Changed 11 years ago by Will Kahn-Greene

The original url for this bug was http://bugs.foocorp.net/issues/535 .

Note: See TracTickets for help on using tickets.