Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#52334 closed task (blessed) (fixed)

Update WordPress packages for 5.7 Beta 1

Reported by: noisysocks's profile noisysocks Owned by: noisysocks's profile noisysocks
Milestone: 5.7 Priority: normal
Severity: normal Version:
Component: Editor Keywords: has-patch commit dev-reviewed
Focuses: Cc:

Description

The @wordpress npm packages need to be updated prior to Beta 1. I'll do this in two parts so that there's less rushing in the days prior to Beta 1:

  1. Update packages as of Gutenberg 9.8 on Jan 20.
  2. Update packages as of Gutenberg 9.9 on Feb 1.

Change History (18)

This ticket was mentioned in PR #887 on WordPress/wordpress-develop by noisysocks.


4 years ago
#1

  • Keywords has-patch added

Update WordPress packages ahead of WP 5.7.

To do:

https://core.trac.wordpress.org/ticket/52334

noisysocks commented on PR #887:


4 years ago
#2

Need https://github.com/constancecchen/object-fit-polyfill to publish src to npm so that we can vendor it.

Spoke to @ajlende about this. He's going to open a PR which adds src to the npm package.

noisysocks commented on PR #887:


4 years ago
#3

Fix failing font-size-picker.test.js test.

This is the only remaining failure I get when I run Gutenberg E2E tests against this branch:

  ● Font Size Picker › should apply a custom font size using the font size input

    expect(received).toMatchSnapshot()

    Snapshot name: `Font Size Picker should apply a custom font size using the font size input 1`

    - Snapshot  - 1
    + Received  + 1

    - 
    + 
      <p style="font-size:23px">Paragraph to be made "small"</p>
      

      76 | 		// Ensure content matches snapshot.
      77 | 		const content = await getEditedPostContent();
    > 78 | 		expect( content ).toMatchSnapshot();
         | 		                  ^
      79 | 	} );
      80 | 
      81 | 	it( 'should reset a named font size using the reset button', async () => {

      at Object.<anonymous> (specs/editor/various/font-size-picker.test.js:78:21)
          at runMicrotasks (<anonymous>)

This seemingly relates to https://github.com/WordPress/gutenberg/pull/27083. I'm having a hard time figuring out if px should be appended in Core or if that's a FSE-related change that should remain in the plugin only. Do you know, @jorgefilipecosta ? If it should be changed in Core, where should that change happen?

noisysocks commented on PR #887:


4 years ago
#4

Need https://github.com/constancecchen/object-fit-polyfill to publish src to npm so that we can vendor it.

Spoke to @ajlende about this. He's going to open a PR which adds src to the npm package.

https://github.com/constancecchen/object-fit-polyfill/pull/59

noisysocks commented on PR #887:


4 years ago
#5

Unsure about whether to port i18n change in client-assets.php. Have asked in WordPress/gutenberg#28279.

Preference is to land Core-46089 as a proper fix for this. Will land the workaround that Gutenberg has if that can't be done for 5.7.

noisysocks commented on PR #887:


4 years ago
#6

Fix failing font-size-picker.test.js test.

...
This seemingly relates to WordPress/gutenberg#27083. I'm having a hard time figuring out if px should be appended in Core or if that's a FSE-related change that should remain in the plugin only. Do you know, @jorgefilipecosta ? If it should be changed in Core, where should that change happen?

It looks like in the plugin it sets the attribute to 23px because there is a px in editorSettings.__experimentalFeatures.global.typography.fontSizes. In my mind it makes sense to leave this as is (no px) because we do not want __experimentalFeatures to be added to Core.

This ticket was mentioned in Slack in #core by noisysocks. View the logs.


4 years ago

noisysocks commented on PR #887:


4 years ago
#8

Only one small issue in the editor: typing /freeform doesn't bring up the Classic block as expected (this works in Gutenberg 9.8)

Hmm... this seems okay to me?

https://i0.wp.com/user-images.githubusercontent.com/612155/106077673-1281ff80-6166-11eb-893c-822b4807e713.gif

#9 @noisysocks
4 years ago

In 50048:

Editor: Update @wordpress npm packages

Update @wordpress npm packages to the latest published versions. This means that
the block editor includes functionality that exists in Gutenberg 9.8.

See #52334.
Props isabel_brison.

#10 in reply to: ↑ description @noisysocks
4 years ago

  • Keywords has-patch removed

Replying to noisysocks:

  1. Update packages as of Gutenberg 9.8 on Jan 20.

[50048] does this.

  1. Update packages as of Gutenberg 9.9 on Feb 1.

This is next. Waiting for Gutenberg 9.9 RC to be published at the end of this week.

#11 @noisysocks
4 years ago

In 50066:

Editor: Move blocks/classic -> blocks/freeform

Moves src/wp-includes/blocks/classic to src/wp-includes/blocks/freeform. This
reflects the 'classic' block being renamed to 'freeform'.

Follows [50048].
See #52334.

This ticket was mentioned in PR #963 on WordPress/wordpress-develop by noisysocks.


4 years ago
#12

  • Keywords has-patch added

Update WordPress packages ahead of WP 5.7. This is everything that is included in Gutenberg 9.9.

I have checked git diff release/9.8 release/9.9 lib for changes and ran Gutenberg E2E tests against Core.

https://core.trac.wordpress.org/ticket/52334

#13 @noisysocks
4 years ago

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

In 50137:

Editor: Update @wordpress npm packages

Update @wordpress npm packages to the latest published versions. This means that
the block editor includes functionality that exists in Gutenberg 9.9.

Fixes #52334.

#14 @desrosj
4 years ago

In 50510:

Update $_old_files for 5.7.

Follow up to [50066].

See #52334.

#15 @desrosj
4 years ago

  • Keywords dev-feedback commit added
  • Resolution fixed deleted
  • Status changed from closed to reopened

Reopening to backport [50510].

This ticket was mentioned in Slack in #core by desrosj. View the logs.


4 years ago

#17 @davidbaumwald
4 years ago

  • Keywords dev-reviewed added; dev-feedback removed

This looks good to backport. Marking as dev-reviewed.

#18 @SergeyBiryukov
4 years ago

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

In 50511:

Update $_old_files for 5.7.

Follow up to [50066].

Props desrosj.
Reviewed by davidbaumwald, SergeyBiryukov.
Fixes #52334.

Note: See TracTickets for help on using tickets.