Skip to content

Merge conflicting wp.editor objects into a single, non-conflicting module #33228

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
youknowriad merged 13 commits into from
Jul 9, 2021
Merged

Merge conflicting wp.editor objects into a single, non-conflicting module #33228

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
01daa00
Export members of legacy window.wp.editor from the new window.wp.edit…
adamziel Jul 6, 2021
b644a7d
Add an explanatory comment
adamziel Jul 6, 2021
d880b51
Use optional chaining for editor/index.js exports
adamziel Jul 6, 2021
8d4a984
Default to an empty object on export from oldEditor
adamziel Jul 6, 2021
ae66155
Use an inline script instead of modifying the @wordpress/editor package
adamziel Jul 7, 2021
ed8f3a8
Lint
adamziel Jul 7, 2021
f4601f7
Move the inline script to gutenberg_override_script
adamziel Jul 8, 2021
723a9c8
Remove gutenberg_enqueue_wp_old_editor_compatibility_snippet
adamziel Jul 8, 2021
59db57c
Formatting
adamziel Jul 8, 2021
ae4b7ac
Adjust wp-editor-meta-box E2E test
adamziel Jul 8, 2021
48c1929
Only add inline script if the handle is wp-editor
adamziel Jul 8, 2021
7277ce9
Add explanatory comment for the e2e test
adamziel Jul 9, 2021
c714c89
Add even more context to that comment
adamziel Jul 9, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Default to an empty object on export from oldEditor
  • Loading branch information
@adamziel
adamziel committed Jul 8, 2021
commit 8d4a984cac919c5241136b941b11a436284ec9f0
9 changes: 2 additions & 7 deletions packages/editor/src/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,13 +11,8 @@ export * from './utils';
// Problem: there is quite some code expecting another, legacy object available under window.wp.editor
// Solution: export all the members of legacy window.wp.editor from this new module to maintain backward compatibility
// For more context, see https://github.com/WordPress/gutenberg/issues/33203
export const {
autop,
removep,
initialize,
remove,
getContent,
} = window?.wp?.oldEditor;
export const { autop, removep, initialize, remove, getContent } =
Copy link
Member

@kevin940726 kevin940726 Jul 7, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't even know this is valid JS 🤯

window?.wp?.oldEditor || {};

/*
* Backward compatibility
Expand Down