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 for device rotation glitches #73

Merged
merged 8 commits into from
Aug 4, 2014

Conversation

Ashton-W
Copy link
Collaborator

@Ashton-W Ashton-W commented Jun 9, 2014

After coming back to this again and again, I finally understand what the problem is: The UIWindow we show above the app must have the same frame as the screen. When the device is rotated, black squares are placed on all sides of the window during the rotation (the black you see in the corners). In our case our window was small and in the middle of the screen, so those black squares were being drawn over our interface :(

📦 ready for merge once all tasks complete:

  • Fix touches sometimes not going through to real interface (may need hit test in more places)
  • Fix Toast view height not adjusting to the match the navigation bar (which changes height on rotation)

@Ashton-W
Copy link
Collaborator Author

@cruffenach this is almost ready for a merge, just having trouble with that last task...

This could be merged now if another issue is raised for the height not adjusting on rotation.

@Ashton-W
Copy link
Collaborator Author

Ashton-W commented Jul 6, 2014

Oh man I just found the problem. CRToast - (UIView*)notificationView isn't a property accessor, its a generator.

@Ashton-W
Copy link
Collaborator Author

Ashton-W commented Jul 6, 2014

Ready for review and merge @cruffenach :)

Ashton-W added a commit that referenced this pull request Aug 4, 2014
Fix for device rotation glitches
@Ashton-W Ashton-W merged commit 23e5456 into cruffenach:master Aug 4, 2014
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.

None yet

1 participant