Skip to content

Commit f99c357

Browse files
Merge branch 'trunk' into replace/confirm-et-al
* trunk: (214 commits) Fix snackbar overflow on nav editor (#34661) Mobile - Allow disabling text and background color via theme.json (#34633) Fix disabled blocks logical error on Widgets screen (#34634) [Mobile] - Global styles - Add support to render font sizes and line height (#34144) ESLint Plugin: Update eslint jsdoc dependency (#34338) Scripts: Add CHANGELOG entry for `jest-dev-server` upgrade (#34657) Bump jest-dev-server to v5 (#34560) Refactor the `core-data` store to thunks (#28389) Only capture toolbars on parent Nav block when not in vertical mode (#34615) Update Changelog for 11.5.0-rc.1 Bump plugin version to 11.5.0-rc.1 Gallery block: Fix media placeholder height in site editor (#34629) Border Controls: Display color indicator and check selected color (#34467) Remove horizontal and vertical navigation block variations from inserter (#34614) AlignmentMatrixControl : Fix/update docs (#34624) Gap block support: force gap change to cause the block to re-render (fix Safari issue) (#34567) [Block Library - Social Links]: Use the new `flex` layout (#34493) [Mobile] Update the bottom sheet header (#34309) Group block: Add a row variation (#34535) Migrate entities.js to thunks (#34582) ...
2 parents 12f0b18 + 126eb8a commit f99c357

File tree

701 files changed

+10556
-6889
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

701 files changed

+10556
-6889
lines changed

.eslintrc.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ module.exports = {
104104
{
105105
name: '@emotion/css',
106106
message:
107-
'Please use `@emotion/react` and `@emotion/styled` in order to maintain iframe support',
107+
'Please use `@emotion/react` and `@emotion/styled` in order to maintain iframe support. As a replacement for the `cx` function, please use the `useCx` hook defined in `@wordpress/components` instead.',
108108
},
109109
],
110110
},

.github/CODEOWNERS

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,9 +40,11 @@
4040

4141
# Widgets
4242
/packages/edit-widgets @draganescu @talldan @noisysocks @tellthemachines @adamziel @kevin940726
43+
/packages/customize-widgets @noisysocks
44+
/packages/widgets @noisysocks
4345

4446
# Navigation
45-
/packages/edit-navigation @draganescu @talldan @noisysocks @tellthemachines @adamziel @kevin940726 @getdave
47+
/packages/edit-navigation @draganescu @talldan @tellthemachines @adamziel @kevin940726 @getdave
4648

4749
# Full Site Editing
4850
/packages/edit-site
@@ -125,6 +127,7 @@
125127
/lib/theme.json @timothybjabocs @spacedmonkey @nosolosw
126128
/lib/class-wp-theme-json-gutenberg.php @timothybjabocs @spacedmonkey @nosolosw
127129
/lib/class-wp-theme-json-resolver-gutenberg.php @timothybjabocs @spacedmonkey @nosolosw
130+
/lib/full-site-editing @janw-me
128131
/phpunit/class-wp-theme-json-test.php @nosolosw
129132

130133
# Web App

.github/ISSUE_TEMPLATE/Bug_report.yml

Lines changed: 20 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
name: Bug report
22
description: Report a bug with the WordPress block editor or Gutenberg plugin
3-
title: '<title>'
43
body:
54
- type: markdown
65
attributes:
@@ -49,10 +48,25 @@ body:
4948
validations:
5049
required: false
5150

52-
- type: checkboxes
51+
- type: dropdown
52+
id: existing
5353
attributes:
54-
label: Pre-checks
55-
description: Please check if the bug has already been reported by searching https://github.com/WordPress/gutenberg/issues and make sure the bug is not related to another plugin.
54+
label: Please confirm that you have searched existing issues in the repo.
55+
description: You can do this by searching https://github.com/WordPress/gutenberg/issues and making sure the bug is not related to another plugin.
56+
multiple: true
5657
options:
57-
- label: I have searched the existing issues.
58-
- label: I have tested with all plugins deactivated except Gutenberg.
58+
- 'Yes'
59+
- 'No'
60+
validations:
61+
required: true
62+
63+
- type: dropdown
64+
id: plugins
65+
attributes:
66+
label: Please confirm that you have tested with all plugins deactivated except Gutenberg.
67+
multiple: true
68+
options:
69+
- 'Yes'
70+
- 'No'
71+
validations:
72+
required: true

.github/ISSUE_TEMPLATE/Bug_report_mobile.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ the bug.
4444
-->
4545

4646
## WordPress information
47-
- WordPress version: <!-- e.g. "5.6.0". Find this in Tools → Site Health → Info → WordPress -->
47+
- WordPress version: <!-- e.g. "5.8.0". Find this in Tools → Site Health → Info → WordPress -->
4848
- Gutenberg version: <!-- e.g. "9.4.0" or "Not installed" -->
4949
- Are all plugins except Gutenberg deactivated? <!-- "Yes" or "No" -->
5050
- Are you using a default theme (e.g. Twenty Twenty-One)? <!-- "Yes" or "No" -->

.github/workflows/rnmobile-android-runner.yml

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,7 @@ concurrency:
1515
jobs:
1616
test:
1717
runs-on: macos-latest
18-
# The false value below disables the test while we investigate a
19-
# foundational error causing failures
20-
if: ${{ false && (github.repository == 'WordPress/gutenberg' || github.event_name == 'pull_request') }}
18+
if: ${{ github.repository == 'WordPress/gutenberg' || github.event_name == 'pull_request' }}
2119
strategy:
2220
matrix:
2321
native-test-name: [gutenberg-editor-initial-html]
@@ -41,9 +39,10 @@ jobs:
4139
path: ~/.gradle/caches
4240
key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }}
4341

44-
- uses: reactivecircus/android-emulator-runner@d2799957d660add41c61a5103e2fbb9e2889eb73 # v2.15.0
42+
- uses: reactivecircus/android-emulator-runner@5de26e4bd23bf523e8a4b7f077df8bfb8e52b50e # v2.19.1
4543
with:
4644
api-level: 28
45+
emulator-build: 7425822 # https://git.io/JE3jX
4746
profile: pixel_xl
4847
script: npm run native test:e2e:android:local ${{ matrix.native-test-name }}
4948

.github/workflows/upload-release-to-plugin-repo.yml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,8 +79,14 @@ jobs:
7979
- name: Commit the Changelog update
8080
run: |
8181
git add changelog.txt
82-
git commit -m "Update Changelog for ${TAG#v}"
83-
git push --set-upstream origin "${{ matrix.branch }}"
82+
# Remove files that are not meant to be commited
83+
# ie. release_notes.txt created on the previous step.
84+
git clean -fd
85+
# Only attempt to commit changelog if it has been modified.
86+
if ! git diff-index --quiet HEAD --; then
87+
git commit -m "Update Changelog for ${TAG#v}"
88+
git push --set-upstream origin "${{ matrix.branch }}"
89+
fi
8490
8591
- name: Upload Changelog artifact
8692
uses: actions/upload-artifact@e448a9b857ee2131e752b06002bf0e093c65e571 # v2.2.2

.vscode/launch-example.json

Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,23 @@
99
"pathMappings": {
1010
"/var/www/html/wp-content/plugins/gutenberg": "${workspaceRoot}/"
1111
}
12+
},
13+
{
14+
"type": "node",
15+
"request": "launch",
16+
"name": "Debug current e2e test",
17+
"program": "${workspaceRoot}/node_modules/@wordpress/scripts/bin/wp-scripts.js",
18+
"args": [
19+
"test-e2e",
20+
"--config=${workspaceRoot}/packages/e2e-tests/jest.config.js",
21+
"--verbose=true",
22+
"--runInBand",
23+
"--watch",
24+
"${file}"
25+
],
26+
"console": "integratedTerminal",
27+
"internalConsoleOptions": "neverOpen",
28+
"trace": "all"
1229
}
1330
]
14-
}
31+
}

bin/packages/build.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -263,7 +263,8 @@ stream
263263
console.error( error );
264264
}
265265

266-
if ( ended && ++complete === files.length ) {
266+
++complete;
267+
if ( ended && complete === files.length ) {
267268
workerFarm.end( worker );
268269
}
269270
} )

bin/plugin/commands/changelog.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -780,7 +780,9 @@ async function createChangelog( settings ) {
780780
try {
781781
changelog = await getChangelog( settings );
782782
} catch ( error ) {
783-
changelog = formats.error( error.stack );
783+
if ( error instanceof Error ) {
784+
changelog = formats.error( error.stack );
785+
}
784786
}
785787

786788
log( changelog );

0 commit comments

Comments
 (0)