Closed Bug 1314044 Opened 8 years ago Closed 8 years ago

Top Tabs for iPad toolbar is scrolled incorrectly after specific steps

Categories

(Firefox for iOS :: General, defect, P1)

All
iOS
defect

Tracking

()

VERIFIED FIXED
Iteration:
1.13
Tracking Status
fxios 7.0+ ---
fxios-v6.0 --- unaffected

People

(Reporter: SimonB, Assigned: farhan)

References

Details

(Whiteboard: [mobileCore])

Attachments

(1 file)

55 bytes, text/x-github-pull-request
jhugman
: feedback+
Details | Review
Build: 1bc8bb71f Devices: - iPad Pro (9.2.1) - iPad Air 2 (10.0.2) Steps to reproduce: 1. Open 12 tabs in Fennec (select the last tab) 2. Background by pressing the home button 3. Restore Fennec Actual results: - The scroll position is incorrect. The Top Tabs bar will be scrolled to the middle and the last tab which is selected will not be visible. Expected result: - Top Tabs toolbar should be scrolled to the selected tab. Note: - This issue is also reproducible in the following scenarios: * When locking the device and restoring Fennec * When switching to another app and after restoring Fennec * When backgrounding Fennec using split screen feature and restoring * When opening a new tab via Today Widget (No scroll to the selected tab) * When opening tapping on 'Help' from the 'Settings' menu (Top tabs will not scroll to the Help tab) * When changing device orientation to landscape See the video for more information: https://www.youtube.com/watch?v=5P53WIbrhvE
Blocks: 1305800
No longer blocks: toptabs
Assignee: nobody → fpatel
Iteration: --- → 1.12
Priority: P3 → P1
Whiteboard: [mobileCore]
Iteration: 1.12 → 1.13
Attached file Pull Request
This is a pretty big change. I'd love some feedback :)
Attachment #8828083 - Flags: review?(sleroux)
Attachment #8828083 - Flags: feedback?(jhugman)
Attachment #8828083 - Flags: feedback?(etoop)
Attachment #8828083 - Flags: feedback?(jhugman) → feedback+
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Attachment #8828083 - Flags: feedback?(etoop)
Attachment #8828083 - Flags: review?(sleroux)
Verifying as fix on latest master 51abcc7ca.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: