Skip to content

fix: account for the View's scroll value in the previewBox calculation#11

Open
Mufanc wants to merge 1 commit into
CoXier:masterfrom
Mufanc:master
Open

fix: account for the View's scroll value in the previewBox calculation#11
Mufanc wants to merge 1 commit into
CoXier:masterfrom
Mufanc:master

Conversation

@Mufanc

@Mufanc Mufanc commented Apr 21, 2025

Copy link
Copy Markdown
Contributor

Previously the previewBox calculation did not account for the View's scroll value which would lead to incorrect coordinate calculations in scenarios like ViewPager.

@Mufanc

Mufanc commented Apr 21, 2025

Copy link
Copy Markdown
Contributor Author

There also seem to be some issues with the rendering I need to make further adjustments.

@Mufanc Mufanc closed this Apr 21, 2025
@Mufanc

Mufanc commented Apr 21, 2025

Copy link
Copy Markdown
Contributor Author

It appears the issue wasn't introduced by my changes. reverting to the old code also encounters incorrect device screenshots in certain cases. The cause remains unidentified. I will investigate further and open a new pull request once the cause is determined.

@Mufanc Mufanc reopened this Apr 21, 2025
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.

1 participant