1334460 - 'Disconnect from sync' prompt will jump out of view after selecting 'Reason for disconnecting' option

Status: VERIFIED FIXED

(Firefox for iOS :: Firefox Accounts, defect)

Description

[Simon Basca [:SimonB]]

Build: 51abcc7c
Device: iPad Pro
iOS: 9.2.1

Prerequisites:
- Logged in to FxA 
- Sync account on at least 2 devices

Steps to reproduce:
1. Launch Firefox
2. Go to Settings 
3. Go to Firefox Accounts page by tapping on the email address
4. Tap 'Show' button next to Device & apps field 
5. Tap 'Disconnect' on any other synced device except the 'current device'
6. Tap 'Reason for disconnecting...'
7. Select any option

Actual result:
- The 'Disconnect from Sync?' window will jump out of view on iPad devices and will be correctly displayed only when scrolling on the page. On iPhone the window will briefly jump out of view and will return to its correct position.

Expected results:
- The 'Disconnect from Sync?' windows position should remain fix.

See video of the issue: 
https://www.youtube.com/watch?v=89_8wgW5fjA

Comment 1

[Stephan Leroux [:sleroux]]

See if https://bugzilla.mozilla.org/show_bug.cgi?id=1331897 fixes this issue.

Comment 2

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

Shane, can you take a peek at the video? Curious what you think about this one.

Comment 3

[Shane Tomlinson [:stomlinson]]

Attachment: Remove account on an iPad, using radio buttons.

We changed the dropdown to be a list of radio buttons, this should no longer be an issue. I don't have an iPad to hand, but tested in on an iPad Retina simulator and it looked good.

Comment 4

[Shane Tomlinson [:stomlinson]]

:SimonB - can you re-test against the current FxA prod and see if this issue is fixed for you? If so, mind closing as RESOLVED FIXED?

Ref https://github.com/mozilla/fxa-content-server/pull/4600 where the dropdown was changed to radio buttons.

Comment 5

[Simon Basca [:SimonB]]

I have re-tested this and the 'Disconnect from Sync?' windows position is correct.

Comment 6

[Simon Basca [:SimonB]]

Verifying as fix on master d66101cc.