Skip to content
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

Paginator for backend #223

Closed
ccab opened this issue Oct 12, 2015 · 9 comments
Closed

Paginator for backend #223

ccab opened this issue Oct 12, 2015 · 9 comments

Comments

@ccab
Copy link
Contributor

ccab commented Oct 12, 2015

I'd like to add the paginator also in the backend and btw sorting the columns to improve the user experience

@bocharsky-bw
Copy link
Contributor

👍 for paginator since we already have it in the frontend

@voronkovich
Copy link
Contributor

👍

@stof
Copy link
Member

stof commented Oct 12, 2015

I'm 👍 on it. Showing unpaginated lists is a bad idea IMO.

@javiereguiluz does it need to be implemented here or in your admin bundle though ?

@javiereguiluz
Copy link
Member

I'm 👎 because the demo application is not something you use in the real world ... but something you use to learn how real-world-looking apps work. If we already have a pagination example in the frontend, that's enough to show the newcomer how it's done.

@stof I don't fully understand your comment because this demo app doesn't use any bundle for the admin backend.

@stof
Copy link
Member

stof commented Oct 12, 2015

oh, I thought we were using your EasyAdminBundle. I should have checked

@voronkovich
Copy link
Contributor

@javiereguiluz, but @ccab wants to add a sorting and etc., no only the pagination

@javiereguiluz
Copy link
Member

@voronkovich in that case I also think it's a bad idea. The "good practice" here should be: if you need sorting, pagination and all those common features ... use Sonata or any other backend bundle.

@ccab
Copy link
Contributor Author

ccab commented Oct 12, 2015

Ok so let's keep it in the frontend

@javiereguiluz
Copy link
Member

Closing it as "won't fix" for the mentioned reasons.

javiereguiluz added a commit that referenced this issue Oct 5, 2016
…simensen)

This PR was merged into the master branch.

Discussion
----------

Ensure default Post sort order shows newer Posts first

In at least #223 the topic of sorting on the **admin backend** was mentioned. This is the first time I've loaded the demo app in quite awhile and wasn't sure it was working because I didn't realize the new post I had added was going to show up at the bottom of the list. It may not be a problem for everyone but the bottom of the list on my laptop was well past the fold.

I appreciate that we do not need to make this **admin backend** fully completed with pagination and column-by-column sorting and what-not. However, I think it would make sense to have newer posts showing up at the top.

Commits
-------

87584a6 Ensure default Post sort order shows newer Posts first
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants