Make WordPress Core

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#45573 closed defect (bug) (fixed)

The Twenty Nineteen version shipped with WordPress 5.0 is not in sync with Github version

Reported by: ismailelkorchi's profile ismail.elkorchi Owned by:
Milestone: 5.0.3 Priority: normal
Severity: normal Version: 5.0
Component: Bundled Theme Keywords:
Focuses: Cc:

Description

Many changes are missing from the Twenty Nineteen version shipped with WordPress 5.0. For exemple, this PR https://github.com/WordPress/twentynineteen/pull/47 was merged in november 30, and the changes made in it are asbent here : https://core.trac.wordpress.org/browser/tags/5.0/src/wp-content/themes/twentynineteen/functions.php#L224.

Change History (5)

#1 @swissspidy
6 years ago

Seems like that was missed in #45424.

Can you perhaps confirm that @allancole?

#2 @allancole
6 years ago

Yes, unfortunately this is correct, @swissspidy. The remaining patches on #45424 we’re meant to keep the two repos in sync, but they were submitted too late to test properly and were not included in the 5.0 release.

We’re aiming to get the more critical patches into 5.0.1, but non-critical fixes and general improvements will likely appear in an 1.0.1 update to the theme itself on the wporg themes repo.

We’re going to cover this in more detail in the #core-themes meeting tomorrow (12/11).

Last edited 6 years ago by allancole (previous) (diff)

#3 @laurelfulford
6 years ago

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

Thanks fore reporting this, @ismail.elkorchi!

In the past week weeks, Twenty Nineteen has fully been moved to Trac/SVN, and the GitHub repo retired; as part of that the different repos have been synced up.

I've double-checked, and the example above (the skip link JavaScript being moved inline) was included in 5.0.3, or Twenty Nineteen 1.2: https://core.trac.wordpress.org/browser/tags/5.0.3/src/wp-content/themes/twentynineteen/functions.php#L245

I'm going to close this issue since it appears to be fixed now, but just let me know if you have any questions about this at all!

#4 @desrosj
6 years ago

  • Milestone Awaiting Review deleted

Removing Awaiting Review milestone from closed tickets.

#5 @ocean90
6 years ago

  • Milestone set to 5.0.3
Note: See TracTickets for help on using tickets.