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

New option 'count' to display arbitrary number of results, rather than always 10 #16

Open
FosseWay opened this issue Jun 5, 2019 · 0 comments

Comments

@FosseWay
Copy link

FosseWay commented Jun 5, 2019

Rather than always displaying the top 10 most-visited pages, it may be useful for some editors/admins to specify a custom limit (e.g. Top 5 or Top 20) on a particular instance of {{top}}, which could be specified with a new option count= (e.g. {{top|count=5}} or {{top|count=20}}).

There should be sensible limits: not fewer than 1 result, and not more than 20 (which is already a hard limit in the code as-is).

Basic valid-value enforcement of the value of count would be useful:

Count value is... Value enforced
Non-numeric 10
<1 1
>20 20
@FosseWay FosseWay changed the title Option to display arbitrary number of results, rather than always 10 New option 'count' to display arbitrary number of results, rather than always 10 Jun 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant