Skip to content

Instantly share code, notes, and snippets.

@purcell
Created September 9, 2011 09:23
Show Gist options
  • Star 15 You must be signed in to star a gist
  • Fork 6 You must be signed in to fork a gist
  • Save purcell/1205828 to your computer and use it in GitHub Desktop.
Save purcell/1205828 to your computer and use it in GitHub Desktop.
Make will_paginate generate HTML that bootstrap.less will render nicely
# Based on https://gist.github.com/1182136
class BootstrapLinkRenderer < ::WillPaginate::ActionView::LinkRenderer
protected
def html_container(html)
tag :div, tag(:ul, html), container_attributes
end
def page_number(page)
tag :li, link(page, page, :rel => rel_value(page)), :class => ('active' if page == current_page)
end
def gap
tag :li, link(super, '#'), :class => 'disabled'
end
def previous_or_next_page(page, text, classname)
tag :li, link(text, page || '#'), :class => [classname[0..3], classname, ('disabled' unless page)].join(' ')
end
end
def page_navigation_links(pages)
will_paginate(pages, :class => 'pagination', :inner_window => 2, :outer_window => 0, :renderer => BootstrapLinkRenderer, :previous_label => '&larr;'.html_safe, :next_label => '&rarr;'.html_safe)
end
@thatfunkymunki
Copy link

I get a NotImplementedError when I use this, relevant part of stack trace:

will_paginate (3.0.2) lib/will_paginate/view_helpers/link_renderer.rb:81:in `url'
will_paginate (3.0.2) lib/will_paginate/view_helpers/link_renderer.rb:93:in `link'
app/helpers/application_helper.rb:14:in `page_number'

Is this a version error (i.e. should i go down some versions in will_paginate)?

@purcell
Copy link
Author

purcell commented Nov 9, 2011

In the latest will_paginate, the correct superclass for the BootstrapLinkRenderer is instead:

::WillPaginate::ActionView::LinkRenderer

@purcell
Copy link
Author

purcell commented Nov 9, 2011

I'd update the gist to reflect this, but I'm in a rush right now. :-)

@thatfunkymunki
Copy link

Thanks, much appreciated.

@thatfunkymunki
Copy link

@purcell
Copy link
Author

purcell commented Nov 9, 2011

Doh, if I'd seen the "Edit" button, I'd have fixed it right away. More coffee needed...

Cheers!

@jinscoe-zz
Copy link

Do I have to call this helper in a different way? Or should it just work? Currently when I put this in my application helper file and restart my server I do not get the pagination wrapped in the ul and li elements that bootstrap likes.

Thanks!

@purcell
Copy link
Author

purcell commented Jan 3, 2012

See the page_navigation_links helper function above for how to generate the full bootstrap-style html.
-Steve

@jinscoe-zz
Copy link

Thanks for the quick answer Steve!

I have replaced "will_paginate @feeditems" in my view with

<%= page_navigation_links @feed_items %>

I get an error of: NotImplementedError in Pages#home

Forgive my ignorance. Just now making my first rails app. Thanks for any help!

@purcell
Copy link
Author

purcell commented Jan 3, 2012

Yep, see @thatfunkymunki's comments and modified version above. I should really go ahead and update this gist to incorporate his changes.

@jinscoe-zz
Copy link

Thanks much for your help!

@dylan-conlin
Copy link

<%= page_navigation_links @users %> worked for me!

@purcell
Copy link
Author

purcell commented Feb 4, 2012

@fellytone84 - Glad it worked for you.

I've updated this gist to directly reflect @thatfunkymunki's fixes here...

@smoothdvd
Copy link

@purcell I've just added classname option to page_navigation_links, now it can support Bootstrap 2.0's flexible alignment option: .pagination-center, .pagination-right, default option is pagination.

@shanepinnell
Copy link

Thanks, works great!

@bertomartin
Copy link

how do you specify that a link is remote, and how about specifying the params hash? There's no way to pass in those parameters to "page_navigation_links". For example, I need to pass in=> :remote => true.

@purcell
Copy link
Author

purcell commented Mar 6, 2012

@bertomartin There's nothing stopping you from changing the definition of page_navigation_links so that it allows you to merge in extra options.

@bertomartin
Copy link

@purcell, I tried doing that. Had a problem though, but I'll look into it again (very late last night). Thanks for this script.

@tbhockey
Copy link

tbhockey commented Apr 6, 2012

Very nice. Thanks!

@sur
Copy link

sur commented Aug 1, 2012

worked out well. thanks!

@leonmitchelli
Copy link

Worked like a charm, thanks!

@shotty01
Copy link

shotty01 commented Oct 2, 2012

this was great. i didn't want to use a gem, and this gist just nails it. thanks a lot!

@scaryguy
Copy link

It worked like a charm! Thank you!

But one thing, all this is not about LESS or SASS. I'm working with SASS and no problem at all. Maybe you can consider to change gist's title.

@mahajanabhij
Copy link

This worked beautifully thanks!

@lucashour
Copy link

Thank you! It works perfect! 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment