@pingux/astro 1.0.0-alpha.9 → 1.1.0-alpha.10

package/lib/components/Chip/Chip.js

@@ -1,6 +1,7 @@
 import _extends from "@babel/runtime-corejs3/helpers/esm/extends";
 import React from 'react';
 import PropTypes from 'prop-types';
+import { ChipContext } from './ChipContext';
 import Box from '../Box/Box';
 import Text from '../Text/Text';
 import * as colors from '../../styles/colors';
@@ -12,21 +13,31 @@ import * as colors from '../../styles/colors';
 
 import { jsx as ___EmotionJSX } from "@emotion/react";
 var Chip = /*#__PURE__*/React.forwardRef(function (props, ref) {
-  var children = props.children,
+  var bg = props.bg,
+      children = props.children,
       textColor = props.textColor,
       textProps = props.textProps,
-      label = props.label;
-  return ___EmotionJSX(Box, _extends({
+      label = props.label,
+      isUppercase = props.isUppercase;
+  return ___EmotionJSX(ChipContext.Provider, {
+    value: {
+      bg: bg
+    }
+  }, ___EmotionJSX(Box, _extends({
     isRow: true,
     variant: "boxes.chip",
+    sx: isUppercase && {
+      paddingBottom: '3px'
+    },
     ref: ref
   }, props), ___EmotionJSX(Text, _extends({
     variant: "label",
-    sx: {
-      textTransform: 'uppercase'
-    },
-    color: textColor
-  }, textProps), label), children);
+    color: textColor,
+    sx: isUppercase && {
+      textTransform: 'uppercase',
+      fontSize: '11px'
+    }
+  }, textProps), label), children));
 });
 Chip.propTypes = {
   /** The text color of the chip. */
@@ -39,10 +50,14 @@ Chip.propTypes = {
   label: PropTypes.oneOfType([PropTypes.string, PropTypes.object]),
 
   /** Props object that is spread directly into the textfield. */
-  textProps: PropTypes.shape({})
+  textProps: PropTypes.shape({}),
+
+  /** When true, display chip label as uppercase. */
+  isUppercase: PropTypes.bool
 };
 Chip.defaultProps = {
   textColor: 'white',
-  bg: colors.neutral[10]
+  bg: colors.neutral[10],
+  isUppercase: false
 };
 export default Chip;

package/lib/components/Chip/Chip.stories.js

@@ -3,8 +3,11 @@ import _slicedToArray from "@babel/runtime-corejs3/helpers/esm/slicedToArray";
 import _mapInstanceProperty from "@babel/runtime-corejs3/core-js-stable/instance/map";
 import React from 'react';
 import Clear from 'mdi-react/CloseIcon';
-import Chip from '../Chip/Chip';
-import Icon from '../Icon/Icon';
+import ContentCopy from 'mdi-react/ContentCopyIcon';
+import Earth from 'mdi-react/EarthIcon';
+import Chip from '../Chip';
+import Icon from '../Icon';
+import IconButton from '../IconButton';
 import { flatColorList } from '../../styles/colors.js';
 import { jsx as ___EmotionJSX } from "@emotion/react";
 export default {
@@ -40,6 +43,12 @@ export default {
       control: {
         type: 'text'
       }
+    },
+    isUppercase: {
+      defaultValue: false,
+      control: {
+        type: 'boolean'
+      }
     }
   }
 };
@@ -59,13 +68,40 @@ export var ChipWithCustomColors = function ChipWithCustomColors() {
   });
 };
 export var ChipWithIcon = function ChipWithIcon() {
-  return ___EmotionJSX(Chip, {
-    label: "Chip with Icon",
+  return ___EmotionJSX(React.Fragment, null, ___EmotionJSX(Chip, {
+    label: "Chip with Icon Button",
     bg: "navy"
+  }, ___EmotionJSX(IconButton, {
+    "aria-label": "Clear Chip with Icon Button",
+    variant: "inverted"
   }, ___EmotionJSX(Icon, {
     icon: Clear,
     ml: "xs",
+    size: "14px"
+  }))), ___EmotionJSX("div", {
+    style: {
+      padding: '5px'
+    }
+  }), ___EmotionJSX(Chip, {
+    label: "Chip with Icon Button"
+  }, ___EmotionJSX(IconButton, {
+    "aria-label": "Clear Chip with Icon Button",
+    variant: "inverted"
+  }, ___EmotionJSX(Icon, {
+    icon: Earth,
+    ml: "xs",
+    size: "14px"
+  }))), ___EmotionJSX("div", {
+    style: {
+      padding: '5px'
+    }
+  }), ___EmotionJSX(Chip, {
+    label: "Chip with Icon",
+    bg: "green"
+  }, ___EmotionJSX(Icon, {
+    icon: ContentCopy,
+    ml: "xs",
     size: "14px",
     color: "white"
-  }));
+  })));
 };

package/lib/components/Chip/Chip.test.js

@@ -31,4 +31,13 @@ test('renders children within Chip component', function () {
   });
   var mockedChildren = screen.getByRole('button');
   expect(mockedChildren).toBeInTheDocument();
+});
+test('renders Chip component with uppercase', function () {
+  var label = 'uppercase';
+  var isUppercase = true;
+  getComponent({
+    label: label,
+    isUppercase: isUppercase
+  });
+  expect(screen.queryByText('uppercase')).toHaveStyleRule('text-transform', 'uppercase');
 });

package/lib/components/Chip/ChipContext.js

@@ -0,0 +1,3 @@
+import React from 'react';
+var defaultValue = 'inherit';
+export var ChipContext = /*#__PURE__*/React.createContext(defaultValue);

package/lib/components/ColorField/ColorField.js

@@ -108,6 +108,7 @@ var ColorField = /*#__PURE__*/forwardRef(function (props, ref) {
     return _concatInstanceProperty(_context2 = _concatInstanceProperty(_context3 = _concatInstanceProperty(_context4 = "rgba(".concat(colorValue === null || colorValue === void 0 ? void 0 : colorValue.red, ", ")).call(_context4, colorValue === null || colorValue === void 0 ? void 0 : colorValue.green, ", ")).call(_context3, colorValue === null || colorValue === void 0 ? void 0 : colorValue.blue, ", ")).call(_context2, colorValue === null || colorValue === void 0 ? void 0 : colorValue.alpha, ")");
   }, []);
   return ___EmotionJSX(Box, fieldContainerProps, label && ___EmotionJSX(Label, fieldLabelProps), ___EmotionJSX(Button, _extends({
+    "aria-label": "Select color",
     bg: getRgbaFromState(state),
     onPress: handleButtonPress,
     ref: triggerRef,

package/lib/components/ComboBoxField/ComboBoxField.js

@@ -116,6 +116,12 @@ var ComboBoxField = /*#__PURE__*/forwardRef(function (props, ref) {
   useImperativeHandle(ref, function () {
     return inputRef.current;
   });
+  /* istanbul ignore next */
+
+  var onSelectionChangeHandler = function onSelectionChangeHandler(key) {
+    var newVal = key || selectedKey || '';
+    if (onSelectionChange) onSelectionChange(newVal);
+  };
 
   var _useFilter = useFilter({
     sensitivity: 'base'
@@ -123,6 +129,7 @@ var ComboBoxField = /*#__PURE__*/forwardRef(function (props, ref) {
       contains = _useFilter.contains;
 
   var state = useComboBoxState(_objectSpread(_objectSpread({}, comboBoxOptions), {}, {
+    onSelectionChange: hasCustomValue ? onSelectionChangeHandler : onSelectionChange,
     defaultFilter: contains
   }));
 

package/lib/components/ComboBoxField/ComboBoxField.stories.js

@@ -330,6 +330,27 @@ export var ControlledFiltering = function ControlledFiltering() {
     }, item.name);
   }));
 };
+export var ControlledWithCustomValue = function ControlledWithCustomValue() {
+  var _useState9 = useState(''),
+      _useState10 = _slicedToArray(_useState9, 2),
+      inputValue = _useState10[0],
+      setInputValue = _useState10[1];
+
+  return ___EmotionJSX(OverlayProvider, null, ___EmotionJSX(ComboBoxField, _extends({
+    label: "Example label",
+    defaultItems: items
+  }, actions, {
+    inputValue: inputValue,
+    selectedKey: inputValue,
+    onInputChange: setInputValue,
+    onSelectionChange: setInputValue,
+    hasCustomValue: true
+  }), function (item) {
+    return ___EmotionJSX(Item, {
+      key: item.name
+    }, item.name);
+  }));
+};
 export var AllowCustomValue = function AllowCustomValue() {
   return ___EmotionJSX(OverlayProvider, null, ___EmotionJSX(ComboBoxField, _extends({
     label: "Example label",

package/lib/components/ComboBoxField/ComboBoxField.test.js

@@ -639,6 +639,41 @@ test('two listbox can not be open at the same time', function () {
     name: 'Tango'
   })).toBeInTheDocument();
 });
+test('should handle selecting custom option', function () {
+  getComponent({
+    hasCustomValue: true
+  });
+  var input = screen.queryByRole('combobox');
+  expect(input).toHaveValue(''); // type something
+
+  userEvent.type(input, 'custom'); // set input value as selected
+
+  userEvent.type(input, '{enter}', {
+    skipClick: true
+  });
+  expect(screen.queryByRole('listbox')).not.toBeInTheDocument();
+  expect(screen.queryByRole('combobox')).toHaveValue('custom'); // blur input
+
+  userEvent.tab();
+  expect(input).toHaveValue('custom');
+});
+test('onSelectionChange works properly with custom value', function () {
+  var onSelectionChange = jest.fn();
+  getComponent({
+    hasCustomValue: true,
+    onSelectionChange: onSelectionChange,
+    onInputChange: onSelectionChange
+  });
+  var input = screen.queryByRole('combobox');
+  expect(input).toHaveValue('');
+  expect(onSelectionChange).not.toHaveBeenCalled(); // Should fire when input value was typed, and enter was pressed
+
+  userEvent.type(input, 'custom{enter}');
+  expect(onSelectionChange).toHaveBeenCalledWith('custom'); // Should fire when input is cleared
+
+  userEvent.type(input, '{selectall}{backspace}{enter}');
+  expect(onSelectionChange).toHaveBeenCalledWith('');
+});
 test('should have no accessibility violations', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee2() {
   var _getComponent6, container, results;
 

package/lib/components/FileInputField/FileItem.js

@@ -37,7 +37,8 @@ var FileItem = function FileItem(props) {
 
       default:
         return {
-          icon: InsertDriveFileIcon
+          icon: InsertDriveFileIcon,
+          color: 'neutral.10'
         };
     }
   }, [status]);

package/lib/components/IconButton/IconButton.js

@@ -14,13 +14,15 @@ function ownKeys(object, enumerableOnly) { var keys = _Object$keys(object); if (
 
 function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var source = arguments[i] != null ? arguments[i] : {}; if (i % 2) { var _context; _forEachInstanceProperty(_context = ownKeys(Object(source), true)).call(_context, function (key) { _defineProperty(target, key, source[key]); }); } else if (_Object$getOwnPropertyDescriptors) { _Object$defineProperties(target, _Object$getOwnPropertyDescriptors(source)); } else { var _context2; _forEachInstanceProperty(_context2 = ownKeys(Object(source))).call(_context2, function (key) { _Object$defineProperty(target, key, _Object$getOwnPropertyDescriptor(source, key)); }); } } return target; }
 
-import React, { forwardRef, useRef, useImperativeHandle } from 'react';
+import React, { forwardRef, useRef, useImperativeHandle, useContext } from 'react';
 import PropTypes from 'prop-types';
 import { IconButton as ThemeUIIconButton } from 'theme-ui';
+import { useButton } from '@react-aria/button';
 import { useFocusRing } from '@react-aria/focus';
-import { Pressable, useHover, usePress } from '@react-aria/interactions';
+import { Pressable, useHover } from '@react-aria/interactions';
 import { mergeProps } from '@react-aria/utils';
 import { useAriaLabelWarning, useStatusClasses } from '../../hooks';
+import { ChipContext } from '../Chip/ChipContext';
 import TooltipTrigger, { Tooltip } from '../TooltipTrigger';
 /**
  * Convenience wrapper for a Button + Icon. This component applies specific styles necessary for
@@ -51,11 +53,12 @@ var IconButton = /*#__PURE__*/forwardRef(function (props, ref) {
     return buttonRef.current;
   });
 
-  var _usePress = usePress(_objectSpread({
-    ref: buttonRef
-  }, props)),
-      isPressed = _usePress.isPressed,
-      pressProps = _usePress.pressProps;
+  var _useButton = useButton(_objectSpread({}, props), buttonRef),
+      buttonProps = _useButton.buttonProps,
+      isPressed = _useButton.isPressed;
+
+  var _useContext = useContext(ChipContext),
+      chipBg = _useContext.bg;
 
   var _useHover = useHover(props),
       hoverProps = _useHover.hoverProps,
@@ -80,8 +83,13 @@ var IconButton = /*#__PURE__*/forwardRef(function (props, ref) {
     tabIndex: 0,
     ref: buttonRef,
     className: classNames,
-    "aria-label": ariaLabel || 'Icon Button'
-  }, others, mergeProps(hoverProps, focusProps, pressProps)), children);
+    "aria-label": ariaLabel,
+    sx: chipBg && isHovered && {
+      'path': {
+        fill: chipBg
+      }
+    }
+  }, mergeProps(hoverProps, focusProps, buttonProps, others)), children);
 
   if (title) {
     return ___EmotionJSX(TooltipTrigger, {

package/lib/components/IconButton/IconButton.stories.js

@@ -35,29 +35,23 @@ export default {
     isDisabled: {},
     variant: {
       control: {
-        type: 'none'
-      }
+        type: 'select',
+        options: ['iconButton', 'inverted', 'square', 'invertedSquare']
+      },
+      defaultValue: 'iconButton'
     }
   }
 };
 export var Default = function Default(args) {
   return ___EmotionJSX(IconButton, _extends({
-    "aria-label": "my-label"
+    "aria-label": "default icon button"
   }, args), ___EmotionJSX(Icon, {
     icon: CreateIcon
   }));
 };
-export var Inverted = function Inverted() {
-  return ___EmotionJSX(IconButton, {
-    "aria-label": "my-label",
-    variant: "inverted"
-  }, ___EmotionJSX(Icon, {
-    icon: CreateIcon
-  }));
-};
 export var WithTooltip = function WithTooltip() {
   return ___EmotionJSX(IconButton, {
-    "aria-label": "my-label",
+    "aria-label": "icon button with tooltip",
     title: "Edit"
   }, ___EmotionJSX(Icon, {
     icon: CreateIcon
@@ -65,7 +59,7 @@ export var WithTooltip = function WithTooltip() {
 };
 export var Disabled = function Disabled() {
   return ___EmotionJSX(IconButton, {
-    "aria-label": "my-label",
+    "aria-label": "disabled icon button",
     isDisabled: true
   }, ___EmotionJSX(Icon, {
     icon: CreateIcon

package/lib/components/IconButton/IconButton.test.js

@@ -17,7 +17,8 @@ var Icon = function Icon(props) {
 var testId = 'test-button';
 var defaultProps = {
   'data-testid': testId,
-  icon: Icon
+  icon: Icon,
+  'aria-label': 'Create'
 };
 
 var getComponent = function getComponent() {
@@ -32,7 +33,6 @@ test('default icon button', function () {
   var button = screen.getByRole('button');
   expect(button).toHaveAttribute('data-testid', testId);
   expect(button).toBeInstanceOf(HTMLButtonElement);
-  expect(button).toHaveAttribute('tabindex', '0');
   expect(button).toBeInTheDocument();
 });
 test('icon button hover', function () {
@@ -116,10 +116,8 @@ test('tooltip is not shown on hover or focus when prop is not passed', function
   expect(screen.queryByRole('tooltip')).not.toBeInTheDocument();
 });
 test('the button should be getting aria label attribute', function () {
-  var testLabel = 'test label';
-  getComponent({
-    'aria-label': testLabel
-  });
+  var testLabel = defaultProps['aria-label'];
+  getComponent();
   expect(screen.getByLabelText(testLabel)).toBeInTheDocument();
 });
 test('show button isDisabled status', function () {

package/lib/components/ImageUploadField/ImagePreviewButton.js

@@ -92,7 +92,8 @@ var ImagePreviewButton = /*#__PURE__*/forwardRef(function (props, ref) {
     ref: buttonRef,
     variant: "imageUpload",
     sx: widthHeightSx,
-    "data-testid": "image-preview-button"
+    "data-testid": "image-preview-button",
+    "aria-label": "Image preview"
   }, mergeProps(focusProps, others)), previewImage && isImageType ? imagePreview : noImagePreview, isLoading && loadingPreview, (isHovered || isFocusVisible) && !isLoading && hoveredPreview));
 });
 ImagePreviewButton.propTypes = {

package/lib/components/List/List.js

@@ -2,6 +2,7 @@ import _extends from "@babel/runtime-corejs3/helpers/esm/extends";
 import _objectWithoutProperties from "@babel/runtime-corejs3/helpers/esm/objectWithoutProperties";
 import React, { forwardRef } from 'react';
 import Box from '../Box/Box';
+import { useDeprecationWarning } from '../../hooks';
 /**
  * The intended use of List is to hold a collection of <ListItem/> components.
  * Accepts most of the styling props from [styled-system](https://styled-system.com/table).
@@ -12,6 +13,7 @@ var List = /*#__PURE__*/forwardRef(function (props, ref) {
   var children = props.children,
       others = _objectWithoutProperties(props, ["children"]);
 
+  useDeprecationWarning('The  List component will be deprecated in Astro-UI 2.0.0, use ListView instead.');
   return ___EmotionJSX(Box, _extends({
     ref: ref,
     role: "list",

package/lib/components/List/List.stories.js

@@ -3,10 +3,14 @@ import List from '.';
 import ListItem from '../ListItem';
 import Text from '../Text';
 import Separator from '../Separator';
+import withDeprecationWarning from '../../utils/devUtils/decorators/withDeprecationWarning';
 import { jsx as ___EmotionJSX } from "@emotion/react";
 export default {
-  title: 'List',
-  component: [List, ListItem]
+  title: 'Deprecated/List',
+  component: [List, ListItem],
+  decorators: [function (Story, context) {
+    return withDeprecationWarning(Story, context, 'The `List` component will be deprecated in Astro-UI 2.0.0, use `ListView` instead.');
+  }]
 };
 export var Default = function Default() {
   return ___EmotionJSX(React.Fragment, null, ___EmotionJSX(Text, {

package/lib/components/ListBox/ListBox.js

@@ -24,7 +24,6 @@ import { useCollator } from '@react-aria/i18n';
 import { ListLayout } from '@react-stately/layout';
 import { ListBoxContext } from './ListBoxContext';
 import { Option } from './index.js';
-import { useAriaLabelWarning } from '../../hooks';
 import { isIterableProp } from '../../utils/devUtils/props/isIterable';
 import Loader from '../Loader';
 import ListBoxSection from './ListBoxSection';
@@ -77,13 +76,12 @@ var ListBox = /*#__PURE__*/forwardRef(function (props, ref) {
       selectedKeys = props.selectedKeys,
       selectionMode = props.selectionMode,
       state = props.state,
+      ariaLabel = props['aria-label'],
       ariaLabelledby = props['aria-labelledby'],
       ariaDescribedby = props['aria-describedby'],
       ariaDetails = props['aria-details'],
-      others = _objectWithoutProperties(props, ["defaultSelectedKeys", "disabledKeys", "hasAutoFocus", "hasFocusWrap", "hasNoEmptySelection", "hasVirtualFocus", "id", "isLoading", "isFocusedOnHover", "isSelectedOnPressUp", "isVirtualized", "items", "keyboardDelegate", "label", "onLoadMore", "onScroll", "onSelectionChange", "renderEmptyState", "selectedKeys", "selectionMode", "state", "aria-labelledby", "aria-describedby", "aria-details"]);
+      others = _objectWithoutProperties(props, ["defaultSelectedKeys", "disabledKeys", "hasAutoFocus", "hasFocusWrap", "hasNoEmptySelection", "hasVirtualFocus", "id", "isLoading", "isFocusedOnHover", "isSelectedOnPressUp", "isVirtualized", "items", "keyboardDelegate", "label", "onLoadMore", "onScroll", "onSelectionChange", "renderEmptyState", "selectedKeys", "selectionMode", "state", "aria-label", "aria-labelledby", "aria-describedby", "aria-details"]);
 
-  var ariaLabel = props['aria-label'];
-  useAriaLabelWarning('ListBox', ariaLabel);
   var focusStrategy = state.focusStrategy; // Object matching React Aria API with all options
 
   var listBoxOptions = {
@@ -107,7 +105,7 @@ var ListBox = /*#__PURE__*/forwardRef(function (props, ref) {
     shouldFocusWrap: hasFocusWrap,
     shouldSelectOnPressUp: isSelectedOnPressUp,
     shouldUseVirtualFocus: hasVirtualFocus,
-    'aria-label': ariaLabel || 'ListBox',
+    'aria-label': ariaLabel,
     'aria-labelledby': ariaLabelledby,
     'aria-describedby': ariaDescribedby,
     'aria-details': ariaDetails

package/lib/components/ListBox/ListBox.test.js

@@ -29,11 +29,13 @@ var itemsWithSections = [{
 }];
 var defaultProps = {
   'data-testid': testId,
+  'aria-label': 'listbox',
   'aria-labelledby': 'label',
   items: items
 };
 var defaultWithSectionsProps = {
   'data-testid': testId,
+  'aria-label': 'listbox',
   'aria-labelledby': 'label',
   items: itemsWithSections
 };

package/lib/components/ListBox/Option.js

@@ -63,7 +63,13 @@ var Option = /*#__PURE__*/forwardRef(function (props, ref) {
     isSelected: isSelected
   }),
       classNames = _useStatusClasses.classNames;
+  /* Related to UIP-4992
+   * Need to remove these properties to avoid errors in the console on the external app.
+   * By the way, these properties return "undefined", so it shouldn't create issues */
 
+
+  delete optionProps.onPressStart;
+  delete optionProps.onPressUp;
   return ___EmotionJSX(Box, _extends({
     as: "li",
     isRow: true,

package/lib/components/ListItem/ListItem.stories.js

@@ -32,7 +32,6 @@ export var Default = function Default(args) {
     isRow: true,
     alignSelf: "center"
   }, ___EmotionJSX(IconButton, {
-    variant: "icon",
     size: 26
   }, ___EmotionJSX(Icon, {
     icon: MoreVertIcon,
@@ -64,7 +63,6 @@ export var WithSubtitle = function WithSubtitle(args) {
     isRow: true,
     alignSelf: "center"
   }, ___EmotionJSX(IconButton, {
-    variant: "icon",
     size: 26
   }, ___EmotionJSX(Icon, {
     icon: MoreVertIcon,

package/lib/components/ListView/ListView.js

@@ -37,18 +37,19 @@ export var collectionTypes = {
   PLACEHOLDER: 'placeholder'
 };
 export function useListLayout(state) {
+  var ROW_HEIGHT = 81;
   var collator = useCollator({
     usage: 'search',
     sensitivity: 'base'
   });
   var layout = useMemo(function () {
     return new ListLayout({
-      estimatedRowHeight: 81,
+      estimatedRowHeight: ROW_HEIGHT,
       estimatedHeadingHeight: 26,
       paddingRight: 4,
       paddingLeft: 4,
-      loaderHeight: 81,
-      placeholderHeight: 81,
+      loaderHeight: ROW_HEIGHT,
+      placeholderHeight: ROW_HEIGHT,
       collator: collator
     });
   }, [collator]);