Skip to content

Update some comments for drag-and-drop of navigation submenus#24708

Merged
talldan merged 3 commits intomasterfrom
update/navigation-block-submenu-dnd-reviews
Aug 26, 2020
Merged

Update some comments for drag-and-drop of navigation submenus#24708
talldan merged 3 commits intomasterfrom
update/navigation-block-submenu-dnd-reviews

Conversation

@kevin940726
Copy link
Member

Description

Continued from #24479 (review), update some comments in the original PR.

Also enable DnD inside submenus in navigation screen as suggested here: #24479 (comment).

Types of changes

New feature

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.
  • I've updated all React Native files affected by any refactorings/renamings in this PR.

@github-actions
Copy link

github-actions bot commented Aug 21, 2020

Size Change: +2.91 kB (0%)

Total Size: 1.16 MB

Filename Size Change
build/api-fetch/index.js 3.44 kB -3 B (0%)
build/block-directory/index.js 7.97 kB +5 B (0%)
build/block-editor/index.js 126 kB +136 B (0%)
build/block-editor/style-rtl.css 10.7 kB -2 B (0%)
build/block-editor/style.css 10.7 kB +2 B (0%)
build/block-library/editor-rtl.css 8.5 kB -2 B (0%)
build/block-library/editor.css 8.49 kB -3 B (0%)
build/block-library/index.js 134 kB +1.57 kB (1%)
build/block-library/style-rtl.css 7.45 kB +23 B (0%)
build/block-library/style.css 7.45 kB +21 B (0%)
build/block-serialization-default-parser/index.js 1.88 kB +1 B
build/blocks/index.js 47.7 kB +21 B (0%)
build/components/index.js 200 kB +15 B (0%)
build/components/style-rtl.css 15.7 kB -1 B
build/components/style.css 15.7 kB -1 B
build/compose/index.js 9.67 kB -3 B (0%)
build/core-data/index.js 12.3 kB +474 B (3%)
build/data/index.js 8.55 kB -1 B
build/edit-navigation/index.js 11.6 kB +585 B (5%) 🔍
build/edit-navigation/style-rtl.css 1.16 kB +45 B (3%)
build/edit-navigation/style.css 1.16 kB +46 B (3%)
build/edit-post/index.js 304 kB -33 B (0%)
build/edit-site/index.js 17 kB -53 B (0%)
build/edit-widgets/index.js 11.8 kB +86 B (0%)
build/editor/index.js 45.3 kB -2 B (0%)
build/element/index.js 4.65 kB -2 B (0%)
build/format-library/index.js 7.71 kB -4 B (0%)
build/list-reusable-blocks/index.js 3.12 kB +5 B (0%)
build/media-utils/index.js 5.32 kB -10 B (0%)
build/primitives/index.js 1.41 kB -7 B (0%)
build/rich-text/index.js 13.9 kB -2 B (0%)
build/token-list/index.js 1.27 kB -2 B (0%)
ℹ️ View Unchanged
Filename Size Change
build/a11y/index.js 1.14 kB 0 B
build/annotations/index.js 3.67 kB 0 B
build/autop/index.js 2.82 kB 0 B
build/blob/index.js 620 B 0 B
build/block-directory/style-rtl.css 953 B 0 B
build/block-directory/style.css 952 B 0 B
build/block-library/theme-rtl.css 729 B 0 B
build/block-library/theme.css 730 B 0 B
build/block-serialization-spec-parser/index.js 3.1 kB 0 B
build/data-controls/index.js 1.29 kB 0 B
build/date/index.js 5.38 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.47 kB 0 B
build/edit-post/style-rtl.css 5.61 kB 0 B
build/edit-post/style.css 5.61 kB 0 B
build/edit-site/style-rtl.css 3.06 kB 0 B
build/edit-site/style.css 3.06 kB 0 B
build/edit-widgets/style-rtl.css 2.45 kB 0 B
build/edit-widgets/style.css 2.45 kB 0 B
build/editor/editor-styles-rtl.css 537 B 0 B
build/editor/editor-styles.css 539 B 0 B
build/editor/style-rtl.css 3.8 kB 0 B
build/editor/style.css 3.79 kB 0 B
build/escape-html/index.js 733 B 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 2.13 kB 0 B
build/html-entities/index.js 621 B 0 B
build/i18n/index.js 3.56 kB 0 B
build/is-shallow-equal/index.js 711 B 0 B
build/keyboard-shortcuts/index.js 2.52 kB 0 B
build/keycodes/index.js 1.94 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/notices/index.js 1.79 kB 0 B
build/nux/index.js 3.4 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.56 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.77 kB 0 B
build/shortcode/index.js 1.7 kB 0 B
build/url/index.js 4.06 kB 0 B
build/viewport/index.js 1.85 kB 0 B
build/warning/index.js 1.14 kB 0 B
build/wordcount/index.js 1.17 kB 0 B

compressed-size-action

Copy link
Contributor

@talldan talldan left a comment

Choose a reason for hiding this comment

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

Thanks for addressing the comments ❤️

@talldan talldan merged commit 7ce5269 into master Aug 26, 2020
@talldan talldan deleted the update/navigation-block-submenu-dnd-reviews branch August 26, 2020 04:41
@github-actions github-actions bot added this to the Gutenberg 8.9 milestone Aug 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Block] Navigation Affects the Navigation Block

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants