1048593 - [B2G][Marketplace][Home Button] Software Home Button overlays on the Install App page in Marketplace

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::System, defect)

Description

[Josh Schmitt [Joshs]]

Attachment: log.txt

Description:
Software Home Button overlaps and covers buttons on the Install app popup in the Marketplace

Repro Steps:
1) Update a Flame to 20140804040204
2) Open Settings -> Device Information -> More Information and turn on Developer Menu
3) Open Developer Menu and turn on Software Home Button
4) Open Marketplace app
5) Select 'free' on an app

Actual:
Software Home Button overlays on the Install App page in Marketplace.

Expected:
Software Home Button does not overlay pages.

Environmental Variables:
Device: Flame Master 319MB
Build ID: 20140804040204
Gaia: 5fd14b8bc428f87f9b5cf9cc49f9a4f362a970fb
Gecko: e6614d8d85f9
Version: 34.0a1 (Master)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

Issue occurs on 319MB

Notes:
Repro frequency: 100%
See attached: Screenshot, ogcat

Comment 1

[Josh Schmitt [Joshs]]

Attachment: Market_Place_HomeButton.png

Comment 2

[KTucker [:KTucker][Inactive 3/4/2016]]

[Blocking Requested - why for this release]:

The buttons can be tapped and they are functional but the user can't really tell what each button does and this just looks bad so nominating 2.1?

Comment 3

[Michael Henretty [:mikehenrty][:mhenretty]]

Assigning to Eli based on: https://bugzilla.mozilla.org/show_bug.cgi?id=1048620#c5

Comment 4

[:Eli Perelman]

Attachment: Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/23047

This patch modifies the system confirmation dialogs to account for the reduced screen height from enabling the software home button.

Comment 5

[:Eli Perelman]

The root cause of this issue is that the System app is responsible for both the display of the software home button and the confirmation dialogs used by the Marketplace and did not account for the height of the software home button when calculating the dimensions and placement of these older confirmation dialogs.

Comment 6

[Michael Henretty [:mikehenrty][:mhenretty]]

Comment on attachment 8475400 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/23047

Working for me. Fabien is not the right guy for a system change like this. You most likely want Alive or Etienne.

Comment 7

[Etienne Segonzac (:etienne)]

Comment on attachment 8475400 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/23047

r=me with the comment on github addressed

Comment 8

[Ryan VanderMeulen [:RyanVM]]

Master: https://github.com/mozilla-b2g/gaia/commit/c85f417d1fbb3feb8ecb845d5b5dbecb782deb43

Comment 9

[pete siphantong [:Petes]]

This bug is verified fixed on the Flame 2.1 (319mb) and the Flame 2.2 (319mb)


Flame 2.2 Master KK (319mb) (Full Flash)

Device: Flame 2.2 Master
BuildID: 20141011040204
Gaia: 95f580a1522ffd0f09302372b78200dab9b6f322
Gecko: 3f6a51950eb5
Gonk: 52c909e821d107d414f851e267dedcd7aae2cebf
Version: 35.0a1 (2.2 Master)
Firmware: V180
User Agent: Mozilla/5.0 (Mobile; rv:35.0) Gecko/35.0 Firefox/35.0

Flame 2.1 KK (319mb) (Full Flash)

Device: Flame 2.1
BuildID: 20141011000201
Gaia: f5d4ff60ffed8961f7d0380ada9d0facfdfd56b1
Gecko: d813d79d3eae
Gonk: 52c909e821d107d414f851e267dedcd7aae2cebf
Version: 34.0a2 (2.1)
Firmware: V180
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0


Result: Software Home Button does not overlay pages

Comment 10

[Kevin Grandon :kgrandon]

Integration test added in bug 1077579.