Closed
Bug 1153779
Opened 9 years ago
Closed 9 years ago
[SHB] Wrong dialog size on SIM PIN dialog with SHB enabled and landscape
Categories
(Firefox OS Graveyard :: Gaia::System::Window Mgmt, defect)
Tracking
(blocking-b2g:2.2+, b2g-v2.2 verified, b2g-master verified)
People
(Reporter: apastor, Assigned: apastor)
Details
(Whiteboard: [systemsfe])
Attachments
(3 files)
STR: 1.- Start the phone with SHB enabled 2.- Skip SIM dialogs 3.- Open the browser app 4.- Enable airplane mode 5.- Disable airplane mode (SIM dialogs will be shown) 6.- Change orientation to landscape Expected: SIM PIN dialog resizes Actual: See screenshot
Assignee | ||
Comment 1•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
blocking-b2g: --- → 2.2?
Assignee | ||
Updated•9 years ago
|
Whiteboard: [systemsfe]
Comment 2•9 years ago
|
||
Ugh this seems like a regression. Didn't we pin the sim dialog to portrait only?
blocking-b2g: 2.2? → 2.2+
Updated•9 years ago
|
Assignee: nobody → apastor
No we didn't. We probably should, esp since the keyboard would cover the dialog anyhow. asking fxos ux. By the way, Alberto, what device are you testing on? I can't repeat the gap that you show.
status-b2g-v2.2:
--- → affected
status-b2g-master:
--- → affected
Flags: needinfo?(firefoxos-ux-bugzilla)
Assignee | ||
Comment 4•9 years ago
|
||
It seems a regression that has been there for a while (Bug 1095805). Naoki, I'm testing on a Flame with SHB enabled. Thanks!
Comment 5•9 years ago
|
||
Assignee | ||
Comment 6•9 years ago
|
||
Comment on attachment 8592863 [details] [review] [gaia] albertopq:1153779-sim-dialog > mozilla-b2g:master As the child elements of dialog-overlay are .generic-dialogs, and [1] is already applying the 'right' value, I think that line is not needed anymore. Is that correct, Kevin? Thanks! [1] https://github.com/mozilla-b2g/gaia/pull/29525/files#diff-76c229a6e5425345d3202703d8948d7aL32
Attachment #8592863 -
Flags: review?(kgrandon)
Comment 7•9 years ago
|
||
Comment on attachment 8592863 [details] [review] [gaia] albertopq:1153779-sim-dialog > mozilla-b2g:master I am very concerned about breaking bug 1095805, especially without tests for both bugs (due to lacking b2g desktop landscape support). Code looks fine to me though, thanks.
Attachment #8592863 -
Flags: review?(kgrandon) → review+
Comment 8•9 years ago
|
||
Can't repro, looks like a regression, suggest pin to portrait if possible and wouldn't block.
Flags: needinfo?(firefoxos-ux-bugzilla)
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Updated•9 years ago
|
Keywords: checkin-needed
Comment 9•9 years ago
|
||
Pull request has landed in master: https://github.com/mozilla-b2g/gaia/commit/7281c3ef59ecf2cf966580102e0f658a0c8bea92
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 10•9 years ago
|
||
Comment on attachment 8592863 [details] [review] [gaia] albertopq:1153779-sim-dialog > mozilla-b2g:master [Approval Request Comment] [Bug caused by] (feature/regressing bug #): 1095805 [User impact] if declined: Dialogs in landscape with SHB on, are misplaced [Testing completed]: Manual testing (we cannot test landscape in marionette) [Risk to taking this patch] (and alternatives if risky): CSS one liner. Low risk [String changes made]: -
Attachment #8592863 -
Flags: approval-gaia-v2.2?(bbajaj)
Updated•9 years ago
|
Attachment #8592863 -
Flags: approval-gaia-v2.2?(bbajaj) → approval-gaia-v2.2+
Comment 11•9 years ago
|
||
v2.2: https://github.com/mozilla-b2g/gaia/commit/fa5444757443e6a5b652b92a00650bb070ad1a1e
Target Milestone: --- → 2.2 S10 (17apr)
Comment 12•9 years ago
|
||
This issue is verified fixed on Flame Master. Result: The SIM PIN dialog resizes. Environmental Variables: Device: Flame 3.0 (KK, 319mb, full flash) Build ID: 20150417010203 Gaia: 3cd0a9facce26c2acc7be3755a17131a6358e33f Gecko: 51e3cb11a258 Gonk: b83fc73de7b64594cd74b33e498bf08332b5d87b Version: 40.0a1 (3.0) Firmware Version: v18D-1 User Agent: Mozilla/5.0 (Mobile; rv:40.0) Gecko/40.0 Firefox/40.0 =============================== Adding verifyme for 2.2 verificaiton.
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Keywords: verifyme
Updated•9 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Comment 13•9 years ago
|
||
According to the STR of Comment 0,this bug has been successfully verified on latest Nightly Flame v2.2. See attachment: verified_v2.2.mp4. Reproduce rate: 0/5 Actual Results: The SIM PIN dialog resizes. Device: Flame 2.2 build(Pass) Build ID 20150419002502 Gaia Revision c15a2b6d3a783813959c2b3bffd2a131f4270b9e Gaia Date 2015-04-17 17:49:32 Gecko Revision https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/cc02ee38b252 Gecko Version 37.0 Device Name flame Firmware(Release) 4.4.2 Firmware(Incremental) eng.cltbld.20150419.040848 Firmware Date Sun Apr 19 04:08:59 EDT 2015 Bootloader L1TC000118D0
Keywords: verifyme
Comment 14•9 years ago
|
||
Updated•9 years ago
|
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage+][MGSEI-Triage+]
You need to log in
before you can comment on or make changes to this bug.
Description
•