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
[CSS] add scroll to live visits table #19273
Conversation
add scroll to live table
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This solves the issue for my tests 👍
Since this issue is a regression it would be good to add a simple UI test screenshot of the dashboard with the window set to a size that causes the visits table header to overflow.
update tests
update tests
show full image
update tests
add timeout and screenshots
update scroll bar
update screenshot
I've bumped the UI tests to run again, just want to be sure that the Movers and Shakers widget overflow is a random test failure and not something else. |
The small screen dashboard UI test movers and shakers overflow is still failing each time on Travis but it passes when run locally, I've increase the delay from 250ms to 500ms to see if that fixes it. |
* add scroll to live table add scroll to live table * add tests add tests * update tests update tests * update tests update tests * Update Dashboard_spec.js show full image * update tests update tests * add timeout and screenshots add timeout and screenshots * update scroll bar update scroll bar * update screenshot update screenshot * Rerun UI tests * Increase UI screenshot delay to fix random failure * update ui update ui Co-authored-by: sgiehl <stefan@matomo.org> Co-authored-by: Ben <ben.burgess@innocraft.com>
Description:
Fixes: #19254
add scroll to live table
Review