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

Jump to bottom

In Acquisition > Overview report, the Row selector is missing #17965

Closed
mattab opened this issue Sep 5, 2021 · 4 comments · Fixed by #17987
Closed

In Acquisition > Overview report, the Row selector is missing #17965

mattab opened this issue Sep 5, 2021 · 4 comments · Fixed by #17987
Assignees
@geekdenz
Labels
Bug For errors / faults / flaws / inconsistencies etc. Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Milestone
4.5.0

Comments

@mattab
Copy link
Member

mattab commented Sep 5, 2021 •
edited

In the Acquisition > Overview report, the Row selector is missing below the evolution at the top.

Whereas you can still see the row selector under the evolution graph in "Visitors > Overview" (other evolution graphs show it as well).

Expected the same behavior as in "Visitors > Overview", but in Acquisition > Overview it is missing.

it is problematic because often you need to see the evolution over multiple more than the default number of days or weeks or months, so it means users can't get more context/historical data.

Reproduced in Matomo Cloud in https://demo.matomo.cloud/index.php?module=CoreHome&action=index&idSite=1&period=day&date=yesterday#?idSite=1&period=day&date=yesterday&segment=&category=Referrers_Referrers&subcategory=General_Overview and also in 4.5.0-b1

(this is a regression as it used to work)
@mattab mattab added Bug For errors / faults / flaws / inconsistencies etc. Regression Indicates a feature used to work in a certain way but it no longer does even though it should. labels Sep 5, 2021
@tsteur
Copy link
Member

tsteur commented Sep 5, 2021

@mattab it's not clear what you are meaning.

maybe you mean #17499 where it works until you select more than 2 records to plot?

@heurteph-ei
Copy link

@mattab @tsteur , maybe this:
image
image
(note: screenshots taken from version 4.3.1)

@mattab
Copy link
Member Author

mattab commented Sep 7, 2021

@heurteph-ei thanks for posting the screenshot 👍

@tsteur tsteur added this to the 4.5.0 milestone Sep 7, 2021
geekdenz added a commit that referenced this issue Sep 10, 2021
@geekdenz
if not comparing, set show_limit_control to true for Evolution Report
1383b0d 
fixes #17965
@geekdenz geekdenz mentioned this issue Sep 10, 2021
Merged
11 tasks
geekdenz pushed a commit that referenced this issue Sep 10, 2021
set config in better place #17965
610de6e
geekdenz pushed a commit that referenced this issue Sep 10, 2021
@justinvelluppillai
if not comparing, show limit control #17965
33c730b 
Co-authored-by: Justin Velluppillai <justin@innocraft.com>
geekdenz pushed a commit that referenced this issue Sep 10, 2021
update ui-test screenshots #17965
797703a
justinvelluppillai pushed a commit that referenced this issue Sep 15, 2021
if not comparing, set show_limit_control to true for Evolution Report (…
9585236 
…#17987)

* if not comparing, show limit control #17965
@sgiehl sgiehl reopened this Sep 15, 2021
@sgiehl
Copy link
Member

sgiehl commented Sep 15, 2021

#17987 didn't solve that properly. See #17987 (comment)

geekdenz pushed a commit that referenced this issue Sep 17, 2021
remove now redundant code #17965
159fafc
@tsteur tsteur mentioned this issue Sep 17, 2021
Merged
11 tasks
@tsteur tsteur closed this as completed Sep 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@geekdenz geekdenz

Labels
Bug For errors / faults / flaws / inconsistencies etc. Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Projects
None yet
Milestone
4.5.0
Development

Successfully merging a pull request may close this issue.

if not comparing, set show_limit_control to true for Evolution Report matomo-org/matomo
5 participants
@tsteur @mattab @geekdenz @sgiehl @heurteph-ei