Pagination span support for disabled / active #1406

Closed
vespakoen opened this Issue Jan 29, 2012 · 4 comments

3 participants

@vespakoen

Having a <span> instead of an <a> for the disabled buttons works better for me.
I think it is better for bootstrap to support this by default.

@mdo
Bootstrap member

It's too late to add this to 2.0, and I'm unsure if we truly need this for the default pagination component. Reason being, we can just nuke the cursor and it behaves the exact same way without changing markup or anything. Sound reasonable?

And, can you elaborate on why you would want this approach? Does another tool lock you into this?

@vespakoen

Hi Mark,

In quite some PHP frameworks the span for inactive page links is default.
When changing the span into an a I ran into a problem with "removing" te action, when i made the URL # it was relative to the base URL so when I clicked on a disabled link it would send me to the homepage.

So in the end I had to go for span elements and change the CSS.

Just trying to save another guy the pain ;)

@mdo
Bootstrap member

Integrated into 2.0.2-wip.

@mdo mdo closed this Feb 20, 2012
@tatsuquito

+1. A few more styles are necessary to display span correctly:

.pagination span,
.pagination li:first-child span,

@stempler stempler pushed a commit to stempler/bootstrap that referenced this issue Apr 11, 2014
@mdo mdo (less) enable use of span in place of a for .disabled in pagination, …
…per #1406
dff8791
@stempler stempler pushed a commit to stempler/bootstrap that referenced this issue Nov 4, 2014
@mdo mdo (less) enable use of span in place of a for .disabled in pagination, …
…per #1406
bf94df5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment