Make WordPress Core

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#34806 closed defect (bug) (fixed)

Core Themes: Consider updating screenshots to 1200 x 900

Reported by: davidakennedy's profile davidakennedy Owned by: karmatosed's profile 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)

34806.diff (1.1 KB) - added by coreymcollins 8 years ago.
Updated screenshots to 1200x900 from Twenty Ten to Twenty Fifteen
updated-screenshots-01-27-16.zip (1.3 MB) - added by coreymcollins 8 years ago.
Updated screenshots at 1200x900 for Twenty Ten through Twenty Fifteen themes
screenshot.png (562.7 KB) - added by iamtakashi 8 years ago.
Updated Scrrenshot of Twenty Fifteen.
screenshot.2.png (911.4 KB) - added by iamtakashi 8 years ago.
Updated Screenshot of Twenty Fourteen.
screenshot.3.png (171.4 KB) - added by iamtakashi 8 years ago.
Updated screenshot of Twenty Thirteen

Change History (24)

#1 @swissspidy
8 years ago

  • Milestone changed from Awaiting Review to Future Release
  • Version trunk deleted

As per a recent update, the maximum size allowed is now 1200x900px. Twenty Sixteen already has a screenshot of that size.

#2 @karmatosed
8 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.

#3 @swissspidy
8 years ago

  • Keywords needs-screenshots good-first-bug added

@coreymcollins
8 years ago

Updated screenshots to 1200x900 from Twenty Ten to Twenty Fifteen

#4 follow-up: @coreymcollins
8 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

Last edited 8 years ago by coreymcollins (previous) (diff)

#5 in reply to: ↑ 4 @swissspidy
8 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 @iamtakashi
8 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 @karmatosed
8 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.

@coreymcollins
8 years ago

Updated screenshots at 1200x900 for Twenty Ten through Twenty Fifteen themes

#8 @coreymcollins
8 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 @iamtakashi
8 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: @davidakennedy
8 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 @iamtakashi
8 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: @karmatosed
8 years ago

@iamtakashi any chance you could tackle this in the next few weeks?

#13 in reply to: ↑ 12 @iamtakashi
8 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.

@iamtakashi
8 years ago

Updated Scrrenshot of Twenty Fifteen.

#14 @iamtakashi
8 years ago

Removed the duplicate.

I've added a link in the content to demonstrate default link style.

Last edited 8 years ago by iamtakashi (previous) (diff)

@iamtakashi
8 years ago

Updated Screenshot of Twenty Fourteen.

#15 @iamtakashi
8 years ago

For Twenty Fourteen, I removed some Jetpack widgets from the image.

@iamtakashi
8 years ago

Updated screenshot of Twenty Thirteen

#16 @iamtakashi
8 years ago

  • Keywords has-patch added; needs-screenshots good-first-bug removed

#17 in reply to: ↑ 10 @iamtakashi
8 years ago

As @davidakennedy pointed out, I don't think we need to update older than Twenty Thirteen at this point.

#18 @karmatosed
8 years ago

  • Owner set to karmatosed
  • Resolution set to fixed
  • Status changed from new to closed

In 37033:

Twenty Thirteen, Twenty Fourteen, Twenty Fifteen: Update screenshots to 1200 x 900
Fixes #34806
Props: iamtakashi

#19 @netweb
8 years ago

  • Milestone changed from Future Release to 4.5
Note: See TracTickets for help on using tickets.