Make WordPress Core

Opened 2 years ago

Closed 7 weeks ago

#26267 closed enhancement (fixed)

Do action before #minor-publishing-actions in publish meta-box

Reported by: jonathanbardo Owned by: chriscct7
Milestone: 4.4 Priority: normal
Severity: normal Version:
Component: Editor Keywords: has-patch commit
Focuses: administration Cc:


I've stumble on a use case where we would have to create a custom preview button on the publish meta-box.

There is currently no hook to add custom content at the end of the #minor-publishing-actions div.

I propose adding a hook right before the division is closed.

Attachments (4)

26267.diff (447 bytes) - added by jonathanbardo 2 years ago.
26267.2.diff (471 bytes) - added by jonathanbardo 21 months ago.
Change name of action
26267.patch (590 bytes) - added by chriscct7 8 weeks ago.
26267.2.patch (1022 bytes) - added by DrewAPicture 7 weeks ago.
+ $post parameter

Download all attachments as: .zip

Change History (17)

2 years ago

#1 @helen
21 months ago

  • Keywords 2nd-opinion added

Adding a hook here is making me nervous for some reason, even though I usually love hooks on the edit screen. The publish metabox is already so cluttered and maintaining compat with hooks can make rethinking difficult. Would like some more opinions here.

#2 @mordauk
21 months ago

I would prefer having the option that might result in a little clutter due to plugins than to not have the option.

#3 follow-up: @helen
21 months ago

I think I'm more worried about maintaining compat into the future :)

#4 in reply to: ↑ 3 @mordauk
21 months ago

Replying to helen:

I think I'm more worried about maintaining compat into the future :)

Maybe I'm just being daft but I'm having trouble thinking of compatibility issues this might raise in the future. Could you give me an example?

#5 @jonathanbardo
21 months ago

I'm also having problem with foreseeing compatibilities issues. Right now the only way we can add a custom button to the publish meta-box is at the end. Having a hook here would make it more flexible.

#6 @SergeyBiryukov
21 months ago

  • Component changed from General to Editor

#7 @SergeyBiryukov
21 months ago

Related: #25279, #12567.

If we add a hook in there, it should probably be named post_submitbox_minor_actions for consistency with post_submitbox_misc_actions added in [12000], and go above <div class="clear"></div>.

21 months ago

Change name of action

#8 @chriscct7
8 weeks ago

  • Focuses administration added
  • Keywords has-patch added; 2nd-opinion removed
  • Milestone changed from Awaiting Review to 4.4
  • Owner set to chriscct7
  • Status changed from new to accepted

#9 @chriscct7
8 weeks ago

  • Keywords needs-docs added

Just needs docs

8 weeks ago

#10 @chriscct7
8 weeks ago

  • Keywords commit added; needs-docs removed

#11 @wonderboymusic
7 weeks ago

  • Keywords needs-refresh added; commit removed

both of those actions should have $post passed to them to not have to rely on globals, which are gross

7 weeks ago

+ $post parameter

#12 @DrewAPicture
7 weeks ago

  • Keywords commit added; needs-refresh removed

26267.2.patch adds a $post parameter to both the new hook and the post_submitbox_misc_actions hook (with a changelog entry).

#13 @wonderboymusic
7 weeks ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 34895:

Meta Boxes: add an action, post_submitbox_minor_actions. post_submitbox_misc_actions already exists. Pass $post to both for context - avoids having to check for a global post.

Props jonathanbardo, chriscct7, DrewAPicture.
Fixes #26267.

Note: See TracTickets for help on using tickets.