#2062 Add pagination to PRs and Issues
Closed by pingou. Opened by akasurde.
akasurde/pagure i2060  into  master

Download 2062.patch

Fixes https://pagure.io/pagure/issue/2060

Signed-off-by: Abhijeet Kasurde akasurde@redhat.com

Hm, I am not entirely sure about this one because do we really want to have 28 links showing up at https://pagure.io/freeipa/issues ?

Maybe we can have something like this -

Still working on this one?

rebased

Update code.

Hm, to be honest I like the current visual better, this one seems big.

I welcome more input there :)

So for people joining, this is the new/proposed layout:

Screenshot_from_2017-03-24_10-33-11.png

@pingou Definitely, the current visual is much better than that.

I think that the bigger are better, current >> are too small.

How about this http://getbootstrap.com/components/#pagination-default?

Bootstrap pagination should actually be the way to go, yes. But until then, don't change it, those round buttons are completely out-of-sync with the rest of the UI.

@akasurde

I am going to close this PR since there has been no comment from you on this since March 24th.

If you are still interested in working on it, please re-open on and I'll be more than happy to review it :)

Thanks your time and work :)

Pull-Request has been closed by pingou

Metadata