Browse Source

Performance optimization.

NaotoshiFujita 3 years ago
parent
commit
2ff36c2437

+ 2 - 2
dist/js/splide.cjs.js

@@ -802,6 +802,7 @@ function Slide$1(Splide2, index, slideIndex, slide) {
   const { resolve } = Components.Direction;
   const isClone = slideIndex > -1;
   const container = child(slide, `.${CLASS_CONTAINER}`);
+  const focusableNodes = options.focusableNodes && queryAll(slide, options.focusableNodes);
   let destroyed;
   function mount() {
     init();
@@ -859,12 +860,11 @@ function Slide$1(Splide2, index, slideIndex, slide) {
     }
   }
   function updateVisibility(visible) {
-    const { focusableNodes } = options;
     const ariaHidden = !visible && !isActive();
     setAttribute(slide, ARIA_HIDDEN, ariaHidden || null);
     setAttribute(slide, TAB_INDEX, !ariaHidden && options.slideFocus ? 0 : null);
     if (focusableNodes) {
-      queryAll(slide, focusableNodes).forEach((node) => {
+      focusableNodes.forEach((node) => {
         setAttribute(node, TAB_INDEX, ariaHidden ? -1 : null);
       });
     }

+ 2 - 2
dist/js/splide.esm.js

@@ -798,6 +798,7 @@ function Slide$1(Splide2, index, slideIndex, slide) {
   const { resolve } = Components.Direction;
   const isClone = slideIndex > -1;
   const container = child(slide, `.${CLASS_CONTAINER}`);
+  const focusableNodes = options.focusableNodes && queryAll(slide, options.focusableNodes);
   let destroyed;
   function mount() {
     init();
@@ -855,12 +856,11 @@ function Slide$1(Splide2, index, slideIndex, slide) {
     }
   }
   function updateVisibility(visible) {
-    const { focusableNodes } = options;
     const ariaHidden = !visible && !isActive();
     setAttribute(slide, ARIA_HIDDEN, ariaHidden || null);
     setAttribute(slide, TAB_INDEX, !ariaHidden && options.slideFocus ? 0 : null);
     if (focusableNodes) {
-      queryAll(slide, focusableNodes).forEach((node) => {
+      focusableNodes.forEach((node) => {
         setAttribute(node, TAB_INDEX, ariaHidden ? -1 : null);
       });
     }

+ 2 - 2
dist/js/splide.js

@@ -909,6 +909,7 @@ function _createClass(Constructor, protoProps, staticProps) { if (protoProps) _d
     var resolve = Components.Direction.resolve;
     var isClone = slideIndex > -1;
     var container = child(slide, "." + CLASS_CONTAINER);
+    var focusableNodes = options.focusableNodes && queryAll(slide, options.focusableNodes);
     var destroyed;
 
     function mount() {
@@ -982,13 +983,12 @@ function _createClass(Constructor, protoProps, staticProps) { if (protoProps) _d
     }
 
     function updateVisibility(visible) {
-      var focusableNodes = options.focusableNodes;
       var ariaHidden = !visible && !isActive();
       setAttribute(slide, ARIA_HIDDEN, ariaHidden || null);
       setAttribute(slide, TAB_INDEX, !ariaHidden && options.slideFocus ? 0 : null);
 
       if (focusableNodes) {
-        queryAll(slide, focusableNodes).forEach(function (node) {
+        focusableNodes.forEach(function (node) {
           setAttribute(node, TAB_INDEX, ariaHidden ? -1 : null);
         });
       }

File diff suppressed because it is too large
+ 0 - 0
dist/js/splide.js.map


File diff suppressed because it is too large
+ 0 - 0
dist/js/splide.min.js


BIN
dist/js/splide.min.js.gz


+ 1 - 1
dist/types/components/Slides/Slide.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"Slide.d.ts","sourceRoot":"","sources":["Slide.ts"],"names":[],"mappings":"AAiCA,OAAO,EAAE,MAAM,EAAE,MAAM,0BAA0B,CAAC;AAClD,OAAO,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAmB5C;;;;GAIG;AACH,MAAM,WAAY,cAAe,SAAQ,aAAa;IACpD,KAAK,EAAE,MAAM,CAAC;IACd,UAAU,EAAE,MAAM,CAAC;IACnB,KAAK,EAAE,WAAW,CAAC;IACnB,SAAS,EAAE,WAAW,CAAC;IACvB,OAAO,EAAE,OAAO,CAAC;IACjB,IAAI,CAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,GAAG,MAAM,EAAE,YAAY,CAAC,EAAE,OAAO,GAAI,IAAI,CAAA;IAC1E,QAAQ,CAAE,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAI,OAAO,CAAC;CACrD;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,KAAK,CAAE,MAAM,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,EAAE,KAAK,EAAE,WAAW,GAAI,cAAc,CAwM7G"}
+{"version":3,"file":"Slide.d.ts","sourceRoot":"","sources":["Slide.ts"],"names":[],"mappings":"AA+BA,OAAO,EAAE,MAAM,EAAE,MAAM,0BAA0B,CAAC;AAClD,OAAO,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAmB5C;;;;GAIG;AACH,MAAM,WAAY,cAAe,SAAQ,aAAa;IACpD,KAAK,EAAE,MAAM,CAAC;IACd,UAAU,EAAE,MAAM,CAAC;IACnB,KAAK,EAAE,WAAW,CAAC;IACnB,SAAS,EAAE,WAAW,CAAC;IACvB,OAAO,EAAE,OAAO,CAAC;IACjB,IAAI,CAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,GAAG,MAAM,EAAE,YAAY,CAAC,EAAE,OAAO,GAAI,IAAI,CAAA;IAC1E,QAAQ,CAAE,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAI,OAAO,CAAC;CACrD;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,KAAK,CAAE,MAAM,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,EAAE,KAAK,EAAE,WAAW,GAAI,cAAc,CAwM7G"}

+ 5 - 7
src/js/components/Slides/Slide.ts

@@ -22,11 +22,9 @@ import {
   EVENT_INACTIVE,
   EVENT_MOVE,
   EVENT_MOVED,
-  EVENT_REFRESH, EVENT_REPOSITIONED,
-  EVENT_RESIZED,
+  EVENT_REPOSITIONED,
   EVENT_SCROLLED,
   EVENT_SLIDE_KEYDOWN,
-  EVENT_UPDATED,
   EVENT_VISIBLE,
 } from '../../constants/events';
 import { FADE, SLIDE } from '../../constants/types';
@@ -83,8 +81,9 @@ export function Slide( Splide: Splide, index: number, slideIndex: number, slide:
   const { Components, root, options } = Splide;
   const { isNavigation, updateOnMove } = options;
   const { resolve } = Components.Direction;
-  const isClone   = slideIndex > -1;
-  const container = child( slide, `.${ CLASS_CONTAINER }` );
+  const isClone        = slideIndex > -1;
+  const container      = child( slide, `.${ CLASS_CONTAINER }` );
+  const focusableNodes = options.focusableNodes && queryAll( slide, options.focusableNodes );
 
   /**
    * Turns into `true` when the component is destroyed.
@@ -192,14 +191,13 @@ export function Slide( Splide: Splide, index: number, slideIndex: number, slide:
    * @param visible - Set `true` if the slide is visible.
    */
   function updateVisibility( this: SlideComponent, visible: boolean ): void {
-    const { focusableNodes } = options;
     const ariaHidden = ! visible && ! isActive();
 
     setAttribute( slide, ARIA_HIDDEN, ariaHidden || null );
     setAttribute( slide, TAB_INDEX, ! ariaHidden && options.slideFocus ? 0 : null );
 
     if ( focusableNodes ) {
-      queryAll( slide, focusableNodes ).forEach( node => {
+      focusableNodes.forEach( node => {
         setAttribute( node, TAB_INDEX, ariaHidden ? -1 : null );
       } );
     }

Some files were not shown because too many files changed in this diff