Opened 4 years ago
Closed 4 years ago
#52334 closed task (blessed) (fixed)
Update WordPress packages for 5.7 Beta 1
Reported by: |
|
Owned by: |
|
---|---|---|---|
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:
- Update packages as of Gutenberg 9.8 on Jan 20.
- 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
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 ifpx
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
#10
in reply to:
↑ description
@
4 years ago
- Keywords has-patch removed
Replying to noisysocks:
- Update packages as of Gutenberg 9.8 on Jan 20.
[50048] does this.
- 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.
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.
#15
@
4 years ago
- Keywords dev-feedback commit added
- Resolution fixed deleted
- Status changed from closed to reopened
Reopening to backport [50510].
Update WordPress packages ahead of WP 5.7.
To do:
client-assets.php
. Have asked in https://github.com/WordPress/gutenberg/pull/28279.src
to npm so that we can vendor it.client-assets.php
diff for changes I may have missed.https://core.trac.wordpress.org/ticket/52334