Skip to content
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.

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

Remove the block-based widgets editor from the customizer. #25626

Merged
merged 2 commits into from
Sep 28, 2020

Conversation

adamziel
Copy link
Contributor

@adamziel adamziel commented Sep 24, 2020

Solves #25625 - let's keep the discussion in that issue.

Removes the block-based widgets editor from the customizer and instead relies on the legacy editor featuring block-based widgets:

Zrzut ekranu 2020-09-24 o 17 03 40

@adamziel adamziel added [Feature] Widgets Screen The block-based screen that replaced widgets.php. [Block] Legacy Widget Affects the Legacy Widget Block - used for displaying Classic Widgets [Package] Edit Widgets /packages/edit-widgets labels Sep 24, 2020
@adamziel adamziel force-pushed the update/remove-widgets-editor-from-customizer branch from 72b9967 to 73ee809 Compare September 24, 2020 15:03
@github-actions
Copy link

github-actions bot commented Sep 24, 2020

Size Change: -325 B (0%)

Total Size: 1.17 MB

Filename Size Change
build/block-directory/index.js 8.53 kB +2 B (0%)
build/block-editor/index.js 128 kB -4 B (0%)
build/block-library/editor-rtl.css 8.58 kB -17 B (0%)
build/block-library/editor.css 8.58 kB -17 B (0%)
build/block-library/index.js 135 kB -296 B (0%)
build/blocks/index.js 47.5 kB -2 B (0%)
build/components/index.js 167 kB +2 B (0%)
build/components/style-rtl.css 15.5 kB +7 B (0%)
build/components/style.css 15.5 kB +8 B (0%)
build/core-data/index.js 12 kB -1 B
build/data-controls/index.js 1.27 kB +2 B (0%)
build/data/index.js 8.43 kB +5 B (0%)
build/edit-navigation/index.js 10.7 kB +286 B (2%)
build/edit-site/index.js 20.5 kB +178 B (0%)
build/edit-site/style-rtl.css 3.54 kB +33 B (0%)
build/edit-site/style.css 3.54 kB +33 B (0%)
build/edit-widgets/index.js 17.3 kB -266 B (1%)
build/edit-widgets/style-rtl.css 2.7 kB -96 B (3%)
build/edit-widgets/style.css 2.7 kB -95 B (3%)
build/editor/index.js 45.4 kB -68 B (0%)
build/element/index.js 4.44 kB -8 B (0%)
build/rich-text/index.js 13.7 kB -11 B (0%)
ℹ️ View Unchanged
Filename Size Change
build/a11y/index.js 1.14 kB 0 B
build/annotations/index.js 3.52 kB 0 B
build/api-fetch/index.js 3.35 kB 0 B
build/autop/index.js 2.72 kB 0 B
build/blob/index.js 620 B 0 B
build/block-directory/style-rtl.css 943 B 0 B
build/block-directory/style.css 942 B 0 B
build/block-editor/style-rtl.css 11.1 kB 0 B
build/block-editor/style.css 11.1 kB 0 B
build/block-library/style-rtl.css 7.61 kB 0 B
build/block-library/style.css 7.6 kB 0 B
build/block-library/theme-rtl.css 741 B 0 B
build/block-library/theme.css 741 B 0 B
build/block-serialization-default-parser/index.js 1.78 kB 0 B
build/block-serialization-spec-parser/index.js 3.1 kB 0 B
build/compose/index.js 9.42 kB 0 B
build/date/index.js 31.9 kB 0 B
build/deprecated/index.js 772 B 0 B
build/dom-ready/index.js 568 B 0 B
build/dom/index.js 4.42 kB 0 B
build/edit-navigation/style-rtl.css 868 B 0 B
build/edit-navigation/style.css 871 B 0 B
build/edit-post/index.js 306 kB 0 B
build/edit-post/style-rtl.css 6.25 kB 0 B
build/edit-post/style.css 6.24 kB 0 B
build/editor/editor-styles-rtl.css 492 B 0 B
build/editor/editor-styles.css 493 B 0 B
build/editor/style-rtl.css 3.83 kB 0 B
build/editor/style.css 3.82 kB 0 B
build/escape-html/index.js 733 B 0 B
build/format-library/index.js 7.49 kB 0 B
build/format-library/style-rtl.css 547 B 0 B
build/format-library/style.css 548 B 0 B
build/hooks/index.js 1.74 kB 0 B
build/html-entities/index.js 621 B 0 B
build/i18n/index.js 3.55 kB 0 B
build/is-shallow-equal/index.js 709 B 0 B
build/keyboard-shortcuts/index.js 2.39 kB 0 B
build/keycodes/index.js 1.85 kB 0 B
build/list-reusable-blocks/index.js 3.02 kB 0 B
build/list-reusable-blocks/style-rtl.css 476 B 0 B
build/list-reusable-blocks/style.css 476 B 0 B
build/media-utils/index.js 5.12 kB 0 B
build/notices/index.js 1.69 kB 0 B
build/nux/index.js 3.27 kB 0 B
build/nux/style-rtl.css 671 B 0 B
build/nux/style.css 668 B 0 B
build/plugins/index.js 2.44 kB 0 B
build/primitives/index.js 1.34 kB 0 B
build/priority-queue/index.js 789 B 0 B
build/redux-routine/index.js 2.85 kB 0 B
build/server-side-render/index.js 2.61 kB 0 B
build/shortcode/index.js 1.7 kB 0 B
build/token-list/index.js 1.24 kB 0 B
build/url/index.js 4.06 kB 0 B
build/viewport/index.js 1.74 kB 0 B
build/warning/index.js 1.13 kB 0 B
build/wordcount/index.js 1.17 kB 0 B

compressed-size-action

@adamziel adamziel changed the title Remove the modern widget editor from the customizer. Remove the block-based widgets editor from the customizer. Sep 25, 2020
Copy link
Contributor

@draganescu draganescu left a comment

Choose a reason for hiding this comment

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

I will approve this since the current state of the customizer in the Gutenberg plugin is broken and there is no clear path forward about how to fix that. With this patch, the normal widgets can still be edited in the customizer and the block ones don't break it anymore.

This is NOT to mean that we won't proceed with fixing the block editor in the customizer, that is still an ongoing discussion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Legacy Widget Affects the Legacy Widget Block - used for displaying Classic Widgets [Feature] Widgets Screen The block-based screen that replaced widgets.php. [Package] Edit Widgets /packages/edit-widgets
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants