Skip to content

Add fix for boundsInWindow incorrectly calculating position on screen#1983

Merged
geoff-powell merged 3 commits intomasterfrom
gpowell/compose-accessibility-fixes
Jul 2, 2025
Merged

Add fix for boundsInWindow incorrectly calculating position on screen#1983
geoff-powell merged 3 commits intomasterfrom
gpowell/compose-accessibility-fixes

Conversation

@geoff-powell
Copy link
Copy Markdown
Contributor

@geoff-powell geoff-powell commented Jun 4, 2025

SemanticsNode.boundsInWindow was incorrectly reporting bounds with compose 1.8.x update.

This uses a combination of boundsInRoot and locationOnScreen to correctly calculate displayBounds.

Example of bug:
472a194c6c73002fce0a4bd046522876087c52a1

@geoff-powell geoff-powell force-pushed the gpowell/compose-accessibility-fixes branch 5 times, most recently from 82464b4 to 81b69fe Compare June 5, 2025 17:31
Copy link
Copy Markdown
Collaborator

@colinmarsch colinmarsch left a comment

Choose a reason for hiding this comment

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

We didn't have any tests in the repo that failed due to this issue?

@jrodbx jrodbx modified the milestones: 2.0.0-alpha03, 2.0.0-alpha04 Jun 20, 2025
@geoff-powell geoff-powell force-pushed the gpowell/compose-accessibility-fixes branch from 81b69fe to bcafc5d Compare July 1, 2025 18:27
@geoff-powell geoff-powell force-pushed the gpowell/compose-accessibility-fixes branch from bcafc5d to 918a05e Compare July 2, 2025 10:47
@geoff-powell geoff-powell merged commit 29cfca9 into master Jul 2, 2025
29 of 30 checks passed
@geoff-powell geoff-powell deleted the gpowell/compose-accessibility-fixes branch July 2, 2025 14:18
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.

3 participants