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

changes of visibility in documentation #5867

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

senpl
Copy link

@senpl senpl commented Jul 9, 2024

Changes in cypress-io/cypress#29741 now updated in documentation.
This is still early so it should be done after this pull request date and version will be finally known.

Copy link

netlify bot commented Jul 9, 2024

👷 Deploy request for cypress-docs pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit d22ad34

@CLAassistant
Copy link

CLAassistant commented Jul 9, 2024

CLA assistant check
All committers have signed the CLA.

@cypress-app-bot
Copy link
Collaborator

Comment on lines 86 to 98
### Visibility

:::tip

Most of the time you should check if element is not hidden. Some elements might be visible but not clickable or interactable.

Cypress check visibility like browsers [checkVisibility](https://developer.mozilla.org/en-US/docs/Web/API/Element/checkVisibility).
There are some exceptions from browser checks:
- in `option` inside `select` visibility is determined by visibility of parent `select` element
- if element is offscreen it is consided invisible
- if parent cover element it is consided invisible
In old browsers that not support [checkVisibility] check is hidden is performed.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are some grammatical and spelling errors in this section.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't comment in more detail because this PR is still in draft state.

senpl and others added 4 commits July 12, 2024 09:25
Co-authored-by: Mike McCready <66998419+MikeMcC399@users.noreply.github.com>
Co-authored-by: Mike McCready <66998419+MikeMcC399@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants