Closed
Bug 1064327
Opened 10 years ago
Closed 10 years ago
The SHB is displayed for apps with fullscreen_layout:true... even when the Software Home Button is disabled :/
Categories
(Firefox OS Graveyard :: Gaia::System, defect)
Tracking
(b2g-v2.1 verified, b2g-v2.2 verified)
VERIFIED
FIXED
2.1 S4 (12sep)
People
(Reporter: etienne, Assigned: Eli)
References
Details
(Keywords: polish, Whiteboard: [systemsfe])
Attachments
(1 file)
(deleted),
text/x-github-pull-request
|
etienne
:
review+
bajaj
:
approval-gaia-v2.1+
|
Details |
STR:
* add "fullscreen_layout":true to the manifest of the UITests app (or other)
* disable the software home button
* launch the UITests app
Expected:
* the app should display normally
Actual:
* a non-functional software home button is overlayed on the app
It's probably a simple fix, where we're missing a `#screen.software-button-enabled` on a rule.
Reporter | ||
Updated•10 years ago
|
Blocks: soft-home-button
Updated•10 years ago
|
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → eperelman
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8487237 -
Flags: review?(etienne)
Assignee | ||
Updated•10 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Updated•10 years ago
|
Target Milestone: --- → 2.1 S4 (12sep)
Comment 2•10 years ago
|
||
Removing from meta as this is not a blocker for 2.1.
No longer blocks: soft-home-button
Reporter | ||
Comment 4•10 years ago
|
||
Comment on attachment 8487237 [details]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/23913
:)
Attachment #8487237 -
Flags: review?(etienne) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed,
polish
Comment 5•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-b2g-v2.2:
--- → fixed
Keywords: checkin-needed
Resolution: --- → FIXED
Assignee | ||
Comment 6•10 years ago
|
||
Comment on attachment 8487237 [details]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/23913
Requesting uplift to v2.1 for feature polish.
[Bug caused by] (feature/regressing bug #): bug 1036339
[User impact] if declined: buggy SHB for fullscreen_layout apps
[Testing completed]: yes, manual
[Risk to taking this patch] (and alternatives if risky): super low, only single CSS change
[String changes made]: n/a
Attachment #8487237 -
Flags: approval-gaia-v2.1?
Updated•10 years ago
|
Attachment #8487237 -
Flags: approval-gaia-v2.1? → approval-gaia-v2.1+
Comment 7•10 years ago
|
||
status-b2g-v2.1:
--- → fixed
Comment 8•10 years ago
|
||
The issue is verified fixed in Flame 2.1 and Flame 2.2:
Flame 2.1 KitKat Base (319mb)(Shallow Flash)
Environmental Variables:
Device: Flame 2.1
BuildID: 20141008000201
Gaia: d71f8804d7229f4b354259d5d8543c25b4796064
Gecko: 7fa82c9acdf2
Gonk: 52c909e821d107d414f851e267dedcd7aae2cebf
Version: 34.0a2 (2.1)
Firmware: V180
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0
Flame 2.2 KitKat Base (319mb)(Shallow Flash)
Environmental Variables:
Device: Flame 2.2 Master
BuildID: 20141008040203
Gaia: 0bc74ce502672cf0265b24cf3a25d117c3de5e71
Gecko: e4cfacb76830
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
The software home button was not appearing in an app when "fullscreen_layout":true was added to the manifest.
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
You need to log in
before you can comment on or make changes to this bug.
Description
•