search

matomo-org / matomo

Empowering People Ethically with the leading open source alternative to Google Analytics that gives you full control over your data. Matomo lets you easily collect data from websites & apps and visualise this data and extract insights. Privacy is built-in. Liberating Web Analytics. Star us on Github? +1. And we love Pull Requests!
https://matomo.org/
GNU General Public License v3.0
19.93k stars 2.66k forks source link

Row evolution layer does not open if datatable is filtered by a search #21070

Open peterbo opened 1 year ago

peterbo commented 1 year ago

Context

Expected Behavior

Clicking the row evolution icon should open a layer displaying the evolution of the given row.

Current Behavior

Layer won't open, clicking the row evolution button does not have any effect.

Possible Solution

Steps to Reproduce (for Bugs)

  1. Navigate to pages report,
  2. filter the table for a given search phrase (e.g. "index")
  3. and then click on a row evolution icon Also reproducible on the demo:

mtm-demo

Your Environment

michalkleiner commented 1 year ago

Replicated on demo.matomo.cloud.

michalkleiner commented 1 year ago

I think this might have something to do with the actual word 'index' where the UI component possibly somehow doesn't correctly create the evolution links. Couldn't replicate it with other keyword.

michalkleiner commented 1 year ago

Hi @peterbo, could you please try upgrading to the latest Matomo 4.15.1 and see if the issue persists? We weren't able to replicate it on the latest version.

peterbo commented 1 year ago

Hi @michalkleiner tested in 4.15.1, unfortunately, this problem is still present. Reproducible only in subtable nodes, not for nodes in the root hierarchy.

michalkleiner commented 1 year ago

Ok, thanks for the confirmation. I tried again on demo.matomo.cloud and it's very intermittent, sometimes it works fine, one out of a few searches for the word 'index' then the evolution icon doesn't open the modal. Very hard to debug.

peterbo commented 1 year ago

@michalkleiner it also seems to work after you wait 20-30 seconds. After that, the click triggers the layover correctly! I agree, that this is quite hard to debug. Would be better, if it just didn't work at all. ;)

peterbo commented 3 months ago

I received two reports from two different instances about this problem (filtering dataTable and then trying to open rowEvolution). Is there anything I can help debugging this?