Add dropdown box to allow selecting more results per page

[daubaris]

#1625

[sgrif]

Rather than adding a dropdown for this, could we just up the default sent by the frontend?

[daubaris]

You mean to just increase the per_page value to 100 to be the default value instead of 10?

crates.io/app/controllers/crates.js

Line 11 in 922a7c9

per_page: 10,

[sgrif]

I'd go with either 20 or 50 instead of 100, but yes

[daubaris]

Wouldn't be better to leave the dropdown just to provide flexibility for the user or will this cause any performance issues? Just curious.

[sgrif]

I'm not sure that a dropdown to configure this really adds that much value, but I'm open to being convinced otherwise. Does anyone ever use these to lower the number of records displayed?

…

On Fri, Jun 14, 2019 at 1:28 PM daubaris ***@***.***> wrote: Wouldn't be better to leave the dropdown just to provide flexibility for the user or will this cause any performance issues? Just curious. — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

-- Thanks, Sean Griffin

[kzys]

FYI, https://metacpan.org/search?size=20&q=foo is having a way to change the page size. https://rubygems.org/search?utf8=✓&query=foo, https://www.npmjs.com/search?q=foo and https://pypi.org/search/?q=foo don't provide the one.

Personally I'd like to increase the default page count to keep it simple.

[locks]

Increasing the default page seems like a nice small improvement!

[sgrif]

@daubaris Are you interested in amending this to increase the default?

[bors]

☔ The latest upstream changes (presumably #1817) made this pull request unmergeable. Please resolve the merge conflicts.

[carols10cents]

I'm sorry we've let this PR sit for so long! I'm going to close it since it's gotten out of date and since there's #1883 that implemented the discussed modifications. Thank you!