Closed
Bug 1087734
Opened 10 years ago
Closed 10 years ago
Use close button ("x") on new tablet tab strip
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 36
People
(Reporter: mcomella, Assigned: mcomella)
References
Details
Attachments
(2 files)
Assets and idea via bug 1064415 comment 14.
Assignee | ||
Comment 1•10 years ago
|
||
Anthony, what do you think?
Attachment #8509919 -
Flags: feedback?(alam)
Assignee | ||
Comment 2•10 years ago
|
||
Trimage was run.
Attachment #8509921 -
Flags: review?(lucasr.at.mozilla)
Assignee | ||
Comment 3•10 years ago
|
||
Comment on attachment 8509921 [details] [diff] [review]
Use new close tab asset on new tablet tab strip
Spare Lucas' review queue.
Attachment #8509921 -
Flags: review?(lucasr.at.mozilla) → review?(mhaigh)
Comment 4•10 years ago
|
||
Comment on attachment 8509919 [details]
Screenshot, new icon
Looks good!
Attachment #8509919 -
Flags: feedback?(alam) → feedback+
Comment 5•10 years ago
|
||
Comment on attachment 8509921 [details] [diff] [review]
Use new close tab asset on new tablet tab strip
Review of attachment 8509921 [details] [diff] [review]:
-----------------------------------------------------------------
You don't need the mdpi or the xhdpi assets. As Commonsware mentions here http://stackoverflow.com/a/19196749/264276, the system will downscale from the largest given asset. You will, however, need to specify an hdpi asset, as neither xhdpi nor xxhdpi existed for API Level 7 and below. Up to you if you wanna play with that - we're tracking it on bug 959203. Patch looks good to go regardless.
Attachment #8509921 -
Flags: review?(mhaigh) → review+
Assignee | ||
Comment 6•10 years ago
|
||
(In reply to Martyn Haigh (:mhaigh) from comment #5)
> You don't need the mdpi or the xhdpi assets. As Commonsware mentions here
> http://stackoverflow.com/a/19196749/264276, the system will downscale from
> the largest given asset. You will, however, need to specify an hdpi asset,
> as neither xhdpi nor xxhdpi existed for API Level 7 and below. Up to you if
> you wanna play with that - we're tracking it on bug 959203. Patch looks
> good to go regardless.
I'm going to let bug 959203 take care of this - just in case we decide we don't want to move forward with this approach or backout, it makes things cleaner.
Assignee | ||
Comment 7•10 years ago
|
||
Comment 8•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 36
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•