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

WordPress Trac noreply at wordpress.org
Sat Sep 23 16:35:22 UTC 2017


#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                        |  administration
-------------------------------------+-------------------------------------

Comment (by westonruter):

 Replying to [comment:16 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.

 Oh thank you for identifying this!! I had opened #41900 for this issue.
 I'll follow up there with working to remove this.

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


More information about the wp-trac mailing list