jquery.suggestions: Combine two isRTL checks
authorFomafix <fomafix@googlemail.com>
Wed, 5 Jun 2019 20:08:48 +0000 (22:08 +0200)
committerFomafix <fomafix@googlemail.com>
Mon, 10 Jun 2019 14:17:47 +0000 (16:17 +0200)
Also update comment about the direction.

Change-Id: Iba9e925eef3148c2964d51cf5e55402dc85d65dc

resources/src/jquery/jquery.suggestions.js

index 4a1d637..e071bed 100644 (file)
@@ -78,7 +78,7 @@
  *
  * @param {string} [options.expandFrom=auto] Which direction to offset the suggestion box from.
  *  Values 'start' and 'end' translate to left and right respectively depending on the directionality
- *   of the current document, according to `$( 'html' ).css( 'direction' )`.
+ *   of the current document, according to `$( document.documentElement ).css( 'direction' )`.
  *   Valid values: "left", "right", "start", "end", and "auto".
  *
  * @param {boolean} [options.positionFromLeft] Sets `expandFrom=left`, for backwards
                                                // Process expandFrom, after this it is set to left or right.
                                                context.config.expandFrom = ( function ( expandFrom ) {
                                                        var regionWidth, docWidth, regionCenter, docCenter,
-                                                               docDir = $( document.documentElement ).css( 'direction' ),
+                                                               isRTL = $( document.documentElement ).css( 'direction' ) === 'rtl',
                                                                $region = context.config.$region;
 
                                                        // Backwards compatible
                                                        }
 
                                                        if ( expandFrom === 'start' ) {
-                                                               expandFrom = docDir === 'rtl' ? 'right' : 'left';
+                                                               expandFrom = isRTL ? 'right' : 'left';
 
                                                        } else if ( expandFrom === 'end' ) {
-                                                               expandFrom = docDir === 'rtl' ? 'left' : 'right';
+                                                               expandFrom = isRTL ? 'left' : 'right';
                                                        }
 
                                                        return expandFrom;