Hide the context menu ui when the homescreen is not visible

RESOLVED FIXED in Firefox OS v2.0

Status

Firefox OS
Gaia::Homescreen
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: vingtetun, Assigned: vingtetun)

Tracking

unspecified
2.0 S5 (4july)

Firefox Tracking Flags

(tracking-b2g:backlog, b2g-v2.0 fixed, b2g-v2.1 fixed)

Details

(Whiteboard: [systemsfe])

Attachments

(1 attachment)

Created attachment 8444431 [details] [diff] [review]
bug1028933.patch
Attachment #8444431 - Flags: review?(kgrandon)
QA Whiteboard: [VH-FL-blocking-][VH-FC-blocking-]
QA Whiteboard: [VH-FL-blocking-][VH-FC-blocking-] → [VH-FL-blocking-][VH-FC-blocking?]

Updated

3 years ago
Whiteboard: [systemsfe]
Comment on attachment 8444431 [details] [diff] [review]
bug1028933.patch

Review of attachment 8444431 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM.
Attachment #8444431 - Flags: review?(kgrandon) → review+
What's the user impact of this bug if we don't fix it?
Flags: needinfo?(21)
Assignee: nobody → 21
Status: NEW → ASSIGNED
(In reply to Jason Smith [:jsmith] from comment #3)
> What's the user impact of this bug if we don't fix it?

Same comment as https://bugzilla.mozilla.org/show_bug.cgi?id=1028940#c4
Flags: needinfo?(21)
QA Whiteboard: [VH-FL-blocking-][VH-FC-blocking?] → [VH-FL-blocking-][VH-FC-blocking+]
Moving this to backlog doesn't classify a blocker
blocking-b2g: 2.0? → backlog
(In reply to Preeti Raghunath(:Preeti) from comment #5)
> Moving this to backlog doesn't classify a blocker

Minor user impact.

Updated

3 years ago
QA Whiteboard: [VH-FL-blocking-][VH-FC-blocking+] → [VH-FL-blocking-][VH-FC-blocking-]
https://tbpl.mozilla.org/?tree=Gaia-Try&rev=ca3af978e4626678d71b26c6ad9827790ee9a55c
https://github.com/mozilla-b2g/gaia/commit/a73b6c18498591a5c7fb0878f5ac7c7dfc14be9a
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Comment on attachment 8444431 [details] [diff] [review]
bug1028933.patch

Review of attachment 8444431 [details] [diff] [review]:
-----------------------------------------------------------------

This is needed for the vertical homescreen.
Attachment #8444431 - Flags: approval-gaia-v2.0?(bbajaj)

Updated

3 years ago
Attachment #8444431 - Flags: approval-gaia-v2.0?(bbajaj) → approval-gaia-v2.0+
v2.0: https://github.com/mozilla-b2g/gaia/commit/7edc43f1eecefa612fc28f0982c55a2bd16555e4
status-b2g-v2.0: --- → fixed
status-b2g-v2.1: --- → fixed
Target Milestone: --- → 2.0 S5 (4july)
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #10)
> v2.0:
> https://github.com/mozilla-b2g/gaia/commit/
> 7edc43f1eecefa612fc28f0982c55a2bd16555e4

Whoops, pasted the wrong link there.

v2.0: https://github.com/mozilla-b2g/gaia/commit/f710da88b402d54a0f5461bc2253eed0b1bd652e
blocking-b2g: backlog → ---
tracking-b2g: --- → backlog
You need to log in before you can comment on or make changes to this bug.