@primer/components 0.0.0-202111023322 → 0.0.0-2021111104725

package/lib/theme.js

@@ -3,14 +3,11 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.colors = exports.default = void 0;
+exports.default = void 0;
 
-const themeExport = require('./theme-preval');
+var _themePreval = require("./theme-preval");
+
+var _default = _themePreval.theme; // NOTE: for now, ThemeColors and ThemeShadows are handcrafted types. It would be nice if these
+// were exports from primitives (or a different shape but derived from those exports).
 
-const {
-  theme,
-  colors
-} = themeExport;
-exports.colors = colors;
-var _default = theme;
 exports.default = _default;

package/lib/utils/create-slots.js

@@ -0,0 +1,105 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.default = void 0;
+
+var _react = _interopRequireDefault(require("react"));
+
+var _useForceUpdate = require("./use-force-update");
+
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+
+/** createSlots is a factory that can create a
+ *  typesafe Slots + Slot pair to use in a component definition
+ *  For example: ActionList.Item uses createSlots to get a Slots wrapper
+ *  + Slot component that is used by LeadingVisual, Description
+ */
+const createSlots = slotNames => {
+  const SlotsContext = /*#__PURE__*/_react.default.createContext({
+    registerSlot: () => null,
+    unregisterSlot: () => null,
+    context: {}
+  });
+  /** Slots uses a Double render strategy inspired by [reach-ui/descendants](https://github.com/reach/reach-ui/tree/develop/packages/descendants)
+   *  Slot registers themself with the Slots parent.
+   *  When all the children have mounted = registered themselves in slot,
+   *  we re-render the parent component to render with slots
+   */
+
+
+  const Slots = ({
+    context = {},
+    children
+  }) => {
+    // initialise slots
+    const slotsDefinition = {};
+    slotNames.map(name => slotsDefinition[name] = null);
+
+    const slotsRef = _react.default.useRef(slotsDefinition);
+
+    const rerenderWithSlots = (0, _useForceUpdate.useForceUpdate)();
+
+    const [isMounted, setIsMounted] = _react.default.useState(false); // fires after all the effects in children
+
+
+    _react.default.useEffect(() => {
+      rerenderWithSlots();
+      setIsMounted(true);
+    }, [rerenderWithSlots]);
+
+    const registerSlot = _react.default.useCallback((name, contents) => {
+      slotsRef.current[name] = contents; // don't render until the component mounts = all slots are registered
+
+      if (isMounted) rerenderWithSlots();
+    }, [isMounted, rerenderWithSlots]); // Slot can be removed from the tree as well,
+    // we need to unregister them from the slot
+
+
+    const unregisterSlot = _react.default.useCallback(name => {
+      slotsRef.current[name] = null;
+      rerenderWithSlots();
+    }, [rerenderWithSlots]);
+    /**
+     * Slots uses a render prop API so abstract the
+     * implementation detail of using a context provider.
+     */
+
+
+    const slots = slotsRef.current;
+    return /*#__PURE__*/_react.default.createElement(SlotsContext.Provider, {
+      value: {
+        registerSlot,
+        unregisterSlot,
+        context
+      }
+    }, children(slots));
+  };
+
+  const Slot = ({
+    name,
+    children
+  }) => {
+    const {
+      registerSlot,
+      unregisterSlot,
+      context
+    } = _react.default.useContext(SlotsContext);
+
+    _react.default.useEffect(() => {
+      registerSlot(name, typeof children === 'function' ? children(context) : children);
+      return () => unregisterSlot(name);
+    }, [name, children, registerSlot, unregisterSlot, context]);
+
+    return null;
+  };
+
+  return {
+    Slots,
+    Slot
+  };
+};
+
+var _default = createSlots;
+exports.default = _default;

package/lib/utils/deprecate.js

@@ -7,11 +7,11 @@ exports.Deprecations = exports.useDeprecation = exports.deprecate = void 0;
 
 var _react = require("react");
 
-/* eslint-disable no-console */
+// eslint-disable-next-line @typescript-eslint/no-empty-function
 const noop = () => {}; // eslint-disable-next-line import/no-mutable-exports
 
 
-let deprecate = null;
+let deprecate = noop;
 exports.deprecate = deprecate;
 
 if (process.env.NODE_ENV !== 'production') {
@@ -26,8 +26,6 @@ if (process.env.NODE_ENV !== 'production') {
       version
     });
   };
-} else {
-  exports.deprecate = deprecate = noop;
 }
 
 // eslint-disable-next-line import/no-mutable-exports
@@ -60,6 +58,8 @@ if (process.env.NODE_ENV !== 'production') {
 }
 
 class Deprecations {
+  static instance = null;
+
   static get() {
     if (!Deprecations.instance) {
       Deprecations.instance = new Deprecations();
@@ -77,7 +77,8 @@ class Deprecations {
     message,
     version
   }) {
-    const msg = `WARNING! ${name} is deprecated and will be removed in version ${version}. ${message}`;
+    const msg = `WARNING! ${name} is deprecated and will be removed in version ${version}. ${message}`; // eslint-disable-next-line no-console
+
     console.warn(msg);
     this.get().deprecations.push({
       name,

package/lib/utils/isNumeric.js

@@ -5,6 +5,7 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = isNumeric;
 
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
 function isNumeric(n) {
   return !isNaN(parseFloat(n)) && isFinite(n);
 }

package/lib/utils/iterateFocusableElements.js

@@ -0,0 +1,113 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.iterateFocusableElements = iterateFocusableElements;
+exports.isFocusable = isFocusable;
+exports.isTabbable = isTabbable;
+
+/**
+ * Options to the focusable elements iterator
+ */
+
+/**
+ * Returns an iterator over all of the focusable elements within `container`.
+ * Note: If `container` is itself focusable it will be included in the results.
+ * @param container The container over which to find focusable elements.
+ * @param reverse If true, iterate backwards through focusable elements.
+ */
+function* iterateFocusableElements(container, options = {}) {
+  var _options$strict, _options$onlyTabbable;
+
+  const strict = (_options$strict = options.strict) !== null && _options$strict !== void 0 ? _options$strict : false;
+  const acceptFn = ((_options$onlyTabbable = options.onlyTabbable) !== null && _options$onlyTabbable !== void 0 ? _options$onlyTabbable : false) ? isTabbable : isFocusable;
+  const walker = document.createTreeWalker(container, NodeFilter.SHOW_ELEMENT, {
+    acceptNode: node => node instanceof HTMLElement && acceptFn(node, strict) ? NodeFilter.FILTER_ACCEPT : NodeFilter.FILTER_SKIP
+  });
+  let nextNode = null; // Allow the container to participate
+
+  if (!options.reverse && acceptFn(container, strict)) {
+    yield container;
+  } // If iterating in reverse, continue traversing down into the last child until we reach
+  // a leaf DOM node
+
+
+  if (options.reverse) {
+    let lastChild = walker.lastChild();
+
+    while (lastChild) {
+      nextNode = lastChild;
+      lastChild = walker.lastChild();
+    }
+  } else {
+    nextNode = walker.firstChild();
+  }
+
+  while (nextNode instanceof HTMLElement) {
+    yield nextNode;
+    nextNode = options.reverse ? walker.previousNode() : walker.nextNode();
+  } // Allow the container to participate (in reverse)
+
+
+  if (options.reverse && acceptFn(container, strict)) {
+    yield container;
+  }
+
+  return undefined;
+}
+/**
+ * Determines whether the given element is focusable. If `strict` is true, we may
+ * perform additional checks that require a reflow (less performant).
+ * @param elem
+ * @param strict
+ */
+
+
+function isFocusable(elem, strict = false) {
+  // Certain conditions cause an element to never be focusable, even if they have tabindex="0"
+  const disabledAttrInert = ['BUTTON', 'INPUT', 'SELECT', 'TEXTAREA', 'OPTGROUP', 'OPTION', 'FIELDSET'].includes(elem.tagName) && elem.disabled;
+  const hiddenInert = elem.hidden;
+  const hiddenInputInert = elem instanceof HTMLInputElement && elem.type === 'hidden';
+
+  if (disabledAttrInert || hiddenInert || hiddenInputInert) {
+    return false;
+  } // Each of the conditions checked below require a reflow, thus are gated by the `strict`
+  // argument. If any are true, the element is not focusable, even if tabindex is set.
+
+
+  if (strict) {
+    const sizeInert = elem.offsetWidth === 0 || elem.offsetHeight === 0;
+    const visibilityInert = ['hidden', 'collapse'].includes(getComputedStyle(elem).visibility);
+    const clientRectsInert = elem.getClientRects().length === 0;
+
+    if (sizeInert || visibilityInert || clientRectsInert) {
+      return false;
+    }
+  } // Any element with `tabindex` explicitly set can be focusable, even if it's set to "-1"
+
+
+  if (elem.getAttribute('tabindex') != null) {
+    return true;
+  } // One last way `elem.tabIndex` can be wrong.
+
+
+  if (elem instanceof HTMLAnchorElement && elem.getAttribute('href') == null) {
+    return false;
+  }
+
+  return elem.tabIndex !== -1;
+}
+/**
+ * Determines whether the given element is tabbable. If `strict` is true, we may
+ * perform additional checks that require a reflow (less performant). This check
+ * ensures that the element is focusable and that its tabindex is not explicitly
+ * set to "-1" (which makes it focusable, but removes it from the tab order).
+ * @param elem
+ * @param strict
+ */
+
+
+function isTabbable(elem, strict = false) {
+  return isFocusable(elem, strict) && elem.getAttribute('tabindex') !== '-1';
+}

package/lib/utils/ssr.js

@@ -0,0 +1,19 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+Object.defineProperty(exports, "SSRProvider", {
+  enumerable: true,
+  get: function () {
+    return _ssr.SSRProvider;
+  }
+});
+Object.defineProperty(exports, "useSSRSafeId", {
+  enumerable: true,
+  get: function () {
+    return _ssr.useSSRSafeId;
+  }
+});
+
+var _ssr = require("@react-aria/ssr");

package/lib/utils/test-deprecations.js

@@ -2,10 +2,11 @@
 
 var _semver = _interopRequireDefault(require("semver"));
 
-var _deprecate = require("../utils/deprecate");
+var _deprecate = require("./deprecate");
 
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
+// eslint-disable-next-line @typescript-eslint/no-var-requires
 const ourVersion = require('../../package.json').version;
 
 beforeEach(() => {

package/lib/utils/test-helpers.js

@@ -0,0 +1,9 @@
+"use strict";
+
+// JSDOM doesn't mock ResizeObserver
+global.ResizeObserver = jest.fn().mockImplementation(() => {
+  return {
+    observe: jest.fn(),
+    disconnect: jest.fn()
+  };
+});

package/lib/utils/test-matchers.js

@@ -1,12 +1,10 @@
 "use strict";
 
-var _react = _interopRequireDefault(require("react"));
-
 require("jest-styled-components");
 
 var _serializer = require("jest-styled-components/serializer");
 
-var _theme = _interopRequireDefault(require("../theme"));
+var _react = _interopRequireDefault(require("react"));
 
 var _testing = require("./testing");
 
@@ -15,14 +13,7 @@ function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { de
 expect.addSnapshotSerializer(_serializer.styleSheetSerializer);
 
 const stringify = d => JSON.stringify(d, null, '  ');
-/**
- * These are props that styled-system aliases for backwards compatibility.
- * For some reason, they don't show up in our toImplementSystemProps() matcher,
- * so we skip over them.
- */
 
-
-const ALIAS_PROP_TYPES = ['w', 'align', 'justify', 'wrap'];
 expect.extend({
   toMatchKeys(obj, values) {
     return {
@@ -49,23 +40,6 @@ expect.extend({
     };
   },
 
-  toImplementSystemProps(Component, propNames) {
-    const propKeys = new Set(Object.keys(Component.propTypes));
-    const expectedPropKeys = Object.keys(propNames.propTypes);
-    const missing = expectedPropKeys.filter(key => !propKeys.has(key)).filter(key => !ALIAS_PROP_TYPES.includes(key));
-    return {
-      pass: missing.length === 0,
-      message: () => `Missing prop${missing.length === 1 ? '' : 's'}: ${stringify(missing)}`
-    };
-  },
-
-  toImplementSxProp(Component) {
-    return {
-      pass: !!Component.propTypes.sx,
-      message: () => 'Missing sx propTypes'
-    };
-  },
-
   toImplementSxBehavior(element) {
     const mediaKey = '@media (max-width:123px)';
     const sxPropValue = {
@@ -74,17 +48,16 @@ expect.extend({
       }
     };
 
-    const elem = _react.default.cloneElement(element, {
+    const elem = /*#__PURE__*/_react.default.cloneElement(element, {
       sx: sxPropValue
     });
 
-    const rendered = (0, _testing.render)(elem);
-
     function checkStylesDeep(rendered) {
-      const className = rendered.props ? rendered.props.className : '';
+      const className = rendered.props.className;
       const styles = (0, _testing.getComputedStyles)(className);
+      const mediaStyles = styles[mediaKey];
 
-      if (styles[mediaKey] && styles[mediaKey].color) {
+      if (mediaStyles && mediaStyles.color) {
         return true;
       } else if (rendered.children) {
         return rendered.children.some(child => checkStylesDeep(child));
@@ -94,28 +67,11 @@ expect.extend({
     }
 
     return {
-      pass: checkStylesDeep(rendered),
+      pass: checkStylesDeep((0, _testing.render)(elem)),
       message: () => 'sx prop values did not change styles of component nor of any sub-components'
     };
   },
 
-  toSetDefaultTheme(Component) {
-    let comp;
-
-    if (Component.type) {
-      comp = Component;
-    } else {
-      comp = /*#__PURE__*/_react.default.createElement(Component, null);
-    }
-
-    const wrapper = (0, _testing.mount)(comp);
-    const pass = this.equals(wrapper.prop('theme'), _theme.default);
-    return {
-      pass,
-      message: () => 'default theme is not set'
-    };
-  },
-
   toSetExports(mod, expectedExports) {
     if (!Object.keys(expectedExports).includes('default')) {
       return {

package/lib/utils/testing.js

@@ -8,7 +8,6 @@ exports.render = render;
 exports.renderRoot = renderRoot;
 exports.renderClasses = renderClasses;
 exports.rendersClass = rendersClass;
-exports.renderWithTheme = renderWithTheme;
 exports.px = px;
 exports.percent = percent;
 exports.renderStyles = renderStyles;
@@ -20,6 +19,7 @@ exports.loadCSS = loadCSS;
 exports.unloadCSS = unloadCSS;
 exports.behavesAsComponent = behavesAsComponent;
 exports.checkExports = checkExports;
+exports.checkStoriesForAxeViolations = checkStoriesForAxeViolations;
 exports.COMPONENT_DISPLAY_NAME_REGEX = void 0;
 
 var _react = _interopRequireDefault(require("react"));
@@ -32,12 +32,17 @@ var _enzyme = _interopRequireDefault(require("enzyme"));
 
 var _enzymeAdapterReact = _interopRequireDefault(require("@wojtekmaj/enzyme-adapter-react-17"));
 
-var _styledComponents = require("styled-components");
+var _react2 = require("@testing-library/react");
+
+var _jestAxe = require("jest-axe");
+
+var _ = require("..");
 
 var _theme = _interopRequireDefault(require("../theme"));
 
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
+// eslint-disable-next-line @typescript-eslint/no-var-requires
 const readFile = (0, _util.promisify)(require('fs').readFile);
 const COMPONENT_DISPLAY_NAME_REGEX = /^[A-Z][A-Za-z]+(\.[A-Z][A-Za-z]+)*$/;
 exports.COMPONENT_DISPLAY_NAME_REGEX = COMPONENT_DISPLAY_NAME_REGEX;
@@ -49,6 +54,7 @@ _enzyme.default.configure({
 function mount(component) {
   return _enzyme.default.mount(component);
 }
+
 /**
  * Render the component (a React.createElement() or JSX expression)
  * into its intermediate object representation with 'type',
@@ -60,10 +66,10 @@ function mount(component) {
  * expect(render(<Foo />)).toEqual(render(<div foo='bar' />))
  * ```
  */
-
-
-function render(component) {
-  return _reactTestRenderer.default.create(component).toJSON();
+function render(component, theme = _theme.default) {
+  return _reactTestRenderer.default.create( /*#__PURE__*/_react.default.createElement(_.ThemeProvider, {
+    theme: theme
+  }, component)).toJSON();
 }
 /**
  * Render the component (a React.createElement() or JSX expression)
@@ -103,12 +109,6 @@ function rendersClass(node, klass) {
   return renderClasses(node).includes(klass);
 }
 
-function renderWithTheme(node, theme = _theme.default) {
-  return render( /*#__PURE__*/_react.default.createElement(_styledComponents.ThemeProvider, {
-    theme: theme
-  }, node));
-}
-
 function px(value) {
   return typeof value === 'number' ? `${value}px` : value;
 }
@@ -132,11 +132,12 @@ function getComputedStyles(className) {
   const computed = {};
 
   for (const sheet of document.styleSheets) {
+    // CSSRulesLists assumes every rule is a CSSRule, not a CSSStyleRule
     for (const rule of sheet.cssRules) {
-      if (rule.type === 1) {
-        readRule(rule, computed);
-      } else if (rule.type === 4) {
+      if (rule instanceof CSSMediaRule) {
         readMedia(rule);
+      } else if (rule instanceof CSSStyleRule) {
+        readRule(rule, computed);
       } else {// console.warn('rule.type =', rule.type)
       }
     }
@@ -144,6 +145,18 @@ function getComputedStyles(className) {
 
   return computed;
 
+  function matchesSafe(node, selector) {
+    if (!selector) {
+      return false;
+    }
+
+    try {
+      return node.matches(selector);
+    } catch (error) {
+      return false;
+    }
+  }
+
   function readRule(rule, dest) {
     if (matchesSafe(div, rule.selectorText)) {
       const {
@@ -164,7 +177,9 @@ function getComputedStyles(className) {
     const dest = {};
 
     for (const rule of mediaRule.cssRules) {
-      readRule(rule, dest);
+      if (rule instanceof CSSStyleRule) {
+        readRule(rule, dest);
+      }
     } // Don't add media rule to computed styles
     // if no styles were actually applied
 
@@ -173,18 +188,6 @@ function getComputedStyles(className) {
       computed[key] = dest;
     }
   }
-
-  function matchesSafe(node, selector) {
-    if (!selector) {
-      return false;
-    }
-
-    try {
-      return node.matches(selector);
-    } catch (error) {
-      return false;
-    }
-  }
 }
 /**
  * This provides a layer of compatibility between the render() function from
@@ -226,26 +229,16 @@ function unloadCSS(path) {
 // returns an element ready to be rendered.
 
 
-function behavesAsComponent(Component, systemPropArray, toRender = null, options) {
-  if (typeof toRender === 'object' && !options) {
-    options = toRender;
-    toRender = null;
-  }
-
+function behavesAsComponent({
+  Component,
+  toRender,
+  options
+}) {
   options = options || {};
 
   const getElement = () => toRender ? toRender() : /*#__PURE__*/_react.default.createElement(Component, null);
 
-  it('implements system props', () => {
-    for (const systemProps of systemPropArray) {
-      expect(Component).toImplementSystemProps(systemProps);
-    }
-  });
-
   if (!options.skipSx) {
-    it('implements the sx prop', () => {
-      expect(Component).toImplementSxProp();
-    });
     it('implements sx prop behavior', () => {
       expect(getElement()).toImplementSxBehavior();
     });
@@ -253,12 +246,12 @@ function behavesAsComponent(Component, systemPropArray, toRender = null, options
 
   if (!options.skipAs) {
     it('respects the as prop', () => {
-      const As = _react.default.forwardRef((_props, ref) => /*#__PURE__*/_react.default.createElement("div", {
+      const As = /*#__PURE__*/_react.default.forwardRef((_props, ref) => /*#__PURE__*/_react.default.createElement("div", {
         className: "as-component",
         ref: ref
       }));
 
-      const elem = _react.default.cloneElement(getElement(), {
+      const elem = /*#__PURE__*/_react.default.cloneElement(getElement(), {
         as: As
       });
 
@@ -269,18 +262,41 @@ function behavesAsComponent(Component, systemPropArray, toRender = null, options
   it('sets a valid displayName', () => {
     expect(Component.displayName).toMatch(COMPONENT_DISPLAY_NAME_REGEX);
   });
-  it('sets the default theme', () => {
-    expect(getElement()).toSetDefaultTheme();
-  });
   it('renders consistently', () => {
     expect(render(getElement())).toMatchSnapshot();
   });
-}
+} // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
 
 function checkExports(path, exports) {
   it('has declared exports', () => {
+    // eslint-disable-next-line @typescript-eslint/no-var-requires
     const mod = require(`../${path}`);
 
     expect(mod).toSetExports(exports);
   });
+}
+
+expect.extend(_jestAxe.toHaveNoViolations);
+
+function checkStoriesForAxeViolations(name) {
+  // eslint-disable-next-line @typescript-eslint/no-var-requires
+  const stories = require(`../stories/${name}.stories`); // eslint-disable-next-line @typescript-eslint/no-unused-vars -- _meta
+
+
+  const {
+    default: _meta,
+    ...Stories
+  } = stories;
+  Object.values(Stories).map(Story => {
+    if (typeof Story !== 'function') return;
+    it(`story ${Story.storyName} should have no axe violations`, async () => {
+      const {
+        container
+      } = (0, _react2.render)( /*#__PURE__*/_react.default.createElement(Story, null));
+      const results = await (0, _jestAxe.axe)(container);
+      expect(results).toHaveNoViolations();
+      (0, _react2.cleanup)();
+    });
+  });
 }