New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add paginator.previous_page_path and paginator.next_page_path #942

Merged
merged 8 commits into from Apr 9, 2013

Conversation

Projects
None yet
3 participants
@parkr
Member

parkr commented Apr 9, 2013

It seems like it'd be useful for the Paginator to return the paths of the next and previous pages, rather than just their numbers.

This PR adds these without affecting the previous functionality.

@parkr

This comment has been minimized.

Member

parkr commented Apr 9, 2013

Docs update looks like this:

Screen Shot 2013-04-09 at 9 18 58 PM

@mattr-

This comment has been minimized.

Member

mattr- commented Apr 9, 2013

👍 Looks good to me.

@parkr

This comment has been minimized.

Member

parkr commented Apr 9, 2013

Yay! Merging.

parkr added a commit that referenced this pull request Apr 9, 2013

Merge pull request #942 from mojombo/pagination-paths
Add paginator.previous_page_path and paginator.next_page_path

@parkr parkr merged commit b7646cc into master Apr 9, 2013

1 check passed

default The Travis build passed
Details

@parkr parkr deleted the pagination-paths branch Apr 9, 2013

parkr added a commit that referenced this pull request Apr 9, 2013

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017

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