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

Move duplicateTemplatePart action to the @wordpress/fields package #65390

Open
wants to merge 41 commits into
base: trunk
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 25 commits
Commits
Show all changes
41 commits
Select commit Hold shift + click to select a range
67ff043
Migrate store and actions from editor package to fields package
gigitux Sep 12, 2024
a71e726
fix documentation
gigitux Sep 12, 2024
0141d8c
fix import
gigitux Sep 12, 2024
f21a974
remove duplicate code
gigitux Sep 12, 2024
7063ee8
Merge branch 'trunk' of github.com:WordPress/gutenberg into add/actio…
gigitux Sep 12, 2024
644dab1
update package-lock
gigitux Sep 12, 2024
3616964
refactor PR
gigitux Sep 12, 2024
f7f1d4d
fix mobile build
gigitux Sep 12, 2024
ae0efeb
update package.json
gigitux Sep 12, 2024
4356a42
migrate actions to @wordpress/fields package
gigitux Sep 12, 2024
bf1462a
Merge branch 'trunk' of github.com:WordPress/gutenberg into add/actio…
gigitux Sep 12, 2024
63adf35
reduce change
gigitux Sep 13, 2024
c938a15
register new action
gigitux Sep 13, 2024
66479da
improve code
gigitux Sep 13, 2024
0bf760d
fix actionId
gigitux Sep 13, 2024
08bb03d
update package-lock.json
gigitux Sep 13, 2024
b599c42
remove comment
gigitux Sep 13, 2024
337b3e5
rename function
gigitux Sep 13, 2024
1016ef8
rename functions
gigitux Sep 13, 2024
39d25d1
improve code
gigitux Sep 13, 2024
dc03f57
Merge branch 'trunk' of github.com:WordPress/gutenberg into add/actio…
gigitux Sep 16, 2024
599c0a4
Migrate duplicate-template-part action to @wordpress/fields package
gigitux Sep 17, 2024
60e7a58
Merge branch 'trunk' of github.com:WordPress/gutenberg into add/actio…
gigitux Sep 17, 2024
81c0980
fix build
gigitux Sep 17, 2024
7b2bb82
restore files
gigitux Sep 17, 2024
d6fcf4a
fix CSS
gigitux Sep 17, 2024
0316536
restore delete post action
gigitux Sep 18, 2024
485237f
fix import order
gigitux Sep 18, 2024
5a4fb59
move to a flat structure
gigitux Sep 18, 2024
0deeed3
remove not necessary file
gigitux Sep 18, 2024
d42430d
Merge branch 'add/action-dataviews-2' of github.com:gigitux/gutenberg…
gigitux Sep 19, 2024
400a67d
Merge branch 'trunk' of github.com:WordPress/gutenberg into add/actio…
gigitux Sep 25, 2024
18d6029
Merge branch 'trunk' of github.com:WordPress/gutenberg into add/actio…
gigitux Oct 1, 2024
ff299b6
Merge branch 'trunk' of github.com:WordPress/gutenberg into add/actio…
gigitux Oct 24, 2024
36adb07
regenerate package-lock.json file
gigitux Oct 24, 2024
47da440
generate documentation
gigitux Oct 24, 2024
64423cf
delete constant file
gigitux Oct 24, 2024
199c313
update class
gigitux Oct 25, 2024
8571360
rename styles.scss to style.scss
gigitux Oct 25, 2024
844a98a
Merge branch 'trunk' of github.com:WordPress/gutenberg into add/actio…
gigitux Oct 28, 2024
0bd24a7
Merge branch 'trunk' of github.com:WordPress/gutenberg into add/actio…
gigitux Nov 1, 2024
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
2 changes: 2 additions & 0 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

9 changes: 8 additions & 1 deletion packages/edit-site/src/components/editor/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,14 @@ export default function EditSiteEditor( { isPostsList = false } ) {
( actionId, items ) => {
switch ( actionId ) {
case 'move-to-trash':
case 'delete-post':
case 'delete-template':
{
history.push( {
postType: items[ 0 ].type,
} );
}
break;
case 'delete-pattern':
{
history.push( {
postType: items[ 0 ].type,
Expand Down
1 change: 1 addition & 0 deletions packages/edit-site/src/style.scss
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
@import "../../dataviews/src/style.scss";
@import "../../fields/src/style.scss";

@import "./components/add-new-template/style.scss";
@import "./components/block-editor/style.scss";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ import { symbolFilled } from '@wordpress/icons';
/**
* Internal dependencies
*/
import CreateTemplatePartModal from '../create-template-part-modal';
import { CreateTemplatePartModal } from '@wordpress/fields';
Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, I agree this is weird, but also not sure what's the best place for these wp-core-data aware components. That said I can live with it being in fields for now.

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we should add a comment explaining the choice.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

With 47da440 I added some documentation.


export default function ConvertToTemplatePart( { clientIds, blocks } ) {
const [ isModalOpen, setIsModalOpen ] = useState( false );
Expand Down
109 changes: 0 additions & 109 deletions packages/editor/src/dataviews/actions/delete-post.tsx

This file was deleted.

147 changes: 0 additions & 147 deletions packages/editor/src/dataviews/actions/reset-post.tsx

This file was deleted.

64 changes: 0 additions & 64 deletions packages/editor/src/dataviews/actions/utils.ts

This file was deleted.

26 changes: 0 additions & 26 deletions packages/editor/src/dataviews/fields/index.ts

This file was deleted.

Loading
Loading