Skip to content
This repository has been archived by the owner on Dec 26, 2019. It is now read-only.

Screen rotation is reset if an alert is visible #300

Closed
mykola-mokhnach opened this issue Sep 27, 2016 · 15 comments
Closed

Screen rotation is reset if an alert is visible #300

mykola-mokhnach opened this issue Sep 27, 2016 · 15 comments
Labels

Comments

@mykola-mokhnach
Copy link
Contributor

mykola-mokhnach commented Sep 27, 2016

Steps to reproduce:

  1. Start a test in landscape mode (use setRotation API call to set Landscape mode on iPad)
  2. Trigger an alert in your app

Actual result:
All the application metrics are forcefully changed to portrait mode, thus the UI and getRotation API say we are still in landscape:
1
Everything returns back to normal after the alert is accepted or dismissed:
2
Although the standard alert.accept API call has no effect, since it internally assumes wrong alert coordinates (in portrait mode it works as expected)

Expected result:

  • The device orientation should not be reset while alert is visible
  • The alert/accept API call should work properly after the described steps
@mykola-mokhnach mykola-mokhnach changed the title Screen rotation is reset if alert is visible Screen rotation is reset if an alert is visible Sep 27, 2016
@marekcirkos
Copy link
Contributor

Looks like XCTest issue.

@marekcirkos
Copy link
Contributor

@mykola-mokhnach can you raise this issue to Apple?

@mykola-mokhnach
Copy link
Contributor Author

@marekcirkos With pleasure. Can you please provide a link to their bugtracker, where I can place this issue?

@marekcirkos
Copy link
Contributor

@mykola-mokhnach
Copy link
Contributor Author

Reported this to Apple. Shell we keep this issue opened?

@marekcirkos
Copy link
Contributor

I think we should keep it for tracking and to let people know it is "Know issue"

@marekcirkos
Copy link
Contributor

@mykola-mokhnach Is this still an issue?

@mykola-mokhnach
Copy link
Contributor Author

I will check it again and comment the issue. In theory, this workaround we did for landscape, should work for it.

@mykola-mokhnach
Copy link
Contributor Author

Yup, the workaround works perfectly now with the recent WDA commits. The issue can be closed.

@mykola-mokhnach
Copy link
Contributor Author

Seems like the problem is relevant only for security alerts generated by iOS itself.
#470 might handle it.

@drapostolos
Copy link

is there a way to follow this issue in the apple bugtracker?

@marekcirkos
Copy link
Contributor

You should open your own, to give Apple stronger signal.

@alexander-poulikakos
Copy link

@marekcirkos what is the reason for closing this? Is there a solution from Apple available?

@marekcirkos
Copy link
Contributor

This should be fix. If it is not please reopen

@alexander-poulikakos
Copy link

alexander-poulikakos commented May 22, 2017

@marekcirkos I'm using the below configuration and I still have this problem. Is there anything I need to update to get this fix?

  • MacOS Sierra version 10.12.5 (16F73)
  • Appium v1.6.5-beta
  • XCUITestDriver v2.29.0
  • Xcode version 8.3.2 (tools v8.3.2.0.1.1492020469)
  • iOS SDK Version 10.3

Device:

  • iphone 6 (10.3.1)

PS. I don't think i have the rights to re-open this issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants