[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