jenkins-bot has submitted this change and it was merged.

Change subject: Add a warning about T130390
......................................................................


Add a warning about T130390

Change-Id: I872d1dc0ec8d0827c3f56aeb1474f4596e08532d
---
M resources/js/ext.uls.interface.js
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  KartikMistry: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/resources/js/ext.uls.interface.js 
b/resources/js/ext.uls.interface.js
index a99cc0f..47b30eb 100644
--- a/resources/js/ext.uls.interface.js
+++ b/resources/js/ext.uls.interface.js
@@ -285,7 +285,8 @@
                                                        lang: previousLang,
                                                        // We could get dir 
from uls.data,
                                                        // but we are trying to 
avoid loading it
-                                                       // and 'auto' is safe 
enough in this context
+                                                       // and 'auto' is safe 
enough in this context.
+                                                       // T130390: must use 
attr
                                                        dir: 'auto'
                                                } );
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I872d1dc0ec8d0827c3f56aeb1474f4596e08532d
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/UniversalLanguageSelector
Gerrit-Branch: master
Gerrit-Owner: Nikerabbit <[email protected]>
Gerrit-Reviewer: KartikMistry <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to