[wp-trac] [WordPress Trac] #29821: bulk_edit_custom_box hook causes tags to display within created fieldset
WordPress Trac
noreply at wordpress.org
Sat Feb 4 19:47:06 UTC 2023
#29821: bulk_edit_custom_box hook causes tags to display within created fieldset
---------------------------------------------+-----------------------------
Reporter: webgeekconsulting | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting
| Review
Component: Quick/Bulk Edit | Version: 4.0
Severity: normal | Resolution:
Keywords: needs-testing reporter-feedback | Focuses:
---------------------------------------------+-----------------------------
Comment (by oglekler):
The action 'bulk_edit_custom_box' is working only if table has additional
custom column,
see: [https://github.com/WordPress/WordPress/blob/master/wp-admin/includes
/class-wp-posts-list-table.php#L1999 wp-admin/includes/class-wp-posts-
list-table.php#L1999]
But this inserting is happening on the admin frontend,
see: [https://github.com/WordPress/WordPress/blob/master/wp-admin/js
/inline-edit-post.js#L131 wp-admin/js/inline-edit-post.js#L131]
{{{
$('#bulk-edit').find('fieldset:first').after(
$('#inline-edit fieldset.inline-edit-
categories').clone()
).siblings( 'fieldset:last' ).prepend(
$( '#inline-edit .inline-edit-tags-wrap' ).clone()
);
}}}
This implementation with insertion after first or to the beginning of the
last element look very uncertain. To make it clear, there should be exact
places to add elements, but also it is possible that we are lucking
developers documentation on the subject. I wasn't able to find it on the
WordPress.org. I found only a post "Custom Fields in Bulk Edit" in
personal blog of Misha Rudrastyh.
So, I am suggesting to rework inserting elements into the Bulk Editor and
check if we need in-house documentation on the subject.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/29821#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list