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

Block Editor Settings: use v2 DropdownMenu #51400

Closed
wants to merge 35 commits into from
Closed
Changes from 1 commit
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
e736ca8
Start using dropdownmenu v2 in block settings dropdown
ciampo Jun 12, 2023
6af23df
Add Shortcut component
ciampo Jun 12, 2023
bb71a23
Refactor block settings dropdown menu items to use dropdownmenu v2
ciampo Jun 12, 2023
d80301a
Add notes about duplicate implementations/classnames
ciampo Jun 12, 2023
53663cb
Convert more menu items to use the dropdownmenu v2 components
ciampo Jun 12, 2023
235a841
Convert menu items from the reusable-blocks package to use the dropdo…
ciampo Jun 12, 2023
b933e85
Add separator to BlockSettingsMenuControls
ciampo Jun 12, 2023
6f50219
Simplify Shortcut suffix component
ciampo Jun 12, 2023
d444e01
Refactor more menu items rendering inside `BlockSettingsMenuControls`
ciampo Jun 12, 2023
04220d4
Remove onKeyDown menu props (it doesn't seem to be needed?)
ciampo Jun 12, 2023
44a5cf3
Migrate from calling onClose to calling preventDefault() when needing…
ciampo Jun 12, 2023
dc78e0d
Fix broken copy/paste by using native clipboard APIs instead of exter…
ciampo Jun 13, 2023
6faf5fd
Remove unnecessary wrapper functions, unused imports, clean up code, …
ciampo Jun 13, 2023
d642c70
Add default z-index
ciampo Jun 13, 2023
e0075d5
Capture pointer events in modals
ciampo Jun 13, 2023
4a7a0ac
DropdownMenuV2: accept onKeyDown prop
ciampo Jun 13, 2023
40fd830
Restore onKeyDown ev listener on the block settings dropdown
ciampo Jun 13, 2023
a27962e
Add a few pointer-related props to DropdownMenuItemProps
ciampo Jun 13, 2023
e5a7b12
Destructure and properly allocate more props in Block settings menu
ciampo Jun 13, 2023
2c2ae3b
DropdownMenuItem cursor styles
ciampo Jun 13, 2023
f6c4937
Clean up classnames:
ciampo Jun 13, 2023
1141cab
Temp fix for e2e test
ciampo Jun 13, 2023
43204d2
Refactor block settings dropdown to controlled mode. This helps with:
ciampo Jun 14, 2023
0425a81
Support rendering anchor tags in DropdownMenuItem
ciampo Jun 14, 2023
e51fdcf
Remove done comment
ciampo Jun 14, 2023
7823fc2
Avoid unnecessary function
ciampo Jun 14, 2023
b4e7cc4
Set the correct value for aria-labelledby, update e2e tests
ciampo Jun 15, 2023
e6342a7
Fix unit test by wrapping the BlockModeToggle in dropdownmenu
ciampo Jun 15, 2023
5347105
Sync package-lock.json
ciampo Jun 15, 2023
d7dcf02
Cleanup code
ciampo Jun 15, 2023
3332023
Back compat styles for legacy menuitems
ciampo Jun 16, 2023
02a2e0a
Resize dropdown menu + show scrollbars when there's not enough space
ciampo Jun 16, 2023
1c5aef5
Forward ref on DropdownMenu component
ciampo Jun 16, 2023
b53783e
Allow specifying container on useCopyToClipboard hook
ciampo Jun 16, 2023
601a07e
Revert to using useCopyToCliboard hook
ciampo Jun 16, 2023
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
Remove done comment
  • Loading branch information
ciampo committed Jun 15, 2023
commit e51fdcf999cd6164d65198e33f15406ccdb69f4f
Original file line number Diff line number Diff line change
@@ -407,7 +407,6 @@ export function BlockSettingsDropdown( {
onInsertBefore();
}
} }
// Missing: pressing esc is not just closing the menu, but also the toolbar?
{ ...props }
>
<DropdownMenuGroupV2>