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

temporary scroll behavior implicitly overrides user preference scrolling #26113

Closed
AtofStryker opened this issue Mar 14, 2023 · 1 comment · Fixed by #26102
Closed

temporary scroll behavior implicitly overrides user preference scrolling #26113

AtofStryker opened this issue Mar 14, 2023 · 1 comment · Fixed by #26102
Assignees
Labels
pkg/reporter This is due to an issue in the packages/reporter directory Reproducible Can be reproduced topic: scrolling ↕️ type: bug

Comments

@AtofStryker
Copy link
Contributor

Current behavior

Currently, The cypress app has auto scroll turned on for user preferences by default. However, when a user scrolls mid test, and either clicks on the user preference menu, or clicks on a command in the command log, the auto scroll user preference is implicitly overridden and turned off

autoscroll-behavior-before

Desired behavior

User preferences should never be overridden except explicitly by the user. If a user scrolls while the test is running, the auto scroll may be temporarily disabled, but should not override the preference itself. When the user reruns the test, the user configured preference that is set should persist and override any temporary behavior.

autoscroll-new

Test code to reproduce

N/A, This is at the reporter level

Cypress Version

10.0.0

Node version

v16.16.0

Operating System

macOS 12.6

Debug Logs

No response

Other

No response

@AtofStryker AtofStryker self-assigned this Mar 14, 2023
@AtofStryker AtofStryker added type: bug pkg/reporter This is due to an issue in the packages/reporter directory Reproducible Can be reproduced topic: scrolling ↕️ routed-to-e2e labels Mar 14, 2023
@AtofStryker AtofStryker linked a pull request Mar 14, 2023 that will close this issue
3 tasks
AtofStryker added a commit that referenced this issue Mar 15, 2023
* fix: Do not magically mutate the user preferences when auto scroll behavior is disabled when a user scrolls or clicks inside the reporter or opens user preferences. Users must explicitly toggle auto-scroll on/off inside the user preferences menu and any implicit behavior will be overidden when this configuration value is set.

* [run ci]

* chore: add CHANGELOG entry

* Update cli/CHANGELOG.md

Co-authored-by: Emily Rohrbough <emilyrohrbough@users.noreply.github.com>

* chore: move bugfix to 12.8.1 in changelog

* chore: remove issue 25084 from fix

* chore: rename autoScrollingUnderUserPreference to autoScrollingUserPref

* empty

* Update packages/reporter/src/lib/events.ts

Co-authored-by: Mark Noonan <mark@cypress.io>

* test: add cy-in-cy regression test to make sure auto scroll is not implicitly turned off

* chore: update changelog entry to include #26113

* Update system-tests/projects/cypress-in-cypress/cypress/e2e/dom-content-scrollable-commands.spec.js

Co-authored-by: Emily Rohrbough <emilyrohrbough@users.noreply.github.com>

* chore: update the specs list app test since we have added a new cy-in-cy test

* chore: update the specChange-subscription tests

* Update CHANGELOG.md

* chore: update missed assertion update in spec change subscription

---------

Co-authored-by: Emily Rohrbough <emilyrohrbough@users.noreply.github.com>
Co-authored-by: Mark Noonan <mark@cypress.io>
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Mar 15, 2023

Released in 12.8.1.

This comment thread has been locked. If you are still experiencing this issue after upgrading to
Cypress v12.8.1, please open a new issue.

@cypress-bot cypress-bot bot locked as resolved and limited conversation to collaborators Mar 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pkg/reporter This is due to an issue in the packages/reporter directory Reproducible Can be reproduced topic: scrolling ↕️ type: bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant