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

Chrome 72 - Scrollbar for Test Runner aligns with AUT instead of Command Log #3253

Closed
jennifer-shehane opened this issue Jan 30, 2019 · 2 comments · Fixed by #3255
Closed

Comments

@jennifer-shehane
Copy link
Member

jennifer-shehane commented Jan 30, 2019

Current behavior:

From @fr0 after using workaround for Chrome 72 proxy issue: #3252 (comment)

if the test log gets too long, the scrollbar appears for the entire page instead of just the test log. This has the weird side effect of scrolling my app off the screen, and then when cypress tries to scroll an element in view, it seems like it's fighting with myself.

With Electron / Chrome 70
electron

With Chrome 72/73 + workaround:

chrome

Desired behavior:

Scrollbar should align with Command Lo

Steps to reproduce: (app code and test code)

  1. Write tests with over 100+ commands
  2. Run tests in Chrome 72+
  3. Add workaround code to plugins.

Versions

3.1.4+

@jennifer-shehane
Copy link
Member Author

jennifer-shehane commented Jan 30, 2019

The scrollbar goes back to the way it was if I remove flex-direction: column from the .reporter-wrap element - changing this also does not change the behavior or scroll in Chrome 70, Electron 59, or Canary 73.

Going to propose this fix and continue to investigate what changed in Chrome 72.

@cypress-bot
Copy link
Contributor

cypress-bot bot commented Jan 30, 2019

Released in 3.1.5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants