@workday/canvas-kit-react 8.0.0-alpha.172-next.1 → 8.0.0-alpha.178-next.2

package/common/lib/utils/elements.ts

@@ -12,15 +12,9 @@ export const isFocusable = (element: HTMLElement) => {
   const validAnchor = nodeName === 'a' && element.hasAttribute('href');
   const validAudioVideo = ['audio', 'video'].includes(nodeName) && element.hasAttribute('controls');
   const validImgObject = ['img', 'object'].includes(nodeName) && element.hasAttribute('usemap');
-  const validNativelyFocusable = [
-    'button',
-    'details',
-    'embed',
-    'iframe',
-    'select',
-    'textarea',
-  ].includes(nodeName) &&
-  element.getAttribute('tabindex') !== '-1';
+  const validNativelyFocusable =
+    ['button', 'details', 'embed', 'iframe', 'select', 'textarea'].includes(nodeName) &&
+    element.getAttribute('tabindex') !== '-1';
   const hasTabIndex = element.getAttribute('tabindex') === '0';
 
   return (

package/dist/commonjs/common/lib/utils/elements.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"elements.d.ts","sourceRoot":"","sources":["../../../../../common/lib/utils/elements.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,eAAO,MAAM,WAAW,YAAa,WAAW,mBA6B/C,CAAC;AAEF;;GAEG;AACH,eAAO,MAAM,wBAAwB,cAAe,WAAW,KAAG,WAAW,GAAG,IAW/E,CAAC;AAEF;;GAEG;AACH,eAAO,MAAM,uBAAuB,cAAe,WAAW,KAAG,WAAW,GAAG,IAW9E,CAAC"}
+{"version":3,"file":"elements.d.ts","sourceRoot":"","sources":["../../../../../common/lib/utils/elements.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,eAAO,MAAM,WAAW,YAAa,WAAW,mBAuB/C,CAAC;AAEF;;GAEG;AACH,eAAO,MAAM,wBAAwB,cAAe,WAAW,KAAG,WAAW,GAAG,IAW/E,CAAC;AAEF;;GAEG;AACH,eAAO,MAAM,uBAAuB,cAAe,WAAW,KAAG,WAAW,GAAG,IAW9E,CAAC"}

package/dist/commonjs/common/lib/utils/elements.js

@@ -14,14 +14,7 @@ var isFocusable = function (element) {
     var validAnchor = nodeName === 'a' && element.hasAttribute('href');
     var validAudioVideo = ['audio', 'video'].includes(nodeName) && element.hasAttribute('controls');
     var validImgObject = ['img', 'object'].includes(nodeName) && element.hasAttribute('usemap');
-    var validNativelyFocusable = [
-        'button',
-        'details',
-        'embed',
-        'iframe',
-        'select',
-        'textarea',
-    ].includes(nodeName) &&
+    var validNativelyFocusable = ['button', 'details', 'embed', 'iframe', 'select', 'textarea'].includes(nodeName) &&
         element.getAttribute('tabindex') !== '-1';
     var hasTabIndex = element.getAttribute('tabindex') === '0';
     return (validInput ||

package/dist/commonjs/pagination/lib/Pagination/usePaginationModel.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"usePaginationModel.d.ts","sourceRoot":"","sources":["../../../../../pagination/lib/Pagination/usePaginationModel.ts"],"names":[],"mappings":"AAIA,OAAO,EAAC,eAAe,EAAC,MAAM,SAAS,CAAC;AAExC,oBAAY,wBAAwB,GAAG;IACrC;;;OAGG;IACH,QAAQ,EAAE,MAAM,CAAC;IACjB;;;OAGG;IACH,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB;;;OAGG;IACH,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B;;OAEG;IACH,YAAY,CAAC,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,IAAI,CAAC;IAC5C;;;OAGG;IACH,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB,CAAC;AAEF,eAAO,MAAM,kBAAkB,0EAM5B,wBAAwB,KAAG,eAgE7B,CAAC"}
+{"version":3,"file":"usePaginationModel.d.ts","sourceRoot":"","sources":["../../../../../pagination/lib/Pagination/usePaginationModel.ts"],"names":[],"mappings":"AAIA,OAAO,EAAC,eAAe,EAAC,MAAM,SAAS,CAAC;AAExC,oBAAY,wBAAwB,GAAG;IACrC;;;OAGG;IACH,QAAQ,EAAE,MAAM,CAAC;IACjB;;;OAGG;IACH,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB;;;OAGG;IACH,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B;;OAEG;IACH,YAAY,CAAC,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,IAAI,CAAC;IAC5C;;;OAGG;IACH,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB,CAAC;AAEF,eAAO,MAAM,kBAAkB,0EAM5B,wBAAwB,KAAG,eAmE7B,CAAC"}

package/dist/commonjs/pagination/lib/Pagination/usePaginationModel.js

@@ -27,7 +27,11 @@ var usePaginationModel = function (_a) {
     var _b = _a.firstPage, firstPage = _b === void 0 ? 1 : _b, _c = _a.initialCurrentPage, initialCurrentPage = _c === void 0 ? 1 : _c, lastPage = _a.lastPage, _d = _a.rangeSize, rangeSize = _d === void 0 ? 5 : _d, onPageChange = _a.onPageChange;
     var _e = React.useState(initialCurrentPage), currentPage = _e[0], setCurrentPage = _e[1];
     var changePage = function (page) {
-        onPageChange === null || onPageChange === void 0 ? void 0 : onPageChange(page);
+        // Don't fire an `onPageChange` event if the page number is the same as the current page.
+        // `setCurrentPage` doesn't need this check, because React won't update the state if the values are the same.
+        if (currentPage !== page) {
+            onPageChange === null || onPageChange === void 0 ? void 0 : onPageChange(page);
+        }
         setCurrentPage(page);
     };
     var first = function () {

package/dist/es6/common/lib/utils/elements.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"elements.d.ts","sourceRoot":"","sources":["../../../../../common/lib/utils/elements.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,eAAO,MAAM,WAAW,YAAa,WAAW,mBA6B/C,CAAC;AAEF;;GAEG;AACH,eAAO,MAAM,wBAAwB,cAAe,WAAW,KAAG,WAAW,GAAG,IAW/E,CAAC;AAEF;;GAEG;AACH,eAAO,MAAM,uBAAuB,cAAe,WAAW,KAAG,WAAW,GAAG,IAW9E,CAAC"}
+{"version":3,"file":"elements.d.ts","sourceRoot":"","sources":["../../../../../common/lib/utils/elements.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,eAAO,MAAM,WAAW,YAAa,WAAW,mBAuB/C,CAAC;AAEF;;GAEG;AACH,eAAO,MAAM,wBAAwB,cAAe,WAAW,KAAG,WAAW,GAAG,IAW/E,CAAC;AAEF;;GAEG;AACH,eAAO,MAAM,uBAAuB,cAAe,WAAW,KAAG,WAAW,GAAG,IAW9E,CAAC"}

package/dist/es6/common/lib/utils/elements.js

@@ -11,14 +11,7 @@ export var isFocusable = function (element) {
     var validAnchor = nodeName === 'a' && element.hasAttribute('href');
     var validAudioVideo = ['audio', 'video'].includes(nodeName) && element.hasAttribute('controls');
     var validImgObject = ['img', 'object'].includes(nodeName) && element.hasAttribute('usemap');
-    var validNativelyFocusable = [
-        'button',
-        'details',
-        'embed',
-        'iframe',
-        'select',
-        'textarea',
-    ].includes(nodeName) &&
+    var validNativelyFocusable = ['button', 'details', 'embed', 'iframe', 'select', 'textarea'].includes(nodeName) &&
         element.getAttribute('tabindex') !== '-1';
     var hasTabIndex = element.getAttribute('tabindex') === '0';
     return (validInput ||

package/dist/es6/pagination/lib/Pagination/usePaginationModel.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"usePaginationModel.d.ts","sourceRoot":"","sources":["../../../../../pagination/lib/Pagination/usePaginationModel.ts"],"names":[],"mappings":"AAIA,OAAO,EAAC,eAAe,EAAC,MAAM,SAAS,CAAC;AAExC,oBAAY,wBAAwB,GAAG;IACrC;;;OAGG;IACH,QAAQ,EAAE,MAAM,CAAC;IACjB;;;OAGG;IACH,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB;;;OAGG;IACH,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B;;OAEG;IACH,YAAY,CAAC,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,IAAI,CAAC;IAC5C;;;OAGG;IACH,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB,CAAC;AAEF,eAAO,MAAM,kBAAkB,0EAM5B,wBAAwB,KAAG,eAgE7B,CAAC"}
+{"version":3,"file":"usePaginationModel.d.ts","sourceRoot":"","sources":["../../../../../pagination/lib/Pagination/usePaginationModel.ts"],"names":[],"mappings":"AAIA,OAAO,EAAC,eAAe,EAAC,MAAM,SAAS,CAAC;AAExC,oBAAY,wBAAwB,GAAG;IACrC;;;OAGG;IACH,QAAQ,EAAE,MAAM,CAAC;IACjB;;;OAGG;IACH,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB;;;OAGG;IACH,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B;;OAEG;IACH,YAAY,CAAC,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,IAAI,CAAC;IAC5C;;;OAGG;IACH,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB,CAAC;AAEF,eAAO,MAAM,kBAAkB,0EAM5B,wBAAwB,KAAG,eAmE7B,CAAC"}

package/dist/es6/pagination/lib/Pagination/usePaginationModel.js

@@ -5,7 +5,11 @@ export var usePaginationModel = function (_a) {
     var _b = _a.firstPage, firstPage = _b === void 0 ? 1 : _b, _c = _a.initialCurrentPage, initialCurrentPage = _c === void 0 ? 1 : _c, lastPage = _a.lastPage, _d = _a.rangeSize, rangeSize = _d === void 0 ? 5 : _d, onPageChange = _a.onPageChange;
     var _e = React.useState(initialCurrentPage), currentPage = _e[0], setCurrentPage = _e[1];
     var changePage = function (page) {
-        onPageChange === null || onPageChange === void 0 ? void 0 : onPageChange(page);
+        // Don't fire an `onPageChange` event if the page number is the same as the current page.
+        // `setCurrentPage` doesn't need this check, because React won't update the state if the values are the same.
+        if (currentPage !== page) {
+            onPageChange === null || onPageChange === void 0 ? void 0 : onPageChange(page);
+        }
         setCurrentPage(page);
     };
     var first = function () {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@workday/canvas-kit-react",
-  "version": "8.0.0-alpha.172-next.1+a9950b51",
+  "version": "8.0.0-alpha.178-next.2+037f11e3",
   "description": "The parent module that contains all Workday Canvas Kit React components",
   "author": "Workday, Inc. (https://www.workday.com)",
   "license": "Apache-2.0",
@@ -49,7 +49,7 @@
     "@emotion/styled": "^11.6.0",
     "@popperjs/core": "^2.5.4",
     "@workday/canvas-colors-web": "^2.0.0",
-    "@workday/canvas-kit-popup-stack": "^8.0.0-alpha.172-next.1+a9950b51",
+    "@workday/canvas-kit-popup-stack": "^8.0.0-alpha.178-next.2+037f11e3",
     "@workday/canvas-system-icons-web": "^3.0.0",
     "@workday/design-assets-types": "^0.2.8",
     "chroma-js": "^2.1.0",
@@ -68,5 +68,5 @@
     "@workday/canvas-accent-icons-web": "^3.0.0",
     "@workday/canvas-applet-icons-web": "^2.0.0"
   },
-  "gitHead": "a9950b51b3dc3b4c063c664c9347ac72c253fa5f"
+  "gitHead": "037f11e39699230003a7ec5f4550fa9d0e46cc86"
 }

package/pagination/lib/Pagination/usePaginationModel.ts

@@ -41,8 +41,11 @@ export const usePaginationModel = ({
   const [currentPage, setCurrentPage] = React.useState(initialCurrentPage);
 
   const changePage = (page: number) => {
-    onPageChange?.(page);
-
+    // Don't fire an `onPageChange` event if the page number is the same as the current page.
+    // `setCurrentPage` doesn't need this check, because React won't update the state if the values are the same.
+    if (currentPage !== page) {
+      onPageChange?.(page);
+    }
     setCurrentPage(page);
   };