@telus-uds/components-base 1.57.0 → 1.58.0

package/CHANGELOG.md

@@ -1,12 +1,20 @@
 # Change Log - @telus-uds/components-base
 
-This log was last generated on Fri, 28 Jul 2023 18:10:45 GMT and should not be manually modified.
+This log was last generated on Wed, 02 Aug 2023 18:10:10 GMT and should not be manually modified.
 
 <!-- Start content -->
 
+## 1.58.0
+
+Wed, 02 Aug 2023 18:10:10 GMT
+
+### Minor changes
+
+- added condition on applying "raised" in Carousel for next/prev buttons  (kyle.king2@telus.com)
+
 ## 1.57.0
 
-Fri, 28 Jul 2023 18:10:45 GMT
+Fri, 28 Jul 2023 18:18:38 GMT
 
 ### Minor changes
 

package/lib/Carousel/Carousel.js

@@ -461,7 +461,8 @@ const Carousel = /*#__PURE__*/_react.default.forwardRef((_ref, ref) => {
 
   const previousNextIconButtonVariants = {
     size: previousNextIconSize,
-    raised: true
+    raised: !(variant !== null && variant !== void 0 && variant.inverse) && true,
+    inverse: variant === null || variant === void 0 ? void 0 : variant.inverse
   };
 
   const getCopyWithPlaceholders = _react.default.useCallback(copyKey => {

package/lib/Carousel/CarouselThumbnail.js

@@ -104,7 +104,9 @@ const CarouselThumbnail = _ref2 => {
         pressed,
         focus: focused
       }));
-      return [pressableStyles, index === activeIndex && styles.selected];
+      return [pressableStyles, index === activeIndex && styles.selected, {
+        outline: 'none'
+      }];
     },
     children: /*#__PURE__*/(0, _jsxRuntime.jsx)(_Image.default, {
       accessibilityIgnoresInvertColors: true,

package/lib-module/Carousel/Carousel.js

@@ -405,7 +405,8 @@ const Carousel = /*#__PURE__*/React.forwardRef((_ref, ref) => {
 
   const previousNextIconButtonVariants = {
     size: previousNextIconSize,
-    raised: true
+    raised: !(variant !== null && variant !== void 0 && variant.inverse) && true,
+    inverse: variant === null || variant === void 0 ? void 0 : variant.inverse
   };
   const getCopyWithPlaceholders = React.useCallback(copyKey => {
     const copyText = getCopy(copyKey).replace(/%\{title\}/g, title).replace(/%\{itemLabel\}/g, itemLabel).replace(/%\{stepNumber\}/g, activeIndex + 1).replace(/%\{stepCount\}/g, childrenArray.length); // First word might be a lowercase placeholder: capitalize the first letter

package/lib-module/Carousel/CarouselThumbnail.js

@@ -88,7 +88,9 @@ const CarouselThumbnail = _ref2 => {
         pressed,
         focus: focused
       }));
-      return [pressableStyles, index === activeIndex && styles.selected];
+      return [pressableStyles, index === activeIndex && styles.selected, {
+        outline: 'none'
+      }];
     },
     children: /*#__PURE__*/_jsx(Image, {
       accessibilityIgnoresInvertColors: true,

package/package.json

@@ -72,5 +72,5 @@
   "standard-engine": {
     "skip": true
   },
-  "version": "1.57.0"
+  "version": "1.58.0"
 }

package/src/Carousel/Carousel.jsx

@@ -420,7 +420,11 @@ const Carousel = React.forwardRef(
     // @TODO: - these are Allium-theme variants and won't have any effect in themes that don't implement them.
     // Normally we avoid setting variants of subcomponents, however this could be re-considered.
     // Related discussion - https://github.com/telus/universal-design-system/issues/1549
-    const previousNextIconButtonVariants = { size: previousNextIconSize, raised: true }
+    const previousNextIconButtonVariants = {
+      size: previousNextIconSize,
+      raised: !variant?.inverse && true,
+      inverse: variant?.inverse
+    }
 
     const getCopyWithPlaceholders = React.useCallback(
       (copyKey) => {

package/src/Carousel/CarouselThumbnail.jsx

@@ -60,7 +60,13 @@ const CarouselThumbnail = ({ accessibilityLabel, alt, index, src }) => {
           })
         )
 
-        return [pressableStyles, index === activeIndex && styles.selected]
+        return [
+          pressableStyles,
+          index === activeIndex && styles.selected,
+          {
+            outline: 'none'
+          }
+        ]
       }}
     >
       <Image