#34806 closed defect (bug) (fixed)
Core Themes: Consider updating screenshots to 1200 x 900
Reported by: | davidakennedy | Owned by: | karmatosed |
---|---|---|---|
Milestone: | 4.5 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Bundled Theme | Keywords: | has-patch |
Focuses: | Cc: |
Description
Consider updating screenshots in bundled themes to 1200 x 900. This would be in Twenty Fifteen, Twenty Fourteen and Twenty Thirteen.
Attachments (5)
Change History (24)
#2
@
9 years ago
I'd not be against having all themes update if someone does want to go through them all. However, also just as happy for it to be something we do with each new theme rather than going through all the old ones. Potentially this could be an awesome contribution day or first patch candidate.
#4
follow-up:
↓ 5
@
9 years ago
Updated screenshots back to Twenty Ten. I used the base WP theme unit test data in my local setup, but if we want to update the content to be more user-friendly/dynamic, I can certainly do that as well.
That may raise a question about potentially altering or offering an alternative set of data for testing themes - one more dev-centric, so we can test various features of the theme and one more user-friendly that focuses more on content and the editorial aspects of WordPress.
Realizing the diff may not work in this instance, so here's a zip of the screenshots as well: https://www.dropbox.com/s/owqckoet7ykygdz/updated-screenshots.zip?dl=0
#5
in reply to:
↑ 4
@
9 years ago
Thanks, @coreymcollins, but those screenshots seem to be only 880 × 660 in size?
As long as the screenshot does show the individual theme's strengths (i.e. featured images, featured posts, etc.), I don't care about the test data being used. Not sure there ever was an "official" set of test data. For the one used in the directory there is a meta ticket.
#6
@
9 years ago
As far as I know, theme designers and devs including me have put a fair amount of thoughts on what's in each screenshot. To respect the effort, the new image should be identical but the size.
#7
@
9 years ago
I would add a +1 to what @iamtakashi said. In my saying this is an easy patch, my assumption was it would be just retaking the same image larger, the demo site for example. It wasn't that someone would create new demo sites.
#8
@
9 years ago
Adding updated screenshots taken from the various demo sites of the themes. Twenty Ten through Twenty Fifteen at 1200x900, as Twenty Sixteen is already at that size.
#9
@
9 years ago
Thanks but I'm not happy with the latest zip.
- Twenty Fifteen - It's off centre.
- Twenty Fourteen - The slider featured content pushes the first post title out of screenshot.
- Twenty Thirteen - Blog Archives is highlighted, the first post looks like an image format but there is no image.
I guess this needs to be done by the designer or who has the access to the demo sites so that it can match with the current screenshots.
@davidakennedy is only suggesting to update these three themes. I'm unclear the reasoning but I'll start with those themes.
#10
follow-up:
↓ 17
@
9 years ago
@iamtakashi
... only suggesting to update these three themes. I'm unclear the reasoning but I'll start with those themes.
I only suggested these since they're currently bundled with new installs (I think). That seemed like a good place to start. Updating all of them is cool too, but I wasn't sure how big of a priority it would be.
Also, I didn't tackle this because of not having access to the original demo sites. It does seem something more easily done by the original designer.
#11
@
9 years ago
@davidakennedy, Thanks for the clarification. I haven't got time to tackle this but this has been on my list :)
#12
follow-up:
↓ 13
@
9 years ago
@iamtakashi any chance you could tackle this in the next few weeks?
#13
in reply to:
↑ 12
@
9 years ago
Replying to karmatosed:
@iamtakashi any chance you could tackle this in the next few weeks?
Sure. Sorry for not being able to work on this yet.
#14
@
9 years ago
Removed the duplicate.
I've added a link in the content to demonstrate default link style.
#17
in reply to:
↑ 10
@
9 years ago
As @davidakennedy pointed out, I don't think we need to update older than Twenty Thirteen at this point.
As per a recent update, the maximum size allowed is now 1200x900px. Twenty Sixteen already has a screenshot of that size.