[Search] Landscape support

RESOLVED FIXED in 2.1 S3 (29aug)

Status

Firefox OS
Gaia::System
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: kgrandon, Assigned: kgrandon)

Tracking

unspecified
2.1 S3 (29aug)
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [systemsfe])

Attachments

(1 attachment)

46 bytes, text/x-github-pull-request
daleharvey
: review+
etienne
: review+
Details | Review | Splinter Review
(Assignee)

Description

4 years ago
From bug 1041620 comment 22

There's no landscape view of top sites
(Assignee)

Updated

4 years ago
Assignee: nobody → kgrandon
Status: NEW → ASSIGNED
Summary: Add landscape view for top sites → [Search] Landscape support
Target Milestone: --- → 2.1 S3 (29aug)
(Assignee)

Comment 1

4 years ago
Created attachment 8476430 [details] [review]
Github pull request
(Assignee)

Comment 2

4 years ago
Comment on attachment 8476430 [details] [review]
Github pull request

Hey guys, anyone have a minute for review? Thanks!
Attachment #8476430 - Flags: review?(etienne)
Attachment #8476430 - Flags: review?(dale)
Comment on attachment 8476430 [details] [review]
Github pull request

Sweet and simple :)

(Leaving Dale flagged since I don't know the grid code enough to review this part of the patch.)
Attachment #8476430 - Flags: review?(etienne) → review+
Comment on attachment 8476430 [details] [review]
Github pull request

Looks great, thanks
Attachment #8476430 - Flags: review?(dale) → review+
(Assignee)

Comment 5

4 years ago
Updated commit with reviewers and landed: https://github.com/mozilla-b2g/gaia/commit/c13356420b355d558fa312b0546ffe8b1e88e863

Thanks guys!
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.