[wp-trac] [WordPress Trac] #28411: Remove misplaced label from submitdiv and remove some obsolete css

WordPress Trac noreply at wordpress.org
Sun Nov 1 23:12:10 UTC 2015


#28411: Remove misplaced label from submitdiv and remove some obsolete css
---------------------------------------+--------------------------------
 Reporter:  iseulde                    |       Owner:
     Type:  defect (bug)               |      Status:  new
 Priority:  normal                     |   Milestone:  Future Release
Component:  Editor                     |     Version:  4.3
 Severity:  normal                     |  Resolution:
 Keywords:  has-patch has-screenshots  |     Focuses:  ui, accessibility
---------------------------------------+--------------------------------
Changes (by afercia):

 * keywords:  needs-patch => has-patch has-screenshots
 * focuses:  ui => ui, accessibility
 * version:   => 4.3


Comment:

 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:

 [[Image(https://cldup.com/zTX-zzmJtR.png)]]

 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.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/28411#comment:11>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list