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
Configurable "Rows to display" for evolutionLimits #5206
Comments
Thanks for the feature suggestion. |
I would also appreciate more flexibility or more reasonable numbers. Once again I stumbled over the uneven ratio for a monthly display: And I do not know, why anybody would want to see only three or six months, since this is included Fritz |
Any news on this topic? |
@fritzgithub we can maybe add some new numbers to the list. The current list is at https://github.com/matomo-org/matomo/blob/3.x-dev/plugins/CoreHome/javascripts/dataTable.js#L655-L661 What would you suggest as the new list? |
Hi, thank you for listening.
|
Any chance we get this by 2022? :) We ask only for a simple array expansion, without resolving to yet another plugin hack or something, that could be easily set in admin dashboard to our need/liking. For example, I would like to extend days selector to 730 and 1095 values (why stop at unnatural "500"?). Also, if we use last30 param to view e.g. pages reports, date selector dropdown is removed from view - why? Why not simply revert to default days selector in that case? |
@dev-101 feel free to create a pull request and we will be happy to review. It should be indeed an easy change and the hardest part be probably coming up with the numbers. For numbers > 500: This would maybe need a test re performance and how long it takes to fetch so many data points. Be indeed great to have more natural numbers in the selection. |
Hi, I think PR for this is trivial and not a point. Creating a PR for this means nothing if it won't be accepted by project owners. Given the age of this issue, it is unlikely it will happen :) |
@dev-101 I'm not seeing why we wouldn't accept the PR. The change sounds useful to me 👍 . The only thing we'd need to look at are whether values over 500 are still loading fast enough. |
Same as #5188 but for evolutionLimits.
Use case: Showing 1 month, 7 days (or 1 week) all seems like interesting values to have when using "Evolution over period" graphs.
Actual numbers taken from /plugins/CoreHome/javascripts/dataTable.js :
Dali
The text was updated successfully, but these errors were encountered: