Remove misplaced label from submitdiv and remove some obsolete css

[iseulde]

There's a label wrapped around "Status:" in the post submit box, which is clickable, but doesn't do anything.
At the same time there are also some CSS rules in forms.css for the icon before it that are a duplicate of the styles found in edit.css for the other icons.

[iseulde]

left: -1px; also doesn't seem to make it look any better at all. top: -1px; for this one icon neither.
There's also no reason to remove the text-decoration, because there is none.

[SergeyBiryukov]

There's a label wrapped around "Status:" in the post submit box, which is clickable, but doesn't do anything.

It actually does focus the dropdown if you click "Edit" first. I'm not sure if it's necessary though.

[iseulde]

Replying to SergeyBiryukov:

Yes, but then it should (1) open the hidden area and (2) work the same way for all sections.
I'd actually prefer that above having an "edit" link next to every section.

[helen]

Probably a good idea to reveal the area when you click the label. Doesn't seem like a good idea to remove the label element - semantics.

[iseulde]

Replying to helen:

Probably a good idea to reveal the area when you click the label. Doesn't seem like a good idea to remove the label element - semantics.

Okay, then labels should be added to each line there.

[DrewAPicture]

Punting per IRC. Sounds like clicking the labels should be made to be more interactive anyway. Patch still applies.

[chriscct7]

Needs a new patch to address Helen's comment as well

[afercia]

I'd agree the "Status" visible text shouldn't be a label, for consistency with other elements in this box. The status select should have a label though and I'd propose to add a hidden one, as already done for the timestamp select.

The "Publish box" probably deserves some major improvements and re-thinking. It would be nice to get rid of the small "Edit" links, which are a bit tricky for accessibility, and make each single "row" clickable. Maybe each "row" (Status, Visibility, etc.) could be marked up as a button that, when clicked, would reveal the related stuff. Or, to keep the "no-js/js" compatibility they could stay as links but have a role=button added via JavaScript. Additionally, aria-expanded attributes should be used to give assistive technologies users some feedback about what happens when the links/buttons get activated. The "Revisions" row is an exception, since it's a real link.
Worth considering also to simplify things, for example removing barely noticeable title attributes :) see screenshot below:

Basically many things can be experimented and tested here but for now I'd recommend to follow the approach of the patch by @iseulde, just adding a new hidden label. See the refreshed patch.

[afercia]

At the very least the label issue should be fixed, hopefully for 4.7 :)

[afercia]

Refreshed patch. Adds a role="button" attribute to the "Edit" links.

[afercia]

In 38700:

Accessibility: Editor, Publish meta box: remove a stray label and redundant CSS.

Also, adds a role="button" attribute to the "Edit" links.

Props iseulde.
Fixes #28411.