Created attachment 8763096 [details] IMG_0090.PNG Build: 5.0b11 Device: Iphone 6 S (ios 10) Steps to reproduce: 1. Launch Firefox and change device orientation to landscape 2. Open the menu Actual: - The menu is not fully displayed.
Saw this on my iPad Pro yesterday (iOS 10 Beta 4)
Tested on: Build: 5.1b3 Device: iPad mini 4 iOS: 10b6 The issue is reproducible.
Possibly related to intrinsticContentSize being zero.
Created attachment 8785962 [details] [review] Link to Github pull-request: https://github.com/mozilla/firefox-ios/pull/2070 Passing the review over to you but if you don't have an iOS 10 device to test on I can assign to someone else!
Comment on attachment 8785962 [details] [review] Link to Github pull-request: https://github.com/mozilla/firefox-ios/pull/2070 I think this fix is doing too much and causing the MenuView layout code to be called too many times when the menu is displayed. I have left a suggested alternative fix in the PR. :farhan - What should I do with this given :sleroux is on PTO?
Steph said he'll be on and off. I think he should be able to land this.
Thanks for the alternative :fluffyemily! I've used that instead and gave it a test and works the same without the performance hit. I'll land that instead.
Tested on iPad mini 4 running iOS 10b8 on 5.2b2. The issue is not reproducible following the above STR.