Closed
Bug 1012544
Opened 10 years ago
Closed 10 years ago
[System] Unable to access software home button after enabling from settings
Categories
(Firefox OS Graveyard :: Gaia::System, defect)
Tracking
(b2g-v2.0 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
b2g-v2.0 | --- | fixed |
People
(Reporter: kgrandon, Assigned: etienne)
References
Details
(Whiteboard: [systemsfe],[p=1])
Attachments
(1 file, 1 obsolete file)
A recent z-index change has made it so that something is over the software home button when first enabling it.
Reporter | ||
Comment 1•10 years ago
|
||
Hey guys - got time for a quick review?
Attachment #8424630 -
Flags: review?(etienne)
Attachment #8424630 -
Flags: review?(alive)
Comment 2•10 years ago
|
||
Comment on attachment 8424630 [details] [review] Github pull request I think the problem here is: why the appWindow is not resized correctly on software button enabled? The overlay should be the same sizing as its appWindow container so I don't think this is a right fix.
Attachment #8424630 -
Flags: review?(alive)
Assignee | ||
Comment 3•10 years ago
|
||
Comment on attachment 8424630 [details] [review] Github pull request Hum, could this be caused by bug 1011262?
Attachment #8424630 -
Flags: review?(etienne)
Reporter | ||
Comment 4•10 years ago
|
||
(In reply to Etienne Segonzac (:etienne) from comment #3) > Comment on attachment 8424630 [details] [review] > Github pull request > > Hum, could this be caused by bug 1011262? I don't think so. I think it was always there, just surfaced due to the z-index changes in bug 992081.
Assignee | ||
Comment 5•10 years ago
|
||
Apparently the appWindow is resized correctly the the homescreen isn't, patch incoming.
Assignee | ||
Comment 7•10 years ago
|
||
What do you think? Sounds fair to resize the homescreen on every system-resize when it's loaded. And that will be one less reflow when coming back to the homescreen :)
Attachment #8424630 -
Attachment is obsolete: true
Attachment #8424956 -
Flags: review?(alive)
Comment 8•10 years ago
|
||
Sorry I don't think so.. that means homescreen will resize when you have a foreground app using keyboard or when you have a landscape foreground app.
Comment 9•10 years ago
|
||
(In reply to Alive Kuo [:alive][NEEDINFO!][5/11-5/15@MV] from comment #8) > Sorry I don't think so.. that means homescreen will resize when you have a > foreground app using keyboard or when you have a landscape foreground app. OK so this is regressed from bug 1011262. My idea is always resized at opened handler. Could we redo that?
Assignee | ||
Comment 10•10 years ago
|
||
(In reply to Alive Kuo [:alive][NEEDINFO!][5/11-5/15@MV] from comment #8) > Sorry I don't think so.. that means homescreen will resize when you have a > foreground app using keyboard or when you have a landscape foreground app. Good point. Resizing at opened won't help with this bug since the system-resize occurs with no app change, we stay on the settings app. We can: * make sure the software button always has a higher z-index than the homescreen * keep the homescreenwindow specific resize method and ignore keyboard related system-resize when !isActive(). Any preference?
Flags: needinfo?(alive)
Assignee | ||
Comment 11•10 years ago
|
||
Comment on attachment 8424956 [details] [review] Gaia PR clearing the r? flag until we decide on a solution
Attachment #8424956 -
Flags: review?(alive)
Comment 12•10 years ago
|
||
(In reply to Etienne Segonzac (:etienne) from comment #10) > (In reply to Alive Kuo [:alive][NEEDINFO!][5/11-5/15@MV] from comment #8) > > Sorry I don't think so.. that means homescreen will resize when you have a > > foreground app using keyboard or when you have a landscape foreground app. > > Good point. > > Resizing at opened won't help with this bug since the system-resize occurs > with no app change, we stay on the settings app. Well, I am sure if we don't resize on opened we will regress bug 1003870. What about do both - resize active app and resize in _opened? Does this regress 1011262? > > We can: > * make sure the software button always has a higher z-index than the > homescreen > * keep the homescreenwindow specific resize method and ignore keyboard > related system-resize when !isActive(). > > Any preference?
Flags: needinfo?(alive)
Comment 13•10 years ago
|
||
I tried to add this.app.resize back and keep appWindowManager resize activeApp -- it seems homescreen correct resizes but settings app is resized with fade-overlay's size larger than its container appWindow which is correctly resized to 100%-2.4rem-5rem Strange..
Assignee | ||
Comment 14•10 years ago
|
||
Comment on attachment 8424956 [details] [review] Gaia PR And how about resizing the homescreen from the launcher just for software-button events?
Attachment #8424956 -
Flags: review?(alive)
Comment 15•10 years ago
|
||
Comment on attachment 8424956 [details] [review] Gaia PR For now Let's do this. I am adjusting resize logic (again) in bug 992085.
Attachment #8424956 -
Flags: review?(alive) → review+
Assignee | ||
Comment 16•10 years ago
|
||
https://github.com/mozilla-b2g/gaia/commit/5f29469b4a9a728781430a0c426f1978529d1de9
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 17•10 years ago
|
||
sorry had to revert this change since i guess this caused https://tbpl.mozilla.org/php/getParsedLog.php?id=40534597&tree=B2g-Inbound
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 20•10 years ago
|
||
Relanded with travis and try green: https://github.com/mozilla-b2g/gaia/commit/9833baf032ad5524b8e7f0e0adcceb450f7c8aac
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 21•10 years ago
|
||
Mass modify - set status-b2g-v2.0 fixed for fixed bugs under vertical homescreen dependency tree.
status-b2g-v2.0:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•