Skip to content

fix(isVisibleOnScreen): account for position: absolute elements inside overflow container#4405

Merged
straker merged 2 commits intodevelopfrom
overflow-position-absolute
Apr 15, 2024
Merged

fix(isVisibleOnScreen): account for position: absolute elements inside overflow container#4405
straker merged 2 commits intodevelopfrom
overflow-position-absolute

Conversation

@straker
Copy link
Copy Markdown
Contributor

@straker straker commented Apr 10, 2024

Also tested various ways to try to get the position: absolute to be hidden by the node. Turns out there are a few cases where it will be hidden:

  • overflow node uses position itself other than static
  • node in-between the overflow node and the positioned child uses position relative or sticky

and cases where it won't be hidden

  • positioned child uses a position of fixed (it won't be hidden by any ancestor overflow, even if the ancestor uses position itself)

Closes: #4016

@straker straker requested a review from a team as a code owner April 10, 2024 17:46
@straker straker merged commit 2940f6e into develop Apr 15, 2024
@straker straker deleted the overflow-position-absolute branch April 15, 2024 14:24
WilcoFiers added a commit that referenced this pull request May 6, 2024
###
[4.9.1](v4.9.0...v4.9.1)
(2024-05-06)

### Bug Fixes

- Prevent errors when loading axe in a page with prototype.js
- **aria-allowed-attr:** allow meter role allowed aria-\* attributes on
meter element
([#4435](#4435))
([7ac6392](7ac6392))
- **aria-allowed-role:** add gridcell, separator, slider and treeitem to
allowed roles of button element
([#4398](#4398))
([4788bf8](4788bf8))
- **aria-roles:** correct abstract roles (types) for
aria-roles([#4421](#4421))
- **aria-valid-attr-value:** aria-controls & aria-haspopup incomplete
([#4418](#4418))
- fix building axe-core translation files with region locales
([#4396](#4396))
([5c318f3](5c318f3)),
closes [#4388](#4388)
- **invalidrole:** allow upper and mixed case role names
([#4358](#4358))
([105016c](105016c)),
closes [#2695](#2695)
- **isVisibleOnScreen:** account for position: absolute elements inside
overflow container
([#4405](#4405))
([2940f6e](2940f6e)),
closes [#4016](#4016)
- **label-content-name-mismatch:** better dismiss and wysiwyg symbolic
text characters
([#4402](#4402))
- **region:** Decorative images ignored by region rule
([#4412](#4412))
- **target-size:** ignore descendant elements in shadow dom
([#4410](#4410))
([6091367](6091367))
- **target-size:** pass for element that has nearby elements that are
obscured ([#4422](#4422))
([3a90bb7](3a90bb7)),
closes [#4387](#4387)


This PR was opened by a robot 🤖 🎉 (And updated by @WilcoFiers
)
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.

Certain visible styling being calculated to not be visible (label-title-only, potentially other rules)

2 participants