This PR is based off of #18955.
Changes:
UI test failure appears to be unrelated.
@peterhashair I think this is a random race condition that happens in Materialize. It came up before but I couldn't find a fix for it. Can you check if it happens every time? Its all working for me locally.
@diosmosis it happens on mine every time, but people claim there is a bug on materialize and VUE. here is the solution for that, hopefully, that helps https://stackoverflow.com/questions/33704122/vuejs-materializecss-select-field/43728599#43728599
@peterhashair that link doesn't mention the tabIndex error, which has been present since before the vue migration. Can you tell me what browser you were using and any other information to help reproduce?
@diosmosis I am using chrome. Nothing special, just pull the code. Click on the video steps. Maybe this will help
https://localcoder.org/materialize-cannot-set-property-tabindex-of-null-at-dropdown-makedropdownfoc
[Development]
enabled = 0
https://user-images.githubusercontent.com/3783072/159612904-d58d7d40-eb64-493e-a52d-eaab9a16be3c.mp4
@sgiehl would you have time to try and reproduce the error @peterhashair is seeing? I'm still not able to reproduce it.
EDIT: ignore this was able to reproduce.
@peterhashair can you try doing a ./console cache:clear
and hard reload? That looked like it sent you to the main dashboard, instead of back to the marketplace.
EDIT: Wait, maybe that's the cause, I was trying from the admin page. Yes, I can reproduce it now. Thanks for uploading a video :+1:
@diosmosis this line causes the problem this.dropdownEL
is null. Haven't look too deep.
](https://github.com/matomo-org/matomo/blob/1a3d0de2d7a7b287de0acc04df7c586fe9f6a53d/node_modules/materialize-css/dist/js/materialize.js#L2571)
@peterhashair should be fixed in latest commit
@peterhashair can you take another look?
@diosmosis when you do performance search a plugin, it redirects to the dashboard page. See screen record
https://user-images.githubusercontent.com/3783072/159815616-23de5809-1ebe-4b59-9447-00b717da0d1f.mp4
@peterhashair pushed a fix, hopefully the last one. Had to modify the Marketplace controller action to allow 'query' to be obtained from $_GET as well as $_POST, so the reporting page marketplace won't redirect to the admin page (cc @sgiehl).
@diosmosis there is another one sorry. When you click More
button. It doesn't do the popup. see screenshot
https://user-images.githubusercontent.com/3783072/159834837-e6d3bfb6-7617-4b3e-be62-710f76ae2353.mp4
@peterhashair should be fixed in the latest push. Thanks for the thorough testing! :+1: