Matmarex has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/77639


Change subject: Actually fix bug 47793 for Opera for real this time
......................................................................

Actually fix bug 47793 for Opera for real this time

I trusted you guys when you told me to use DOM .focus() instead of
jQuery's .focus() and didn't test well enough :( The former doesn't
work, the latter does.

Follow-up to I7962f59b.

Bug: 47793
Change-Id: Iddfb8d7c99325b6c7a5d151948b57cfa5f0a6a62
---
M modules/ve/ce/ve.ce.Surface.js
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor 
refs/changes/39/77639/1

diff --git a/modules/ve/ce/ve.ce.Surface.js b/modules/ve/ce/ve.ce.Surface.js
index 113b257..3c554f4 100644
--- a/modules/ve/ce/ve.ce.Surface.js
+++ b/modules/ve/ce/ve.ce.Surface.js
@@ -60,7 +60,7 @@
        $documentNode.on( 'focus', 'a', function () {
                // Opera triggers 'blur' on document node before any link is
                // focused and we don't want that
-               $documentNode[0].focus();
+               $documentNode.focus();
        } );
 
        this.$.on( {

-- 
To view, visit https://gerrit.wikimedia.org/r/77639
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iddfb8d7c99325b6c7a5d151948b57cfa5f0a6a62
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Matmarex <matma....@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to