Skip to content

fix: use wdFrame instead of raw frame for scroll gesture anchor - #1256

Merged
mykola-mokhnach merged 1 commit into
masterfrom
scroll
Sep 11, 2026
Merged

fix: use wdFrame instead of raw frame for scroll gesture anchor#1256
mykola-mokhnach merged 1 commit into
masterfrom
scroll

Conversation

@mykola-mokhnach

Copy link
Copy Markdown

anchorElement.frame can be pre-scaled for compatibility-mode window mismatches or dimension-swapped in landscape, neither of which scrollingFrame (snapshot-derived) reflects - causing scroll gestures to land outside the scroll view. wdFrame matches scrollingFrame's coordinate space instead. Follow-up to #1249 per review discussion on that PR (appium/appium#16185).

anchorElement.frame can be pre-scaled for compatibility-mode window
mismatches or dimension-swapped in landscape, neither of which
scrollingFrame (snapshot-derived) reflects - causing scroll gestures
to land outside the scroll view. wdFrame matches scrollingFrame's
coordinate space instead. Follow-up to #1249 per review discussion
on that PR (appium/appium#16185).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01U6fXkU2kt87W6cmWPyoBgF
@mykola-mokhnach

Copy link
Copy Markdown
Author

thanks for review @KazuCocoa
have you had a chance to check the fix on a real device?

@KazuCocoa

Copy link
Copy Markdown
Member

yup, looks good to me @mykola-mokhnach

@mykola-mokhnach
mykola-mokhnach merged commit 9ada9fd into master Sep 11, 2026
56 of 60 checks passed
@mykola-mokhnach
mykola-mokhnach deleted the scroll branch September 11, 2026 12:04
github-actions Bot pushed a commit that referenced this pull request Sep 11, 2026
## [16.12.8](v16.12.7...v16.12.8) (2026-09-11)

### Bug Fixes

* use wdFrame instead of raw frame for scroll gesture anchor ([#1256](#1256)) ([9ada9fd](9ada9fd))
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 16.12.8 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants