Closed Bug 1152858 Opened 9 years ago Closed 9 years ago

Tab tray thumbnails don't fill tiles

Categories

(Firefox for iOS :: Theme & Visual Design, defect)

x86
iOS 8
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
fennec + ---

People

(Reporter: dhenein, Assigned: sleroux)

Details

Attachments

(3 files, 1 obsolete file)

After the collection view port, tab tiles are sized dynamically. When in landscape mode, the thumbnail we get from the transitioning webview is very short, and does not fill the tile completely.

Would be nice to find a way to capture taller screenshots.
tracking-fennec: ? → +
Assignee: nobody → sleroux
Attached patch PR (obsolete) — Splinter Review
Attachment #8603482 - Flags: review?(bnicholson)
Attached file PR
Attachment #8603482 - Attachment is obsolete: true
Attachment #8603482 - Flags: review?(bnicholson)
Attachment #8603486 - Flags: review?(bnicholson)
Comment on attachment 8603486 [details] [review]
PR

Looks like this will need a bit more investigation.
Attachment #8603486 - Flags: review?(bnicholson)
I spoke with @darrinhenein and I think we can make the thumbnail image scale enough to fill the cell when the screenshot will be too narrow to fill the cell's height. The only downside of this is the right edge of the snapshot on iPhone landscape won't align with the edge of the web page like it does on portrait
Doesn't look like it's an issue anymore. I can't seem to pinpoint the change but the UIImageView frame is now scaling down correctly. I've attached a before and after screenshot
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: