Closed Bug 774895 Opened 10 years ago Closed 10 years ago
Change thumbnail sizing to 180x270
As per the new mobile designs, the maximum dimensions for app thumbnails are 180x270. We'll need to start proportionally scaling the thumbnails to that size. Additionally, when this change goes into effect, we will need to re-process all the existing screenshots for apps to meet this requirement.
If we haven't yet merged the gaia-ui branch when this work is completed, it should be landed there instead of master.
OS: Mac OS X → All
Hardware: x86 → All
Maria, can we confirm this is the correct size we want for mobile?
(In reply to Chris Van Wiemeersch [:cvan] from comment #2) > Maria, can we confirm this is the correct size we want for mobile? Yes - in fact I'm pretty sure that Potch's original comment came from me confirming this in IRC or an email.
Awesome. And for Desktop, same size restriction? Or did we want something proportional?
This seems like an awesome Basta bug.
We should really do this before we go public so we don't get stuck with a bunch of blurry screenshots such as this one for Twitter: http://f.cl.ly/items/3q2F170I221N1s2i3q0i/Screen%20shot%202012-10-07%20at%208.47.47%20PM.png
Target Milestone: --- → 2012-10-11
This isn't much of a change, but it serves the purpose that cvan and I discussed offline: https://github.com/mozilla/zamboni/commit/2aee52cb8240449c5a8eabdfa69cf889f7797575 We're not going to change the proportions of the image, because that's an irreversible change. Instead, potch can vertically center the images on the front-end. They should never end up being wider than 180 or taller than 270, though, so they'll always fit nicely. There's also no migration here. On the front-end, we can probably say (safely) `max-width: 180px; max-height: 270px;` and it'll do what we expect. We should send an email to developers saying that they should upload new screenshots that meet the dimensions.
Let's make screenshot minimum sizes be 320x480 now.
Assignee: nobody → mattbasta
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Verified as fixed in https://marketplace-dev.allizom.org/developers/ on FF20 (Win 7) Postfix screencast http://screencast.com/t/6voCS1UvZo Closing bug.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.