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

[CSS] add scroll to live visits table #19273

Merged
merged 14 commits into from Jun 9, 2022
Merged

[CSS] add scroll to live visits table #19273

merged 14 commits into from Jun 9, 2022

Conversation

peterhashair
Copy link
Contributor

Description:

Fixes: #19254
add scroll to live table

Review

add scroll to live table
@peterhashair peterhashair added this to the 4.11.0 milestone May 30, 2022
@peterhashair peterhashair added not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. Needs Review PRs that need a code review labels May 30, 2022
Copy link
Contributor

@bx80 bx80 left a 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.

@bx80 bx80 removed the Needs Review PRs that need a code review label May 30, 2022
Peter and others added 8 commits May 31, 2022 12:43
Peter added 2 commits June 2, 2022 12:08
update scroll bar
update screenshot
@peterhashair peterhashair added the Needs Review PRs that need a code review label Jun 6, 2022
@bx80
Copy link
Contributor

bx80 commented Jun 7, 2022

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.

@bx80
Copy link
Contributor

bx80 commented Jun 7, 2022

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.

update ui
@justinvelluppillai justinvelluppillai modified the milestones: 4.11.0, 4.12.0 Jun 7, 2022
@peterhashair peterhashair merged commit 9697bbb into 4.x-dev Jun 9, 2022
@peterhashair peterhashair deleted the m19254 branch June 9, 2022 00:58
peterhashair pushed a commit that referenced this pull request Jun 16, 2022
* 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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review PRs that need a code review not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix adjacent elements to dashboard-dropdown-menu-modal overlapping on narrower screens
4 participants