[wp-trac] [WordPress Trac] #41872: Code Editor: Minor accessibility improvements to the CodeMirror editing areas

[WordPress Trac]

#41872: Code Editor: Minor accessibility improvements to the CodeMirror editing
areas
-------------------------------------+-------------------------------------
 Reporter:  afercia                  |       Owner:  afercia
     Type:  defect (bug)             |      Status:  assigned
 Priority:  high                     |   Milestone:  4.9
Component:  Customize                |     Version:  trunk
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch has-           |     Focuses:  ui, accessibility,
  screenshots needs-refresh          |  administration
-------------------------------------+-------------------------------------

Comment (by afercia):

 > When clicking on the label it should focus on the CodeMirror editor.

 Trying this on the Additional CSS, focus always goes to the last line (and
 if the content is very long, it's out of view). Seems to me because of:
 {{{
                         // Refresh when receiving focus.
                         control.editor.codemirror.on( 'focus', function(
 codemirror ) {
                                 codemirror.refresh();
                         });
 }}}


 Is this really necessary? And why it is used only for the Additional CSS
 and not for the other CodeMirror instances? If there are no particular
 reasons, I'd consider to remove it.

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