Closed
Bug 1118390
Opened 10 years ago
Closed 10 years ago
Lock task manager (and browser tab view) orientation to vertical
Categories
(Firefox OS Graveyard :: Gaia::System::Window Mgmt, defect)
Tracking
(blocking-b2g:2.2+, b2g-v2.2 verified, b2g-master verified)
People
(Reporter: sfoster, Assigned: sfoster)
References
Details
(Whiteboard: systemsfe)
Attachments
(1 file)
(deleted),
text/x-github-pull-request
|
etienne
:
review+
bajaj
:
approval-gaia-v2.2+
|
Details |
For parity with 2.1 and as a stop-gap while the screenshot and task manager orientation issues are being resolved, the task manager should be locked to vertical orientation.
Updated•10 years ago
|
blocking-b2g: --- → 2.2+
Assignee | ||
Comment 1•10 years ago
|
||
Patch just calls lock/unlock orientation when showing/hiding task-manager, to ensure task-manager is always in default orientation. No luck with writing a marionette to test the expected behavior - I gather orientation support is a work-in-progress - so I've just asserted this the lock/unlock calls occur in a unit test.
This seems pretty solid from the testing I've done, nothing unexpected in the show or hide animations - excepting the known issues in bug 1100610 and bug 1107139.
Attachment #8555039 -
Flags: review?(etienne)
Comment 2•10 years ago
|
||
Comment on attachment 8555039 [details]
PR lock/unlock orientation in task manager
We should let the opened app |setOrientation| (and it's probably already doing it, then being overridden by the |screen.mozUnlockOrientation()| call, but I haven't checked for sure).
Ie. with browser and dialer open.
If I open the cardview from the browser then switch to the dialer. I don't want the screen orientation to be unlocked, I want the dialer to lock it to portrait.
I'm currently looking at a landscape dialer :)
Attachment #8555039 -
Flags: review?(etienne) → review-
Assignee | ||
Comment 3•10 years ago
|
||
Comment on attachment 8555039 [details]
PR lock/unlock orientation in task manager
Removed the unlockOrientation, this is now consistent with how we handled orientation in 1.* and before the new design landed. In the transition to 'opened' app windows will have setOrientation -> lockOrientation called via the transitionController. That is covered in the transition controller unit tests and seems to be the expectation as you indicated.
Attachment #8555039 -
Flags: review- → review?(etienne)
Comment 4•10 years ago
|
||
Comment on attachment 8555039 [details]
PR lock/unlock orientation in task manager
r=me with the comments addressed (one of them is optional)
Attachment #8555039 -
Flags: review?(etienne) → review+
Assignee | ||
Comment 8•10 years ago
|
||
Filed follow-up bug 1127426 for exposing defaultOrientation via Service.registerState
PR is updated with requested test change, I'll land when Gaia-Try looks good
Assignee | ||
Comment 9•10 years ago
|
||
Merged to master: https://github.com/mozilla-b2g/gaia/commit/499f79dc3ca05c7d9438473725ec3597dde7b300
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 10•10 years ago
|
||
Comment on attachment 8555039 [details]
PR lock/unlock orientation in task manager
[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): Task Manager
[User impact] if declined: Task Manager will operate in landscape, but transitions, screenshots and other details in this mode are incomplete/janky/buggy
[Testing completed]: On device, Gaia-Try
[Risk to taking this patch] (and alternatives if risky): Low risk, restores expected behavior from <= 2.1
[String changes made]: None
Attachment #8555039 -
Flags: approval-gaia-v2.2?
Updated•10 years ago
|
Attachment #8555039 -
Flags: approval-gaia-v2.2? → approval-gaia-v2.2+
Comment 11•10 years ago
|
||
status-b2g-v2.2:
--- → fixed
status-b2g-master:
--- → fixed
Target Milestone: 2.2 S3 (9jan) → 2.2 S5 (6feb)
Comment 13•10 years ago
|
||
This issue is verified fixed on Flame Master and 2.2.
Result: The task manager and the browser tab view are locked to vertical mode. When the device is set in landscape mode in Browser, Gallery and Video, the task manager appears in vertical mode. Changing orientation in task manager is not allowed.
Device: Flame Master (KK, 319mb, full flash)
Build ID: 20150319010201
Gaia: c39e15f631de80c69467fda0d4ea0bcda9e194ca
Gecko: cf1060d8ce9f
Gonk: b83fc73de7b64594cd74b33e498bf08332b5d87b
Version: 39.0a1 (3.0)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:39.0) Gecko/39.0 Firefox/39.0
Device: Flame 2.2 (KK, 319mb, full flash)
Build ID: 20150319002500
Gaia: 9043c11f699c15bb6072422d1dad6518d1b5ddda
Gecko: c0442d170bec
Gonk: ebad7da532429a6f5efadc00bf6ad8a41288a429
Version: 37.0 (2.2)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Assignee | ||
Updated•9 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•