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

Date comparison selector should stay when changing site #20321

Closed
heurteph-ei opened this issue Feb 6, 2023 · 4 comments · Fixed by #21242
Closed

Date comparison selector should stay when changing site #20321

heurteph-ei opened this issue Feb 6, 2023 · 4 comments · Fixed by #21242
Labels
Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Milestone

Comments

@heurteph-ei
Copy link

heurteph-ei commented Feb 6, 2023

Expected Behavior

When I select a date comparison other than Previous period, then change the selected site, I would like the comparison selector stays at the one selected (and displayed).

Current Behavior / Steps to Reproduce (for Bugs)

  1. Go to your favorite Matomo instance: https://demo.matomo.cloud/index.php
  2. Select a date comparison other than Previous period (eg. another day in the same month)
  3. Change the site selector (from Demo site to Example site for example)
  4. OK, the comparison is still taken into account... But when the comparison date needs to be changed, the date comparison selector has been reset to Previous period

Possible Solution

Minor display bug. Just need to select the date comparison type again...

Your Environment

Visible in demo site, but was already buggy in 4.3.1

@heurteph-ei heurteph-ei added Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. To Triage An issue awaiting triage by a Matomo core team member labels Feb 6, 2023
@sgiehl
Copy link
Member

sgiehl commented Feb 6, 2023

Thanks @heurteph-ei for reporting this one.
I'm not sure if that is something that would be expected by everyone. We could though implement that for date comparison only, which would make it a bit inconsistent maybe, as a segment comparison couldn't be kept (as they might not exist).
I'll let the product team decide to consider this as a future improvement.

@sgiehl sgiehl added Enhancement For new feature suggestions that enhance Matomo's capabilities or add a new report, new API etc. and removed Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. To Triage An issue awaiting triage by a Matomo core team member labels Feb 6, 2023
@sgiehl sgiehl added this to the For Prioritization milestone Feb 6, 2023
@heurteph-ei
Copy link
Author

Hi @sgiehl
This is not a new feature, it is just an UI bug.
When changing displayed site, the date comparison is kept for the reports calculations, but the date selector is reset to "Compare to previous period".

@sgiehl sgiehl added Regression Indicates a feature used to work in a certain way but it no longer does even though it should. and removed Enhancement For new feature suggestions that enhance Matomo's capabilities or add a new report, new API etc. labels Apr 4, 2023
@sgiehl
Copy link
Member

sgiehl commented Apr 4, 2023

Ah ok, guess now I understood. So the selector isn't initialized with the correct values after loading a new site.

@michalkleiner
Copy link
Contributor

The issue is not related to switching websites, a simple reload presents the issue as well.
The date range selector doesn't correctly pick up the query parameters and always displays 'Previous period'.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants