Closed
Bug 934894
Opened 11 years ago
Closed 11 years ago
[Flatfish]CardView snapshot are in portrait
Categories
(Firefox OS Graveyard :: Gaia::System::Window Mgmt, defect)
Tracking
(blocking-b2g:1.3+)
People
(Reporter: gasolin, Assigned: gduan)
References
Details
(Whiteboard: [Flatfish only][developer+])
Attachments
(1 file)
CardView snapshot in tablet should be in landscape
Reporter | ||
Comment 1•11 years ago
|
||
reproduce step:
in tablet:
1. open some apps and swipe up to homescreen
2. use 2 fingers swipe up and see this issue
Updated•11 years ago
|
blocking-b2g: --- → koi?
Whiteboard: [Flatfish only]
Updated•11 years ago
|
Component: Gaia::System → Gaia::System::Window Mgmt
Updated•11 years ago
|
Whiteboard: [Flatfish only] → [Flatfish only][developer+]
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → gduan
Assignee | ||
Comment 2•11 years ago
|
||
This patch should works on tablet on mobile.
However, there's something wrong on my test env, so I cannot run any unit test locally. I will update test cases and set review asap.
Comment 3•11 years ago
|
||
until we have final decision of Flatfish base version (v1.2 or v1.3), change to --- for now.
blocking-b2g: koi? → ---
Comment 4•11 years ago
|
||
as we confirmed that Flatfish will use v1.3 and we need to respect current testing cycle, set target milestone to 12/6 since 1.3FC tag is not created yet.
blocking-b2g: --- → 1.3+
Target Milestone: --- → 1.3 Sprint 6 - 12/6
Assignee | ||
Comment 5•11 years ago
|
||
Comment on attachment 828090 [details]
Pr to master
Hi Alive,
could you help to review this patch?
Attachment #828090 -
Flags: review?(alive)
Comment 6•11 years ago
|
||
Comment on attachment 828090 [details]
Pr to master
Thanks for fixing my bug.
BTW, I'm writing unit test for window.js now. So just land it first.
Attachment #828090 -
Flags: review?(alive) → review+
Assignee | ||
Comment 7•11 years ago
|
||
Thanks Alive,
merge into master,
https://github.com/mozilla-b2g/gaia/commit/278842dfeb86c6e0634a7ff3dc76e479bf72ad99
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•