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

fix: resolve issue with unwanted incremental scaling in landscape #990

Merged
merged 1 commit into from
Jul 20, 2023

Conversation

eglitise
Copy link
Collaborator

This fixes a problem I've mainly noticed when working with an app in landscape orientation.
When opening such an app with the default inspector app window dimensions, the screenshot width is around 450 pixels. However, if I refresh the source, the screenshot width slightly increases, and continues increasing until it reaches the max limit of 500 pixels:

old-behavior.mp4

Now, the default screenshot width is a bit smaller at 430 pixels, but it no longer changes upon source refresh:

new-behavior.mp4

Of course, the screenshot size can still be adjusted by changing the app window size.

@eglitise eglitise added the fix This resolves a user-facing problem label Jul 19, 2023
Copy link
Member

@jlipps jlipps left a comment

Choose a reason for hiding this comment

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

seems good to me. this probably resulted from me trying to fix some other css problem without adequate knowledge

@eglitise eglitise merged commit 41136de into appium:main Jul 20, 2023
4 checks passed
@eglitise eglitise deleted the fix-incremental-landscape-scaling branch July 20, 2023 05:50
shiva-guntoju pushed a commit to shiva-guntoju/appium-inspector that referenced this pull request Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix This resolves a user-facing problem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants