WordPress.org
Get WordPress

Make WordPress Core

Opened 7 years ago

Closed 6 years ago

#49316 closed defect (bug) (fixed)

Twenty Twenty missed license for images.

Reported by: h71 Owned by: ianbelanger
Priority: normal Milestone: 5.4.2
Component: Bundled Theme Version: 5.3
Severity: normal Keywords: commit has-patch fixed-major
Cc: Focuses:

Description

Used images in the Twenty Twenty theme are not listed with license information in the readme file. This is a requirement based on theme review team comments.

This is important as many theme developers are using default themes as a standard convention.

Attachments (1)

49316.patch​ (528 bytes ) - added by poena 6 years ago.
updated image credits

Download all attachments as: .zip

Change History (10)

#1 @SergeyBiryukov
7 years ago

  • Component Themes → Bundled Theme
  • Summary Tweny Tweny missed license for images. → Twenty Twenty missed license for images.

Hi there, welcome back to WordPress Trac! Thanks for the ticket.

Just adding links to previous similar tickets here:

#2 @ianbelanger
7 years ago

  • Keywords needs-patch added
  • Milestone Awaiting Review → 5.4
  • Type enhancement → defect (bug)
  • Version → 5.3

Adding this to 5.4 milestone as it is a simple fix.

#3 @poena
7 years ago

Hi

The theme review team did indeed talk about this on Slack but can you specify exactly which images?
The illustrations that are in the assets/images folder are already mentioned in the readmefile.

Last edited 7 years ago by poena (previous) (diff)

#4 @h71
7 years ago

Hi, I see the term

Illustrations in screenshot.png ...

But not all of the illustrations are used in the screenshot.png

#5 @ianbelanger
7 years ago

  • Keywords 2nd-opinion added; needs-patch removed
  • Milestone 5.4 → Future Release

Punting this out of 5.4 as it needs more feedback.

@poena
6 years ago

updated image credits

#6 @ianbelanger
6 years ago

  • Keywords commit has-patch added; 2nd-opinion removed
  • Milestone Future Release → 5.5
  • Owner set to ianbelanger
  • Status new → reviewing

Reviewing for commit

#7 @ianbelanger
6 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 47608:

Bundled Themes: Twenty Twenty missed license for images.

Fixes the image license in the readme by removing screenshot.png.

Props h71, poena.
Fixes #49316.

#8 @ianbelanger
6 years ago

  • Keywords fixed-major added
  • Milestone 5.5 → 5.4.2
  • Resolution fixed
  • Status closed → reopened

Reopening for backport.

#9 @whyisjake
6 years ago

  • Resolution → fixed
  • Status reopened → closed

In 47827:

Bundled Themes: Twenty Twenty missed license for images.
Fixes the image license in the readme by removing screenshot.png.

Brings the changes from [47608] to the 5.4 branch.

Props h71, poena.
Fixes #49316.

Note: See TracTickets for help on using tickets.