Closed Bug 903918 Opened 11 years ago Closed 11 years ago

[Flatfish][Settings] support 2 column layout for tablet

Categories

(Firefox OS Graveyard :: Gaia::Settings, defect)

Product:
Firefox OS Graveyard
Component:
Gaia::Settings
Platform:
Other
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
1.3 Sprint 4 - 11/8

People

(Reporter: gasolin, Assigned: dwi2)

References

Details

(Whiteboard: [Flatfish only][developer+])

Attachments

(9 files)

Change_Homescreen.png
(deleted), image/png
Details
Display_Select.png
(deleted), image/png
Details
Passcode.png
(deleted), image/png
Details
Settings.png
(deleted), image/png
Details
Sound.png
(deleted), image/png
Details
Wi-Fi.png
(deleted), image/png
Details
Passcode_device.png
(deleted), image/png
Details
Settings layout spec
(deleted), application/zip
Details
pull request
(deleted), text/x-github-pull-request
arthurcc
: review+
Details
Expect: Modify and Merge experiment tablet work from https://github.com/gaia-local/gaia to master
Blocks: flatfish
blocking-b2g: --- → koi+
Assignee: nobody → arthur.chen
Attached image Display_Select.png (deleted) —
Attached image Passcode.png (deleted) —
Attached image Settings.png (deleted) —
Attached image Sound.png (deleted) —
Attached image Wi-Fi.png (deleted) —
Comment on attachment 791923 [details] Change_Homescreen.png Currently change homescreen won't work: press the change button and nothing happens, except it will goes back to the homescreen.
Attached image Passcode_device.png (deleted) —
(In reply to Greg Weng [:snowmantw] from comment #3) > Created attachment 791925 [details] > Passcode.png UPDATE: Passcode seems fine at the device. See Attachment 792575 [details].
pre comment 7, I think its because we dont have another homescreen app. Maybe a bug that should disabled the change button if current is the default homescreen.
Nice to have at current stage.
blocking-b2g: koi+ → ---
It's a must for flatfish. koi+
blocking-b2g: --- → leo+
Whiteboard: [Flatfish only]
blocking-b2g: leo+ → koi+
Assignee: arthur.chen → ejchen
Flatfish only UI change, move to 1.3+.
blocking-b2g: koi+ → 1.3+
Summary: [Flatfish][Settings] add support for large device → [Flatfish][Settings] support 2 column layout for tablet
Assignee: ejchen → tzhuang
according to 10/18 meeting result, ETA date will be 11/5 (ready for review), 11/14 landed.
Target Milestone: --- → 1.3 Sprint 4 - 11/8
Depends on: 934326
Attached file Settings layout spec (deleted) —
Attached file pull request (deleted) —
Hi Arthur, Please kindly help to review this patch, Thanks
Attachment #827810 - Flags: review?(arthur.chen)
implementation resource and ETA date are both agree with RD team. this has to be done before 1.3FC hi arthur, can you please kindly support this review?
Flags: needinfo?(arthur.chen)
Whiteboard: [Flatfish only] → [Flatfish only][developer+]
As there are changes of multiple tablet apps in common, we will have a meeting discussing this today. Plan to finish the review within this week.
Flags: needinfo?(arthur.chen)
Comment on attachment 827810 [details] pull request Cancel the review at first per a offline discussion with tzu-lin. He needs some time to rebase to master.
Attachment #827810 - Flags: review?(arthur.chen)
blocking-b2g: 1.3+ → ---
Comment on attachment 827810 [details] pull request Hi Arthur, I've finish all rebase and clean up work, please help to review the patch. Thanks
Attachment #827810 - Flags: review?(arthur.chen)
Blocks: 943173
Comment on attachment 827810 [details] pull request Thank you for the patch. r=me with the followup bug (removing the dup code with building block) created and green travis.
Attachment #827810 - Flags: review?(arthur.chen) → review+
The followup bug is bug 943173
master https://github.com/mozilla-b2g/gaia/commit/2d1f0e90f9918ce6059eed878f7ea5e68a9879b3
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.

Attachment

General

Creator:
Created:
Updated:
Size: