Improve the new labels for the post templates select

[afercia]

The new labels introduced in [38951] for the Post/Page/Attachment Attributes Meta box make perfectly sense when they're used for the Meta box heading. By the way, they shouldn't be used also for the hidden label associated with the "template" select:

The ones used for the select would need new, more meaningful text. Worth noting the previous one was "Page Template" because only pages had templates. Now, even a generic "Template" could probably be appropriate.

[afercia]

The whole meta box could probably be improved a bit, I don't see a good reason why the visible "titles" (parent, template, order) should then be repeated for the hidden labels. Probably something for 4.8 :)

[afercia]

Well, after all this is a nice opportunity to improve things a bit. I'd propose to:

• make the labels visible (and remove the existing visible "titles")
• use a generic string "Template"
• slightly adjust the CSS

Things to check:

• the position of the "Template" label in respect of the action hook
• escaping with esc_html_e()

Any feedback welcome.

[ocean90]

Replying to afercia:

• escaping with esc_html_e()

There is no need to use esc_html_e() for the labels.

[afercia]

Refreshed patch to use _e() instead of esc_html_e().

My only concern here is the position of the "Template" label in the source. From an accessibility perspective, there shouldn't be anything between the label and the select so it would be better to move it after the page_attributes_meta_box_template hook.

[swissspidy]

No objections to improving things here from my side. A before/after screenshot would be nice.

My only concern here is the position of the "Template" label in the source. From an accessibility perspective, there shouldn't be anything between the label and the select.

I wonder what use cases there are for the page_attributes_meta_box_template hook. Should it run before the label or rather after the input?

[afercia]

The hook was added in [34340] to add an "Edit link" for templates. Seems it was added for just a single use case... and in the middle of a not so ideal HTML. We have to deal with it now, will try to see what can be reasonably done here.

In the meanwhile, screenshot before and after:

[afercia]

Refreshed patch. Restoring the paragraphs to wrap the labels is not ideal but still valid HTML and would allow to use the hook without requiring any change. See screenshots below:

[afercia]

In 39247:

Accessibility: Improve the Post Attributes meta box fields labels.

With [38951] the support for custom page templates has been extended to all post
types. By making the visible meta box titles real labels, accessibility improves
for all users and form fields have meaningful labels.

Fixes #38790.