[wp-trac] [WordPress Trac] #31575: Press This: post options toggle accessibility improvements

[WordPress Trac]

#31575: Press This: post options toggle accessibility improvements
------------------------------+--------------------------------------------
 Reporter:  afercia           |       Owner:
     Type:  defect (bug)      |      Status:  new
 Priority:  normal            |   Milestone:  Awaiting Review
Component:  Press This        |     Version:  trunk
 Severity:  normal            |  Resolution:
 Keywords:  has-patch dev-    |     Focuses:  ui, accessibility, javascript
  feedback                    |
------------------------------+--------------------------------------------

Comment (by afercia):

 Thanks very much @azaozz :) tested with Firefox + NVDA and Chrome +
 ChromeVox the button works nicely. Just one thing: setting `aria-expanded`
 on a non-focusable element makes this information not available for screen
 readers, they just can't focus that div:
 {{{
 <div class="options-panel" tabindex="-1" aria-expanded="true">
 }}}
 Thinking a bit about this, I'd say either change this approach and do the
 `aria-expanded` thing on the button or further simplify and remove the
 `aria-expanded` thing. After all, the button status is implicitly conveyed
 by its text because it will be announced as "Show post options" or "Hide
 post options". I would be in favor of the latter option, but I'd like to
 have a feedback from the accessibility team.

 @michaelarestad: sorry for bothering :) maybe the Add category button
 clickable area needs some love:
 [[Image(https://cldup.com/eqQb0BQsw3.png)]]

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