wecobble/Subtitles

Move Permalink / view post to below subtitle

bryceadams opened this issue · 13 comments

Interested on your thoughts on this!

I think visually this is a lot cleaner and doesn't have any impact on UX.

screen shot 2014-09-17 at 8 06 12 am

I don't think there's a way to do this. Unless I'm missing something I can only either hook into edit_form_top or edit_form_after_title.

I haven't checked this yet either but will take a look tonight. If not, could make for a nice little patch to WP core. Will get back to you tonight :)

So as you said it can't be done at the moment. I honestly think that edit_form_after_title is in the wrong position and should actually be after the title field itself but I doubt this will change. I've opened up a trac ticket for people to discuss it in.

I think adding a new hook in between .titlewrap and .inside would be nice - I've never made a core patch before so hopefully it's something I could do :)

I've opened up a trac ticket for people to discuss it in.

Can you link to the ticket?

I've never made a core patch before so hopefully it's something I could do.

Sweet!

Yep, 2 minutes. Just adding a patch as well ☕

Here is the ticket - https://core.trac.wordpress.org/ticket/29691#ticket

Making the patch now. I'm proposing to add the following in between those two divs:

<?php
/**
 * Fires before the permalink field
 *
 * @since 4.0.1
 *
 * @param WP_Post $post Post object.
 */
do_action( 'edit_form_before_permalink', $post );
?>

Made a patch - https://core.trac.wordpress.org/ticket/29691

I hope I did it right! 🎉

Waiting on Core to do anything here. Should be a very simple fix if your patch lands into WordPress.

Fingers crossed. Not a big issue but it will make Subtitles look a lot better :)

Looks like it will be in WordPress 4.1 :) When the time comes I'll create a branch and PR for this.

Fantastic.
On Sep 18, 2014 4:43 PM, "Bryce Adams" notifications@github.com wrote:

Looks like it will be in WordPress 4.1 :) When the time comes I'll create
a branch and PR for this.


Reply to this email directly or view it on GitHub
#30 (comment)
.

@bryceadams Thanks for bringing this up and submitting that Core patch. This will be ready when WordPress 4.1 lands.

@philiparthurmoore thanks for pushing me make my first core contribution :)