Add classes to .post-attributes-label-wrapper paragraphs to make them unique

[garrett-eclipse]

Hello,

Currently the Post Attributes meta box markup for the individual post attributes doesn't allow for easy targetting for either css or js.

<div class="inside">
<p class="post-attributes-label-wrapper"><label class="post-attributes-label" for="parent_id">Parent</label></p>
<select name="parent_id" id="parent_id"></select>
<p class="post-attributes-label-wrapper"><label class="post-attributes-label" for="page_template">Template</label></p>
<select name="page_template" id="page_template"></select>
<p class="post-attributes-label-wrapper"><label class="post-attributes-label" for="menu_order">Order</label></p>
<input name="menu_order" type="text" size="4" id="menu_order" value="0">
<p>Need help? Use the Help tab above the screen title.</p>
</div>

Could each attribute either be wrapped in a div with ID or class on it. Otherwise can an ID or unique class be added to the post-attributes-label-wrapper paragraphs in order to target them.

Personally I prefer an entire div wrapping to allow better targetting the attribute as a whole to hide or style it or update it.

Thank you

[noisysocks]

Makes sense! I think this would be a good simple enhancement.

[sathyapulse]

44408.patch

[sathyapulse]

@noisysocks As we discussed in the APAC meeting, I have attached the patch to the ticket. I have additionally added class to "Help text" as well.

CC: @garrett-eclipse

[sathyapulse]

44408.1.patch

[sathyapulse]

44408.2.patch

[sathyapulse]

44408.3.patch

[sathyapulse]

@garrett-eclipse I have generated the patch outside of src directory. Please use the latest 44408.3.patch file.

[sabernhardt]

To avoid any chance of duplicating an ID used in a plugin or theme, I would recommend using classes instead (unless these paragraph tags do need an ID attribute, for example, if a link might direct to them).

<p class="post-attributes-label-wrapper parent-id-label-wrapper">

[sathyapulse]

44408.4.patch

[sabernhardt]

Thanks @sathyapulse! I tested 44408.3.patch and 44408.4.patch, and both patches applied the classes and/or IDs as I expected. (Though I prefer # 4, I figured I ought to check both ways.)

[sabernhardt]

@garrett-eclipse Do you see any benefit to using an ID instead of a class for these? I checked wpdirectory.net and did not find any of these class/ID names in any of those plugins or themes. (There's still a slight possibility one is used in a paid or custom plugin/theme.)

[garrett-eclipse]

Thanks for testing @sabernhardt and the patches @sathyapulse let's move forward with classes (Patch 44408.4.patch as there's no real benefit with IDs and there's the slight possibility of duplicated IDs so classes in this case are safer. Marking for commit.

[SergeyBiryukov]

In 47228:

Editor: Add unique classes to .post-attributes-label-wrapper paragraphs for easier JS or CSS targeting.

Props sathyapulse, sabernhardt, garrett-eclipse, noisysocks.
Fixes #44408.