@telus-uds/components-base 1.5.0 → 1.7.0

package/lib/Button/ButtonGroup.js

@@ -21,9 +21,7 @@ var _ViewportProvider = require("../ViewportProvider");
 
 var _ThemeProvider = require("../ThemeProvider");
 
-var _props = require("../utils/props");
-
-var _input = require("../utils/input");
+var _utils = require("../utils");
 
 var _pressability = require("../utils/pressability");
 
@@ -35,7 +33,8 @@ function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "functio
 
 function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || typeof obj !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
 
-const [selectProps, selectedSystemPropTypes] = (0, _props.selectSystemProps)([_props.a11yProps, _props.pressProps, _props.viewProps]);
+const [selectProps, selectedSystemPropTypes] = (0, _utils.selectSystemProps)([_utils.a11yProps, _utils.viewProps]);
+const [selectItemProps, selectedItemPropTypes] = (0, _utils.selectSystemProps)([_utils.a11yProps, _utils.focusHandlerProps, _utils.pressProps, _utils.viewProps]);
 const ButtonGroup = /*#__PURE__*/(0, _react.forwardRef)(({
   variant,
   tokens,
@@ -57,7 +56,7 @@ const ButtonGroup = /*#__PURE__*/(0, _react.forwardRef)(({
   const themeTokens = (0, _ThemeProvider.useThemeTokens)('ButtonGroup', tokens, variant, {
     viewport
   });
-  const stackTokens = (0, _props.selectTokens)('StackView', themeTokens);
+  const stackTokens = (0, _utils.selectTokens)('StackView', themeTokens);
   const {
     direction,
     space
@@ -66,7 +65,7 @@ const ButtonGroup = /*#__PURE__*/(0, _react.forwardRef)(({
   const {
     currentValues,
     toggleOneValue
-  } = (0, _input.useMultipleInputValues)({
+  } = (0, _utils.useMultipleInputValues)({
     initialValues,
     values,
     maxValues,
@@ -78,6 +77,12 @@ const ButtonGroup = /*#__PURE__*/(0, _react.forwardRef)(({
     ...rest
   });
   const itemA11yRole = systemProps.accessibilityRole === 'radiogroup' ? 'radio' : 'checkbox';
+  const uniqueFields = ['id', 'label'];
+
+  if (!(0, _utils.containUniqueFields)(items, uniqueFields)) {
+    throw new Error(`ButtonGroup items must have unique ${uniqueFields.join(', ')}`);
+  }
+
   return /*#__PURE__*/(0, _jsxRuntime.jsx)(_StackView.StackWrap, { ...systemProps,
     space: space,
     direction: direction,
@@ -87,7 +92,8 @@ const ButtonGroup = /*#__PURE__*/(0, _react.forwardRef)(({
       label,
       id = label,
       accessibilityLabel,
-      ref: itemRef
+      ref: itemRef,
+      ...itemRest
     }, index) => {
       const isSelected = currentValues.includes(id); // Pass an object of relevant component state as first argument for any passed-in press handlers
 
@@ -98,7 +104,7 @@ const ButtonGroup = /*#__PURE__*/(0, _react.forwardRef)(({
       }]);
 
       const handlePress = event => {
-        if (pressHandlers.onPress) pressHandlers.onPress();
+        if (pressHandlers.onPress) pressHandlers.onPress(event);
         toggleOneValue(id, event);
       };
 
@@ -108,7 +114,7 @@ const ButtonGroup = /*#__PURE__*/(0, _react.forwardRef)(({
         },
         accessibilityRole: itemA11yRole,
         accessibilityLabel,
-        ..._props.a11yProps.getPositionInSet(items.length, index)
+        ..._utils.a11yProps.getPositionInSet(items.length, index)
       }; // Ensure button is direct child of group as MacOS voiceover only applies "X of Y" to
       // "radio" if it's a direct child of "radiogroup", even if aria-posinset etc exists
 
@@ -120,6 +126,7 @@ const ButtonGroup = /*#__PURE__*/(0, _react.forwardRef)(({
         selected: isSelected,
         inactive: inactive,
         ...itemA11y,
+        ...selectItemProps(itemRest),
         children: label
       }, id);
     })
@@ -127,8 +134,8 @@ const ButtonGroup = /*#__PURE__*/(0, _react.forwardRef)(({
 });
 ButtonGroup.displayName = 'ButtonGroup';
 ButtonGroup.propTypes = { ...selectedSystemPropTypes,
-  tokens: (0, _props.getTokensPropType)('ButtonGroup'),
-  variant: _props.variantProp.propType,
+  tokens: (0, _utils.getTokensPropType)('ButtonGroup'),
+  variant: _utils.variantProp.propType,
 
   /**
    * The maximum number of items a user may select at once. Defaults to 1 and behaves
@@ -139,7 +146,8 @@ ButtonGroup.propTypes = { ...selectedSystemPropTypes,
   /**
    * The options a user may select
    */
-  items: _propTypes.default.arrayOf(_propTypes.default.shape({
+  items: _propTypes.default.arrayOf(_propTypes.default.shape({ ...selectedItemPropTypes,
+
     /**
      * The text displayed to the user in the button, describing this option,
      * passed to the button as its child.

package/lib/Card/PressableCardBase.js

@@ -29,7 +29,7 @@ function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "functio
 
 function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || typeof obj !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
 
-const [selectProps, selectedSystemPropTypes] = (0, _utils.selectSystemProps)([_utils.a11yProps, _utils.viewProps]);
+const [selectProps, selectedSystemPropTypes] = (0, _utils.selectSystemProps)([_utils.a11yProps, _utils.focusHandlerProps, _utils.viewProps]);
 const tokenKeys = [...(0, _utils.getTokenNames)('Card'), // Outer border tokens. TODO: centralise common token sets like these as part of
 // https://github.com/telus/universal-design-system/issues/782
 'outerBorderColor', 'outerBorderWidth', 'outerBorderGap'];

package/lib/Checkbox/Checkbox.js

@@ -41,7 +41,7 @@ function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "functio
 
 function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || typeof obj !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
 
-const [selectProps, selectedSystemPropTypes] = (0, _utils.selectSystemProps)([_utils.a11yProps, _utils.viewProps]);
+const [selectProps, selectedSystemPropTypes] = (0, _utils.selectSystemProps)([_utils.a11yProps, _utils.focusHandlerProps, _utils.viewProps]);
 
 const selectInputStyles = ({
   iconBackgroundColor,
@@ -232,24 +232,31 @@ const Checkbox = /*#__PURE__*/(0, _react.forwardRef)(({
             pressed
           });
           const iconTokens = selectIconTokens(stateTokens);
+          const labelStyles = selectLabelStyles(stateTokens);
+          const alignWithLabel = label ? [staticStyles.alignWithLabel, {
+            height: labelStyles.lineHeight
+          }] : null;
           return /*#__PURE__*/(0, _jsxRuntime.jsxs)(_View.default, {
             style: staticStyles.container,
-            children: [/*#__PURE__*/(0, _jsxRuntime.jsxs)(_View.default, {
-              style: _StyleSheet.default.flatten([staticStyles.defaultInputStyles, selectInputStyles(stateTokens, isChecked)]),
-              testID: "Checkbox-Input",
-              children: [/*#__PURE__*/(0, _jsxRuntime.jsx)(_CheckboxInput.default, {
-                checked: isChecked,
-                defaultChecked: defaultChecked,
-                disabled: inactive,
-                id: inputId,
-                isControlled: isControlled,
-                name: name,
-                value: value
-              }), isChecked && IconComponent && /*#__PURE__*/(0, _jsxRuntime.jsx)(IconComponent, { ...iconTokens,
-                testID: "Checkbox-Icon"
-              })]
+            children: [/*#__PURE__*/(0, _jsxRuntime.jsx)(_View.default, {
+              style: alignWithLabel,
+              children: /*#__PURE__*/(0, _jsxRuntime.jsxs)(_View.default, {
+                style: [staticStyles.defaultInputStyles, selectInputStyles(stateTokens, isChecked)],
+                testID: "Checkbox-Input",
+                children: [/*#__PURE__*/(0, _jsxRuntime.jsx)(_CheckboxInput.default, {
+                  checked: isChecked,
+                  defaultChecked: defaultChecked,
+                  disabled: inactive,
+                  id: inputId,
+                  isControlled: isControlled,
+                  name: name,
+                  value: value
+                }), isChecked && IconComponent && /*#__PURE__*/(0, _jsxRuntime.jsx)(IconComponent, { ...iconTokens,
+                  testID: "Checkbox-Icon"
+                })]
+              })
             }), Boolean(label) && /*#__PURE__*/(0, _jsxRuntime.jsx)(_Text.default, {
-              style: selectLabelStyles(stateTokens),
+              style: labelStyles,
               children: /*#__PURE__*/(0, _jsxRuntime.jsx)(_LabelContent.default, {
                 forId: inputId,
                 children: label
@@ -347,5 +354,9 @@ const staticStyles = _StyleSheet.default.create({
   defaultInputStyles: {
     alignItems: 'center',
     justifyContent: 'center'
+  },
+  alignWithLabel: {
+    alignSelf: 'flex-start',
+    justifyContent: 'center'
   }
 });

package/lib/Checkbox/CheckboxGroup.js

@@ -31,6 +31,8 @@ function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "functio
 
 function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || typeof obj !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
 
+const [selectProps, selectedSystemPropTypes] = (0, _utils.selectSystemProps)([_utils.a11yProps, _utils.viewProps]);
+const [selectItemProps, selectedItemPropTypes] = (0, _utils.selectSystemProps)([_utils.a11yProps, _utils.focusHandlerProps, _utils.viewProps]);
 /**
  * A group of Checkboxs that behave as a fieldset. Use when users select any number of choices from options.
  *
@@ -78,6 +80,7 @@ function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj &&
  * />
  * ```
  */
+
 const CheckboxGroup = /*#__PURE__*/(0, _react.forwardRef)(({
   tokens,
   radioTokens,
@@ -93,7 +96,8 @@ const CheckboxGroup = /*#__PURE__*/(0, _react.forwardRef)(({
   onChange,
   readOnly,
   name: inputGroupName,
-  inactive
+  inactive,
+  ...rest
 }, ref) => {
   const viewport = (0, _ViewportProvider.useViewport)();
   const {
@@ -111,11 +115,18 @@ const CheckboxGroup = /*#__PURE__*/(0, _react.forwardRef)(({
     onChange,
     readOnly: readOnly || inactive
   });
+  const uniqueFields = ['id', 'label'];
+
+  if (!(0, _utils.containUniqueFields)(items, uniqueFields)) {
+    throw new Error(`CheckboxGroup items must have unique ${uniqueFields.join(', ')}`);
+  }
+
   const checkboxes = items.map(({
     label,
     id,
     onChange: itemOnChange,
-    ref: itemRef
+    ref: itemRef,
+    ...itemRest
   }, index) => {
     const checkboxId = id || `Checkbox[${index}]`;
 
@@ -133,7 +144,8 @@ const CheckboxGroup = /*#__PURE__*/(0, _react.forwardRef)(({
       label: label,
       name: inputGroupName,
       tokens: radioTokens,
-      variant: variant
+      variant: variant,
+      ...selectItemProps(itemRest)
     }, checkboxId);
   });
   return /*#__PURE__*/(0, _jsxRuntime.jsx)(_Fieldset.default, {
@@ -146,6 +158,7 @@ const CheckboxGroup = /*#__PURE__*/(0, _react.forwardRef)(({
     feedback: feedback,
     inactive: inactive,
     validation: validation,
+    ...selectProps(rest),
     children: (0, _StackView.getStackedContent)(checkboxes, {
       space,
       direction: 'column'
@@ -153,7 +166,8 @@ const CheckboxGroup = /*#__PURE__*/(0, _react.forwardRef)(({
   });
 });
 CheckboxGroup.displayName = 'CheckboxGroup';
-CheckboxGroup.propTypes = {
+CheckboxGroup.propTypes = { ...selectedSystemPropTypes,
+
   /**
    * Optional theme token overrides for the outer CheckboxGroup component
    */
@@ -172,7 +186,7 @@ CheckboxGroup.propTypes = {
   /**
    * Array of objects containing specifics for each Checkbox to be rendered in the group.
    */
-  items: _propTypes.default.arrayOf(_propTypes.default.exact({
+  items: _propTypes.default.arrayOf(_propTypes.default.exact({ ...selectedItemPropTypes,
     label: _propTypes.default.string,
     id: _propTypes.default.string,
     onChange: _propTypes.default.func,

package/lib/ExpandCollapse/Panel.js

@@ -86,12 +86,13 @@ const ExpandCollapsePanel = /*#__PURE__*/(0, _react.forwardRef)(({
   };
 
   const onContainerLayout = event => {
-    if (containerHeight === null) {
-      const {
-        layout: {
-          height = 0
-        } = {}
-      } = event.nativeEvent;
+    const {
+      layout: {
+        height = 0
+      } = {}
+    } = event.nativeEvent;
+
+    if (_Platform.default.OS === 'web' || _Platform.default.OS !== 'web' && containerHeight === null) {
       setContainerHeight(height);
     }
   };
@@ -116,12 +117,11 @@ const ExpandCollapsePanel = /*#__PURE__*/(0, _react.forwardRef)(({
       tokens: controlTokens,
       onPress: handleControlPress,
       children: control
-    }), /*#__PURE__*/(0, _jsxRuntime.jsx)(_View.default, {
-      style: overflowContainerStyles,
+    }), /*#__PURE__*/(0, _jsxRuntime.jsx)(_Animated.default.View, {
+      style: [overflowContainerStyles, animatedStyles, staticStyles.itemsContainer],
       ...focusabilityProps,
-      children: /*#__PURE__*/(0, _jsxRuntime.jsx)(_Animated.default.View, {
+      children: /*#__PURE__*/(0, _jsxRuntime.jsx)(_View.default, {
         onLayout: onContainerLayout,
-        style: [staticStyles.itemsContainer, animatedStyles],
         children: /*#__PURE__*/(0, _jsxRuntime.jsx)(_View.default, {
           style: selectContainerStyles(themeTokens),
           children: children

package/lib/FlexGrid/Col/Col.js

@@ -46,6 +46,7 @@ const Col = /*#__PURE__*/(0, _react.forwardRef)(({
   mdOffset,
   lgOffset,
   xlOffset,
+  flex,
   ...viewProps
 }, ref) => {
   const gutter = (0, _react.useContext)(_GutterContext.default);
@@ -119,8 +120,11 @@ const Col = /*#__PURE__*/(0, _react.forwardRef)(({
     paddingLeft: gutter ? 16 : 0,
     paddingRight: gutter ? 16 : 0
   };
-  let hidingStyles = {};
-  const shown = _Platform.default.OS === 'web' ? 'block' : 'flex';
+  let hidingStyles = {}; // TODO: consider setting this to always 'flex' in a major release.
+  // `display: block` is invalid in native apps.
+  // See https://telusdigital.atlassian.net/browse/UDS1-92
+
+  const shown = !flex && _Platform.default.OS === 'web' ? 'block' : 'flex';
 
   if (viewPort === _systemConstants.viewports.xs) {
     hidingStyles = {
@@ -286,7 +290,13 @@ Col.propTypes = {
    *
    * Accepts a `PropType.string` following the [responsive prop](#/Layout?id=responsive) structure.
    */
-  horizontalAlign: _utils.responsiveProps.getTypeOptionallyByViewport(_propTypes.default.oneOf(['left', 'center', 'right']))
+  horizontalAlign: _utils.responsiveProps.getTypeOptionallyByViewport(_propTypes.default.oneOf(['left', 'center', 'right'])),
+
+  /**
+   * (web only) Stretches the column to fill vertical space using `display: flex`.
+   * In native apps, FlexGrid.Col behaves as if this is always true (as do all `View`s).
+   */
+  flex: _propTypes.default.bool
 };
 var _default = Col;
 exports.default = _default;

package/lib/FlexGrid/Row/Row.js

@@ -106,25 +106,31 @@ const Row = /*#__PURE__*/(0, _react.forwardRef)(({
   const viewPort = (0, _ViewportProvider.useViewport)();
   const reverseLevel = (0, _helpers.default)([xsReverse, smReverse, mdReverse, lgReverse, xlReverse]);
   let flexDirection = '';
+  let flexWrap = '';
 
   if (viewPort === _systemConstants.viewports.xs) {
     flexDirection = reverseLevel[0] ? 'row-reverse' : 'row';
+    flexWrap = reverseLevel[0] ? 'wrap-reverse' : 'wrap';
   }
 
   if (viewPort === _systemConstants.viewports.sm) {
     flexDirection = reverseLevel[1] ? 'row-reverse' : 'row';
+    flexWrap = reverseLevel[1] ? 'wrap-reverse' : 'wrap';
   }
 
   if (viewPort === _systemConstants.viewports.md) {
     flexDirection = reverseLevel[2] ? 'row-reverse' : 'row';
+    flexWrap = reverseLevel[2] ? 'wrap-reverse' : 'wrap';
   }
 
   if (viewPort === _systemConstants.viewports.lg) {
     flexDirection = reverseLevel[3] ? 'row-reverse' : 'row';
+    flexWrap = reverseLevel[3] ? 'wrap-reverse' : 'wrap';
   }
 
   if (viewPort === _systemConstants.viewports.xl) {
     flexDirection = reverseLevel[4] ? 'row-reverse' : 'row';
+    flexWrap = reverseLevel[4] ? 'wrap-reverse' : 'wrap';
   }
 
   return /*#__PURE__*/(0, _jsxRuntime.jsx)(_View.default, {
@@ -132,6 +138,7 @@ const Row = /*#__PURE__*/(0, _react.forwardRef)(({
     ...rest,
     style: [styles.row, {
       flexDirection,
+      flexWrap,
       ...horizontalAlignStyles(horizontalAlign),
       ...verticalAlignStyles(verticalAlign),
       ...distributeStyles(distribute)
@@ -149,8 +156,7 @@ const styles = _StyleSheet.default.create({
     flexGrow: 0,
     flexShrink: 1,
     flexBasis: 'auto',
-    flexDirection: 'row',
-    flexWrap: 'wrap'
+    flexDirection: 'row'
   }
 });
 

package/lib/InputLabel/InputLabel.js

@@ -69,25 +69,29 @@ const InputLabel = /*#__PURE__*/(0, _react.forwardRef)(({
   const themeTokens = (0, _ThemeProvider.useThemeTokens)('InputLabel', tokens, variant);
   const hasTooltip = tooltip !== undefined;
   const isHintInline = hintPosition === 'inline';
-  return /*#__PURE__*/(0, _jsxRuntime.jsxs)(_View.default, {
-    ref: ref,
-    style: [staticStyles.container, !isHintInline && staticStyles.containerWithHintBelow],
-    ...selectProps(rest),
-    children: [/*#__PURE__*/(0, _jsxRuntime.jsx)(_Text.default, {
-      style: [selectLabelStyles(themeTokens), selectGapStyles(themeTokens), staticStyles.label],
-      children: /*#__PURE__*/(0, _jsxRuntime.jsx)(_LabelContent.default, {
-        forId: forId,
-        children: label
-      })
-    }), hint && isHintInline && /*#__PURE__*/(0, _jsxRuntime.jsx)(_Text.default, {
-      style: [selectHintStyles(themeTokens), hasTooltip && selectGapStyles(themeTokens)],
-      nativeID: hintId,
-      children: hint
-    }), hasTooltip && /*#__PURE__*/(0, _jsxRuntime.jsx)(_View.default, {
-      style: staticStyles.tooltipAlign,
-      children: /*#__PURE__*/(0, _jsxRuntime.jsx)(_Tooltip.default, {
-        content: tooltip
-      })
+  return /*#__PURE__*/(0, _jsxRuntime.jsxs)(_jsxRuntime.Fragment, {
+    children: [/*#__PURE__*/(0, _jsxRuntime.jsxs)(_View.default, {
+      ref: ref,
+      style: staticStyles.container,
+      ...selectProps(rest),
+      children: [/*#__PURE__*/(0, _jsxRuntime.jsx)(_Text.default, {
+        style: [selectLabelStyles(themeTokens), selectGapStyles(themeTokens), staticStyles.label],
+        children: /*#__PURE__*/(0, _jsxRuntime.jsx)(_LabelContent.default, {
+          forId: forId,
+          children: label
+        })
+      }), hint && isHintInline && /*#__PURE__*/(0, _jsxRuntime.jsx)(_Text.default, {
+        style: [selectHintStyles(themeTokens), hasTooltip && selectGapStyles(themeTokens), staticStyles.label],
+        nativeID: hintId,
+        children: hint
+      }), hasTooltip && /*#__PURE__*/(0, _jsxRuntime.jsx)(_View.default, {
+        style: [staticStyles.tooltipAlign, {
+          height: themeTokens.fontSize * themeTokens.lineHeight
+        }],
+        children: /*#__PURE__*/(0, _jsxRuntime.jsx)(_Tooltip.default, {
+          content: tooltip
+        })
+      })]
     }), hint && !isHintInline && /*#__PURE__*/(0, _jsxRuntime.jsx)(_Text.default, {
       style: [selectHintStyles(themeTokens), staticStyles.hintBelow],
       nativeID: hintId,
@@ -135,21 +139,19 @@ exports.default = _default;
 
 const staticStyles = _StyleSheet.default.create({
   container: {
-    display: 'flex',
+    flexShrink: 1,
     flexDirection: 'row',
     alignItems: 'baseline'
   },
-  containerWithHintBelow: {
-    flexWrap: 'wrap'
-  },
   label: {
-    flexShrink: 0
+    flexShrink: 1
   },
   hintBelow: {
     flexBasis: '100%',
     flexShrink: 0
   },
   tooltipAlign: {
-    alignSelf: 'center'
+    alignSelf: 'flex-start',
+    justifyContent: 'center'
   }
 });

package/lib/Link/LinkBase.js

@@ -60,13 +60,9 @@ _Platform.default.OS === 'web' ? {
 } : {};
 
 const selectTextStyles = ({
-  color,
-  textLine,
-  textLineStyle
+  color
 }) => ({
   color,
-  textDecorationLine: textLine,
-  textDecorationStyle: textLineStyle,
   ..._Platform.default.select({
     web: {
       // TODO: https://github.com/telus/universal-design-system/issues/487
@@ -87,6 +83,22 @@ const selectBlockStyles = ({
   fontName: blockFontName
 });
 
+const selectDecorationStyles = ({
+  color,
+  textLine,
+  textLineStyle
+}) => ({
+  color,
+  textDecorationLine: textLine,
+  textDecorationStyle: textLineStyle,
+  ..._Platform.default.select({
+    web: {
+      // TODO: https://github.com/telus/universal-design-system/issues/487
+      transition: 'color 200ms'
+    }
+  })
+});
+
 const selectIconTokens = ({
   color,
   iconSize,
@@ -165,8 +177,9 @@ const LinkBase = /*#__PURE__*/(0, _react.forwardRef)(({
     style: linkState => {
       const themeTokens = resolveLinkTokens(linkState);
       const outerBorderStyles = selectOuterBorderStyles(themeTokens);
+      const decorationStyles = selectDecorationStyles(themeTokens);
       const hasIcon = Boolean(icon || themeTokens.icon);
-      return [outerBorderStyles, blockLeftStyle, hasIcon && staticStyles.rowContainer];
+      return [outerBorderStyles, blockLeftStyle, decorationStyles, hasIcon && staticStyles.rowContainer];
     },
     children: linkState => {
       const themeTokens = resolveLinkTokens(linkState);

package/lib/Link/TextButton.js

@@ -15,8 +15,6 @@ var _LinkBase = _interopRequireDefault(require("./LinkBase"));
 
 var _jsxRuntime = require("react/jsx-runtime");
 
-var _TextButton$propTypes;
-
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
 function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function (nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
@@ -33,8 +31,6 @@ const TextButton = /*#__PURE__*/(0, _react.forwardRef)(({
   children,
   variant,
   tokens,
-  // TODO: this may need to use `link` role on Web in the case of being passed both `href` and
-  // `onPress` in an omniplatform app that uses React Navigation's useLinkProps for internal nav.
   accessibilityRole = 'button',
   ...linkProps
 }, ref) => {
@@ -51,11 +47,6 @@ const TextButton = /*#__PURE__*/(0, _react.forwardRef)(({
 TextButton.displayName = 'TextButton';
 TextButton.propTypes = { ..._LinkBase.default.propTypes,
   onPress: _propTypes.default.func.isRequired
-}; // Remove incompatible Link prop (if this build includes propTypes)
-// TODO: test if this works with web navigation in omniplatform React Navigation
-// https://github.com/telus/universal-design-system/issues/665
-// eslint-disable-next-line react/forbid-foreign-prop-types
-
-if ((_TextButton$propTypes = TextButton.propTypes) !== null && _TextButton$propTypes !== void 0 && _TextButton$propTypes.href) delete TextButton.propTypes.href;
+};
 var _default = TextButton;
 exports.default = _default;

package/lib/List/ListItem.js

@@ -45,8 +45,7 @@ const selectBulletContainerStyles = ({
   itemBulletContainerAlign
 }) => ({
   width: itemBulletContainerWidth,
-  alignItems: itemBulletContainerAlign,
-  justifyContent: itemBulletContainerAlign
+  alignItems: itemBulletContainerAlign
 });
 
 const selectItemIconTokens = ({
@@ -57,16 +56,20 @@ const selectItemIconTokens = ({
   color: itemIconColor
 });
 
-const selectCommonIconStyles = ({
+const selectSideItemContainerStyles = ({
+  listGutter,
   iconMarginTop
 }) => ({
-  marginTop: iconMarginTop
-});
+  marginTop: iconMarginTop,
+  marginRight: listGutter
+}); // Align bullets with the top line of text the same way icons are aligned
 
-const selectSideItemContainerStyles = ({
-  listGutter
+
+const selectBulletPositioningStyles = ({
+  itemIconSize
 }) => ({
-  marginRight: listGutter
+  width: itemIconSize,
+  height: itemIconSize
 });
 
 const selectItemStyles = ({
@@ -119,8 +122,8 @@ const ListItem = /*#__PURE__*/(0, _react.forwardRef)(({
   const dividerStyles = selectDividerStyles(themeTokens);
   const itemBulletContainerStyles = selectBulletContainerStyles(themeTokens);
   const itemBulletStyles = selectBulletStyles(themeTokens);
+  const itemBulletPositioningStyles = selectBulletPositioningStyles(themeTokens);
   const iconTokens = selectItemIconTokens(themeTokens);
-  const commonIconStyles = selectCommonIconStyles(themeTokens);
   const sideItemContainerStyles = selectSideItemContainerStyles(themeTokens);
 
   const accessibilityRole = _Platform.default.select({
@@ -164,7 +167,7 @@ const ListItem = /*#__PURE__*/(0, _react.forwardRef)(({
 
     if (icon) {
       return /*#__PURE__*/(0, _jsxRuntime.jsx)(_View.default, {
-        style: [sideItemContainerStyles, commonIconStyles],
+        style: sideItemContainerStyles,
         children: /*#__PURE__*/(0, _jsxRuntime.jsx)(IconComponent, {
           size: iconSize || iconTokens.size,
           color: iconColor || iconTokens.color
@@ -175,8 +178,11 @@ const ListItem = /*#__PURE__*/(0, _react.forwardRef)(({
     return /*#__PURE__*/(0, _jsxRuntime.jsx)(_View.default, {
       style: [sideItemContainerStyles, itemBulletContainerStyles],
       children: /*#__PURE__*/(0, _jsxRuntime.jsx)(_View.default, {
-        style: itemBulletStyles,
-        testID: "unordered-item-bullet"
+        style: [staticStyles.bulletPositioning, itemBulletPositioningStyles],
+        children: /*#__PURE__*/(0, _jsxRuntime.jsx)(_View.default, {
+          style: itemBulletStyles,
+          testID: "unordered-item-bullet"
+        })
       })
     });
   };
@@ -197,6 +203,10 @@ const staticStyles = _StyleSheet.default.create({
   },
   wrap: {
     flex: 1
+  },
+  bulletPositioning: {
+    alignItems: 'center',
+    justifyContent: 'center'
   }
 });