Closed Bug 1254239 Opened 4 years ago Closed 4 years ago

Align bookmark folders with other home panel items

Categories

(Firefox for Android :: General, defect)

ARM
Android
defect
Not set

Tracking

()

VERIFIED FIXED
Firefox 48
Tracking Status
firefox47 --- verified
firefox48 --- verified

People

(Reporter: liuche, Assigned: liuche)

References

(Blocks 1 open bug)

Details

Attachments

(3 files)

Next evolution step for bookmarks - instead of having a different alignment level from history items, everything should be aligned the same.
Assignee: nobody → liuche
Attachment #8727551 - Flags: feedback?(alam)
Comment on attachment 8727552 [details]
MozReview Request: Bug 1254239 - Align bookmark folders with other home panel items. r=mcomella

https://reviewboard.mozilla.org/r/38503/#review35169
Attachment #8727552 - Flags: review?(michael.l.comella) → review+
Comment on attachment 8727551 [details]
Screenshot: Aligned folder items

beautiful!
Attachment #8727551 - Flags: feedback?(alam) → feedback+
Comment on attachment 8727552 [details]
MozReview Request: Bug 1254239 - Align bookmark folders with other home panel items. r=mcomella

Approval Request Comment
[Feature/regressing bug #]: Home panels item styling
[User impact if declined]: Alignment will be off for bookmark folders
[Describe test coverage new/current, TreeHerder]: in Nightly
[Risks and why]: very, very low - changing two dimensions
[String/UUID change made/needed]: none
Attachment #8727552 - Flags: approval-mozilla-aurora?
https://hg.mozilla.org/mozilla-central/rev/56b9ab351458
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Comment on attachment 8727552 [details]
MozReview Request: Bug 1254239 - Align bookmark folders with other home panel items. r=mcomella

Seems like a simple change, taking it.
Attachment #8727552 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Flags: qe-verify+
Tested using:
Device: Huawei Honor (Android 5.1.1)
Build: Firefox for Android Beta - 47.0b6 and Aurora - 48.0a2(2016-05-15)
Since this is verified I'll mark it as verified fixed.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.