@pingux/astro 1.16.1-alpha.0 → 1.17.0-alpha.2

package/lib/components/Image/Image.js

@@ -14,7 +14,7 @@ import _objectWithoutProperties from "@babel/runtime-corejs3/helpers/esm/objectW
 
 var _templateObject;
 
-var _excluded = ["className", "fallbackImage", "fallbackTimeout", "isDisabled", "src", "sx"];
+var _excluded = ["className", "fallbackImage", "fallbackTimeout", "isDisabled", "src", "alt", "sx"];
 
 function ownKeys(object, enumerableOnly) { var keys = _Object$keys(object); if (_Object$getOwnPropertySymbols) { var symbols = _Object$getOwnPropertySymbols(object); enumerableOnly && (symbols = _filterInstanceProperty(symbols).call(symbols, function (sym) { return _Object$getOwnPropertyDescriptor(object, sym).enumerable; })), keys.push.apply(keys, symbols); } return keys; }
 
@@ -25,7 +25,7 @@ import PropTypes from 'prop-types';
 import { Image as ThemeUIImage } from 'theme-ui';
 import { useHover } from '@react-aria/interactions';
 import { keyframes } from '@emotion/react';
-import { usePropWarning, useStatusClasses, useFallbackImage } from '../../hooks';
+import { usePropWarning, useStatusClasses, useFallbackImage, useAriaLabelWarning } from '../../hooks';
 import { Box } from '../../index';
 import { neutral } from '../../styles/colors';
 /**
@@ -41,6 +41,7 @@ var Image = /*#__PURE__*/forwardRef(function (props, ref) {
       fallbackTimeout = props.fallbackTimeout,
       isDisabled = props.isDisabled,
       src = props.src,
+      alt = props.alt,
       sx = props.sx,
       others = _objectWithoutProperties(props, _excluded);
 
@@ -86,6 +87,8 @@ var Image = /*#__PURE__*/forwardRef(function (props, ref) {
     return imgRef.current;
   });
   usePropWarning(props, 'disabled', 'isDisabled');
+  var ariaLabel = props['aria-label'] || alt;
+  useAriaLabelWarning('Image', ariaLabel);
 
   var _useHover = useHover(props),
       hoverProps = _useHover.hoverProps,
@@ -142,7 +145,7 @@ var Image = /*#__PURE__*/forwardRef(function (props, ref) {
   var themeUiImage = ___EmotionJSX(ThemeUIImage, _extends({
     className: classNames,
     ref: imgRef,
-    alt: "Image",
+    alt: alt,
     role: "img",
     src: imgSrc,
     sx: sx
@@ -176,7 +179,15 @@ Image.propTypes = {
   fallbackImage: PropTypes.oneOfType([PropTypes.string, PropTypes.object]),
 
   /**  Time in milliseconds that component should wait for a response from src address. */
-  fallbackTimeout: PropTypes.number
+  fallbackTimeout: PropTypes.number,
+
+  /** Descriptive text for an image. This is *highly* recommended in most cases.
+   * See [MDN](https://developer.mozilla.org/en-US/docs/Web/HTML/Element/img#attr-alt).
+   * */
+  alt: PropTypes.string,
+
+  /** Defines a string value that labels the current element. */
+  'aria-label': PropTypes.string
 };
 Image.defaultProps = {
   isDisabled: false,

package/lib/components/Image/Image.stories.js

@@ -35,12 +35,15 @@ export default {
 export var Default = function Default(_ref) {
   var args = _extends({}, _ref);
 
-  return ___EmotionJSX(Image, args);
+  return ___EmotionJSX(Image, _extends({}, args, {
+    alt: "Ping identity square logo"
+  }));
 };
 export var Avatar = function Avatar() {
   return ___EmotionJSX(Image, {
     src: pingImg,
-    variant: "images.avatar"
+    variant: "images.avatar",
+    alt: "Ping identity round avatar"
   });
 };
 export var CustomSizeAndRadius = function CustomSizeAndRadius() {
@@ -51,13 +54,15 @@ export var CustomSizeAndRadius = function CustomSizeAndRadius() {
       height: '70px',
       borderRadius: 8,
       bg: 'neutral.90'
-    }
+    },
+    alt: "Terry Crews in a red suite"
   });
 };
 export var Disabled = function Disabled() {
   return ___EmotionJSX(Image, {
     src: pingImg,
-    isDisabled: true
+    isDisabled: true,
+    alt: "Ping identity square logo"
   });
 };
 export var FallbackImage = function FallbackImage() {
@@ -67,7 +72,8 @@ export var FallbackImage = function FallbackImage() {
     sx: {
       width: '150px',
       height: '150px'
-    }
+    },
+    alt: "Random image"
   });
 };
 export var WithSkeletonLoadSuccess = function WithSkeletonLoadSuccess(_ref2) {
@@ -78,7 +84,8 @@ export var WithSkeletonLoadSuccess = function WithSkeletonLoadSuccess(_ref2) {
     sx: {
       width: '150px',
       height: '150px'
-    }
+    },
+    alt: "Random image"
   });
 };
 WithSkeletonLoadSuccess.args = {
@@ -90,7 +97,8 @@ export var WithSkeletonLoadTimeout = function WithSkeletonLoadTimeout() {
     sx: {
       width: '150px',
       height: '150px'
-    }
+    },
+    alt: "Random image"
   });
 };
 export var UpdatingImageSrc = function UpdatingImageSrc() {
@@ -116,6 +124,7 @@ export var UpdatingImageSrc = function UpdatingImageSrc() {
       width: '200px',
       height: '200px',
       mt: '25px'
-    }
+    },
+    alt: image === pingImg ? 'Ping identity square logo' : 'Terry Crews in a red suite'
   }));
 };

package/lib/components/Image/Image.test.js

@@ -76,6 +76,15 @@ test('image shows disabled status', function () {
   var img = screen.getByRole('img');
   expect(img).toHaveClass('is-disabled');
 });
+test('image with alt text', function () {
+  getComponent({
+    alt: 'Test'
+  });
+  act(function () {
+    fallbackImageObj.onImageLoad();
+  });
+  expect(screen.getByAltText('Test')).toBeInTheDocument();
+});
 test('image source is able to be changed', function () {
   var src2 = 'second-src';
 

package/lib/components/MultivaluesField/MultivaluesField.stories.js

@@ -1,5 +1,5 @@
-import _slicedToArray from "@babel/runtime-corejs3/helpers/esm/slicedToArray";
 import _extends from "@babel/runtime-corejs3/helpers/esm/extends";
+import _slicedToArray from "@babel/runtime-corejs3/helpers/esm/slicedToArray";
 import React, { useState } from 'react';
 import { Box, Item, MultivaluesField, OverlayProvider } from '../..';
 import { jsx as ___EmotionJSX } from "@emotion/react";
@@ -7,7 +7,9 @@ export default {
   title: 'Form/MultivaluesField',
   component: MultivaluesField,
   argTypes: {
-    direction: {},
+    direction: {
+      defaultValue: 'bottom'
+    },
     hasAutoFocus: {},
     hasNoStatusIndicator: {},
     isDisabled: {},
@@ -89,7 +91,27 @@ var items = [{
   name: 'Shark',
   key: 'Shark'
 }];
+
+var setOverlayStyle = function setOverlayStyle(direction, isOpen, mr, ml, mt) {
+  return {
+    marginRight: direction === 'right' && isOpen ? mr : 0,
+    marginLeft: direction === 'left' && isOpen ? ml : 0,
+    marginTop: direction === 'top' && isOpen ? mt : 0
+  };
+};
+
 export var Default = function Default(args) {
+  var _useState = useState(false),
+      _useState2 = _slicedToArray(_useState, 2),
+      isOpen = _useState2[0],
+      setIsOpen = _useState2[1];
+
+  var direction = args.direction;
+
+  var onOpenChange = function onOpenChange() {
+    setIsOpen(true);
+  };
+
   return (// const items = [
     // { id: 1, name: 'Aardvark', key: 'Aardvark' },
     // { id: 2, name: 'Kangaroo', key: 'Kangaroo' },
@@ -99,9 +121,14 @@ export var Default = function Default(args) {
     // { id: 6, name: 'Orangutan', key: 'Orangutan' },
     // { id: 7, name: 'Shark', key: 'Shark' },
     // ];
-    ___EmotionJSX(OverlayProvider, null, ___EmotionJSX(MultivaluesField, _extends({
+    ___EmotionJSX(OverlayProvider // note: spacing for demo purpose only so that the select list renders in the correct place
+    , {
+      style: setOverlayStyle(direction, isOpen, '50%', '50%', '20%')
+    }, ___EmotionJSX(MultivaluesField, _extends({
       items: items
-    }, args), function (item) {
+    }, args, {
+      onOpenChange: onOpenChange
+    }), function (item) {
       return ___EmotionJSX(Item, {
         key: item.key,
         "data-id": item.name
@@ -110,10 +137,26 @@ export var Default = function Default(args) {
   );
 };
 export var WithCustomValues = function WithCustomValues(args) {
-  return ___EmotionJSX(OverlayProvider, null, ___EmotionJSX(MultivaluesField, _extends({
+  var _useState3 = useState(false),
+      _useState4 = _slicedToArray(_useState3, 2),
+      isOpen = _useState4[0],
+      setIsOpen = _useState4[1];
+
+  var direction = args.direction;
+
+  var onOpenChange = function onOpenChange() {
+    setIsOpen(true);
+  };
+
+  return ___EmotionJSX(OverlayProvider // note: spacing for demo purpose only so that the select list renders in the correct place
+  , {
+    style: setOverlayStyle(direction, isOpen, '50%', '50%', '20%')
+  }, ___EmotionJSX(MultivaluesField, _extends({
     items: items,
     mode: "non-restrictive"
-  }, args), function (item) {
+  }, args, {
+    onOpenChange: onOpenChange
+  }), function (item) {
     return ___EmotionJSX(Item, {
       key: item.key,
       "data-id": item.name
@@ -126,10 +169,26 @@ WithCustomValues.argTypes = {
   }
 };
 export var WithDisabledKeys = function WithDisabledKeys(args) {
-  return ___EmotionJSX(OverlayProvider, null, ___EmotionJSX(MultivaluesField, _extends({
+  var _useState5 = useState(false),
+      _useState6 = _slicedToArray(_useState5, 2),
+      isOpen = _useState6[0],
+      setIsOpen = _useState6[1];
+
+  var direction = args.direction;
+
+  var onOpenChange = function onOpenChange() {
+    setIsOpen(true);
+  };
+
+  return ___EmotionJSX(OverlayProvider // note: spacing for demo purpose only so that the select list renders in the correct place
+  , {
+    style: setOverlayStyle(direction, isOpen, '50%', '50%', '20%')
+  }, ___EmotionJSX(MultivaluesField, _extends({
     disabledKeys: ['Aardvark'],
     items: items
-  }, args), function (item) {
+  }, args, {
+    onOpenChange: onOpenChange
+  }), function (item) {
     return ___EmotionJSX(Item, {
       key: item.key,
       "data-id": item.name
@@ -137,10 +196,26 @@ export var WithDisabledKeys = function WithDisabledKeys(args) {
   }));
 };
 export var Uncontrolled = function Uncontrolled(args) {
-  return ___EmotionJSX(OverlayProvider, null, ___EmotionJSX(MultivaluesField, _extends({
+  var _useState7 = useState(false),
+      _useState8 = _slicedToArray(_useState7, 2),
+      isOpen = _useState8[0],
+      setIsOpen = _useState8[1];
+
+  var direction = args.direction;
+
+  var onOpenChange = function onOpenChange() {
+    setIsOpen(true);
+  };
+
+  return ___EmotionJSX(OverlayProvider // note: spacing for demo purpose only so that the select list renders in the correct place
+  , {
+    style: setOverlayStyle(direction, isOpen, '50%', '50%', '20%')
+  }, ___EmotionJSX(MultivaluesField, _extends({
     defaultSelectedKeys: ['Aardvark', 'Snake'],
     items: items
-  }, args), function (item) {
+  }, args, {
+    onOpenChange: onOpenChange
+  }), function (item) {
     return ___EmotionJSX(Item, {
       key: item.key,
       "data-id": item.name
@@ -148,17 +223,32 @@ export var Uncontrolled = function Uncontrolled(args) {
   }));
 };
 export var Controlled = function Controlled(args) {
-  var _useState = useState(['Aardvark', 'Snake']),
-      _useState2 = _slicedToArray(_useState, 2),
-      selectedKeys = _useState2[0],
-      setSelectedKeys = _useState2[1];
+  var _useState9 = useState(['Aardvark', 'Snake']),
+      _useState10 = _slicedToArray(_useState9, 2),
+      selectedKeys = _useState10[0],
+      setSelectedKeys = _useState10[1];
+
+  var _useState11 = useState(false),
+      _useState12 = _slicedToArray(_useState11, 2),
+      isOpen = _useState12[0],
+      setIsOpen = _useState12[1];
+
+  var direction = args.direction;
 
-  return ___EmotionJSX(OverlayProvider, null, ___EmotionJSX(MultivaluesField, _extends({
+  var onOpenChange = function onOpenChange() {
+    setIsOpen(true);
+  };
+
+  return ___EmotionJSX(OverlayProvider // note: spacing for demo purpose only so that the select list renders in the correct place
+  , {
+    style: setOverlayStyle(direction, isOpen, '50%', '50%', '20%')
+  }, ___EmotionJSX(MultivaluesField, _extends({
     label: "Field Label"
   }, args, {
     items: items,
     onSelectionChange: setSelectedKeys,
-    selectedKeys: selectedKeys
+    selectedKeys: selectedKeys,
+    onOpenChange: onOpenChange
   }), function (item) {
     return ___EmotionJSX(Item, {
       key: item.key,
@@ -167,6 +257,17 @@ export var Controlled = function Controlled(args) {
   }));
 };
 export var WithCustomSize = function WithCustomSize(args) {
+  var _useState13 = useState(false),
+      _useState14 = _slicedToArray(_useState13, 2),
+      isOpen = _useState14[0],
+      setIsOpen = _useState14[1];
+
+  var direction = args.direction;
+
+  var onOpenChange = function onOpenChange() {
+    setIsOpen(true);
+  };
+
   return ___EmotionJSX(Box, {
     sx: {
       width: '100%',
@@ -175,9 +276,14 @@ export var WithCustomSize = function WithCustomSize(args) {
     }
   }, ___EmotionJSX(Box, {
     width: 300
-  }, ___EmotionJSX(OverlayProvider, null, ___EmotionJSX(MultivaluesField, _extends({
+  }, ___EmotionJSX(OverlayProvider // note: spacing for demo purpose only so that the select list renders in the correct place
+  , {
+    style: setOverlayStyle(direction, isOpen, '25%', '25%', '75%')
+  }, ___EmotionJSX(MultivaluesField, _extends({
     items: items
-  }, args), function (item) {
+  }, args, {
+    onOpenChange: onOpenChange
+  }), function (item) {
     return ___EmotionJSX(Item, {
       key: item.key,
       "data-id": item.name
@@ -185,10 +291,26 @@ export var WithCustomSize = function WithCustomSize(args) {
   }))));
 };
 export var WithReadOnlyValues = function WithReadOnlyValues(args) {
-  return ___EmotionJSX(OverlayProvider, null, ___EmotionJSX(MultivaluesField, _extends({
+  var _useState15 = useState(false),
+      _useState16 = _slicedToArray(_useState15, 2),
+      isOpen = _useState16[0],
+      setIsOpen = _useState16[1];
+
+  var direction = args.direction;
+
+  var onOpenChange = function onOpenChange() {
+    setIsOpen(true);
+  };
+
+  return ___EmotionJSX(OverlayProvider // note: spacing for demo purpose only so that the select list renders in the correct place
+  , {
+    style: setOverlayStyle(direction, isOpen, '50%', '50%', '20%')
+  }, ___EmotionJSX(MultivaluesField, _extends({
     readOnlyKeys: ['Aardvark', 'Snake'],
     items: items
-  }, args), function (item) {
+  }, args, {
+    onOpenChange: onOpenChange
+  }), function (item) {
     return ___EmotionJSX(Item, {
       key: item.key,
       "data-id": item.name

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pingux/astro",
-  "version": "1.16.1-alpha.0",
+  "version": "1.17.0-alpha.2",
   "description": "PingUX themeable React component library",
   "author": "ux-development@pingidentity.com",
   "license": "Apache-2.0",