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

Regression: "be.visible" is fragile when used with fixed position or disabled pointer-events #27782

Open
mryellow opened this issue Sep 11, 2023 · 7 comments
Labels
existing workaround prevent-stale mark an issue so it is ignored by stale[bot]

Comments

@mryellow
Copy link

Current behavior

I'm experiencing #6675 and #9227 in 13.1.0.

Have been forced to work around by settings pointer-events: initial on the element in question.

Desired behavior

be.visible should detect visible elements regardless of position or pointer-events.

Test code to reproduce

.loader-spinner {
    left: 0;
    pointer-events: none;
    position: fixed;
    top: 0;
    z-index: 9999;
}

cy.get('.loader-spinner').should('be.visible');

Cypress Version

13.1.0

Node version

v16.19.1

Operating System

Linux debian 4.19.0-25-amd64 #1 SMP Debian 4.19.289-2 (2023-08-08) x86_64 GNU/Linux

Debug Logs

No response

Other

No response

@mryellow
Copy link
Author

mryellow commented Sep 11, 2023

Overloading CSS with either pointer-events: initial !important; or position: initial !important; has be.visible function as intended. z-index does not seem to be involved.

@cypress-app-bot
Copy link
Collaborator

This issue has not had any activity in 180 days. Cypress evolves quickly and the reported behavior should be tested on the latest version of Cypress to verify the behavior is still occurring. It will be closed in 14 days if no updates are provided.

@cypress-app-bot cypress-app-bot added the stale no activity on this issue for a long period label Mar 10, 2024
@mryellow
Copy link
Author

What bot would know the difference between a valid and invalid issue?

@cypress-app-bot cypress-app-bot removed the stale no activity on this issue for a long period label Mar 11, 2024
@cypress-app-bot
Copy link
Collaborator

This issue has not had any activity in 180 days. Cypress evolves quickly and the reported behavior should be tested on the latest version of Cypress to verify the behavior is still occurring. It will be closed in 14 days if no updates are provided.

@cypress-app-bot cypress-app-bot added the stale no activity on this issue for a long period label Sep 7, 2024
@mryellow
Copy link
Author

mryellow commented Sep 9, 2024

Bump. Hate stalebots. So much. So very much.

@cypress-app-bot cypress-app-bot removed the stale no activity on this issue for a long period label Sep 10, 2024
@jennifer-shehane jennifer-shehane added the prevent-stale mark an issue so it is ignored by stale[bot] label Sep 16, 2024
@jennifer-shehane
Copy link
Member

@mryellow Added a label to prevent the stale bot from commenting.

@mryellow
Copy link
Author

Thankyou. Can you add that label automatically to all issues? ;-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
existing workaround prevent-stale mark an issue so it is ignored by stale[bot]
Projects
None yet
Development

No branches or pull requests

4 participants