@bundle-stats/ui 3.2.4 → 3.2.5-alpha.0

package/src/components/metric/metric.module.css

@@ -1,6 +1,9 @@
 .root {
   display: block;
-  letter-spacing: 0.05em;
+}
+
+.display {
+  vertical-align: middle;
 }
 
 .displayUnit {
@@ -8,7 +11,9 @@
 }
 
 .delta {
-  display: block;
+  display: inline;
+  vertical-align: middle;
+  margin: 0 0 0 var(--space-xxxsmall);
   font-size: var(--size-xsmall);
 }
 
@@ -23,8 +28,3 @@
   line-height: 1;
   margin-top: -2px;
 }
-
-.inline .delta {
-  display: inline-block;
-  margin: 0 0 0 var(--space-xxxsmall);
-}

package/src/components/metrics-table-search/metrics-table-search.module.css

@@ -5,6 +5,7 @@
 }
 
 .input {
+  border-radius: var(--radius-medium) 0 0 var(--radius-medium);
   height: var(--space-large);
   border-color: var(--color-outline);
 }
@@ -21,12 +22,12 @@
   cursor: pointer;
   border-radius: 50%;
   position: absolute;
-  right: 0;
+  right: 2px;
   top: 50%;
   transform: translateY(-50%);
-  padding: var(--space-xxsmall);
-  width: var(--space-large);
-  height: var(--space-large);
+  padding: var(--space-xxxsmall);
+  width: var(--space-medium);
+  height: var(--space-medium);
   background: radial-gradient(var(--color-background) 50%, transparent);
   color: var(--color-text-ultra-light);
   transition: var(--transition-in);

package/src/components/summary-item/summary-item.jsx

@@ -78,13 +78,13 @@ export const SummaryItem = ({
             value={current}
             formatter={metric.formatter}
             enhanced
-            inline
           >
             {showDelta && (
               <Delta
                 className={css.delta}
                 displayValue={runInfo.displayDeltaPercentage}
                 deltaType={runInfo.deltaType}
+                inverted
               />
             )}
           </Metric>

package/src/components/summary-item/summary-item.module.css

@@ -29,21 +29,14 @@
 .currentMetric {
   display: block;
   color: var(--color-heading);
-  font-size: var(--size-large);
+  font-size: var(--size-xlarge);
   line-height: var(--space-medium);
   white-space: nowrap;
   font-weight: bold;
 }
 
-.delta {
-  font-size: var(--size-small);
-  line-height: var(--space-medium);
-}
-
 .baselineMetric {
   color: var(--color-text-light);
-  font-size: var(--size-small);
-  line-height: var(--space-small);
 }
 
 /* Loading state */
@@ -57,6 +50,6 @@
 
 /* Size large */
 .large .currentMetric {
-  font-size: var(--size-xxxlarge);
+  font-size: var(--size-xxlarge);
   line-height: var(--space-medium);
 }

package/src/css/default.css

@@ -1,25 +1,31 @@
-:root {
-  --chart-height: 180px;
+@import url("~modern-css-reset/dist/reset.css");
+
+/** modern-css-reset updates */
+pre,
+ul,
+ol,
+button {
+  margin: 0;
 }
 
 html {
+  font-family: var(--font-family);
   font-size: 81.25%;
-  line-height: 1.8461;
 }
 
 body {
+  background: var(--color-background);
   color: var(--color-text);
+  line-height: var(--line-height);
 }
 
-h1, h2, h3, h4, h5 {
-  margin-bottom: var(--space-small);
+h1, h2, h3, h4, h5, h6 {
   color: var(--color-heading);
-  font-weight: 500;
+  font-weight: bold;
 }
 
 h1 {
   font-size: var(--size-xxxlarge);
-  line-height: var(--space-xlarge)
 }
 
 h2 {

package/src/css/variables.css

@@ -13,40 +13,40 @@
   --space-xxlarge: 48px;
   --space-xxxlarge: 64px;
 
-  /* Typography scale */
-  --size-xsmall: 0.77rem;
+  /* Typography scale - https://type-scale.com 1.2 */
+  --size-xsmall: 0.8rem;
   --size-small: 0.92rem;
   --size: 1rem;
   --size-medium: var(--size);
-  --size-large: 1.125rem;
-  --size-xlarge: 1.266rem;
-  --size-xxlarge: 1.424rem;
-  --size-xxxlarge: 1.602rem;
+  --size-large: 1.2rem;
+  --size-xlarge: 1.44rem;
+  --size-xxlarge: 1.728rem;
+  --size-xxxlarge: 2.074rem;
 
-  --line-height: var(--space-medium);
+  --line-height: 1.66em;
 
   /*
    * Color scheme:
-   * http://mcg.mbitson.com/#!?mcgpalette0=%23dd555e&mcgpalette1=%2363bf5e&mcgpalette2=%23f7e37d&mcgpalette3=%23636568&mcgpalette4=%233385d6&themename=mcgtheme
    * https://coolors.co/3385d6-dd555e-63bf5e-f7e37d-192339
+   * http://mcg.mbitson.com/#!?mcgpalette0=%23dd555e&mcgpalette1=%2363bf5e&mcgpalette2=%23f7e37d&mcgpalette3=%235b5d60&mcgpalette4=%233385d6&themename=mcgtheme
    */
   --color-gray-25: #fafafa;
-  --color-gray-50: #eceded;
-  --color-gray-100: #d0d1d2;
-  --color-gray-200: #b1b2b4;
-  --color-gray-300: #929395;
-  --color-gray-400: #7a7c7f;
-  --color-gray-500: #636568;
-  --color-gray-600: #5b5d60;
-  --color-gray-700: #515355;
-  --color-gray-800: #47494b;
-  --color-gray-900: #35373a;
+  --color-gray-50: #e4e4e5;
+  --color-gray-100: #cececf;
+  --color-gray-200: #adaeb0;
+  --color-gray-300: #8c8e90;
+  --color-gray-400: #747578;
+  --color-gray-500: #5b5d60;
+  --color-gray-600: #535558;
+  --color-gray-700: #494b4e;
+  --color-gray-800: #404144;
+  --color-gray-900: #2f3033;
 
   --color-gray-ultra-light: var(--color-gray-200);
   --color-gray-light: var(--color-gray-300);
   --color-gray: var(--color-gray-500);
-  --color-gray-dark: var(--color-gray-700);
-  --color-gray-ultra-dark: var(--color-gray-900);
+  --color-gray-dark: var(--color-gray-600);
+  --color-gray-ultra-dark: var(--color-gray-700);
 
   --color-red-25: #fdf5f6;
   --color-red-50: #fbebec;
@@ -170,7 +170,7 @@
   --color-outline-warning: var(--color-yellow-200);
   --color-outline-danger: var(--color-red-100);
 
-  --color-shadow: rgba(0, 0, 0, 0.2);
+  --color-shadow: rgba(0, 0, 0, 0.1);
 
   --transition-in: all 0.2s ease-in;
   --transition-out: all 0.17s ease-out;
@@ -181,9 +181,9 @@
   --layer-high: 1000;
   --layer-xhigh: 10000;
 
-  --radius-small: var(--space-xxxsmall);
-  --radius-medium: var(--space-xxsmall);
-  --radius-large: var(--space-small);
+  --radius-small: calc(var(--space-xxxsmall) / 2);
+  --radius-medium: var(--space-xxxsmall);
+  --radius-large: var(--space-xxsmall);
 
   /** Layout */
   --max-width: 80rem;

package/src/layout/box/box.module.css

@@ -4,6 +4,15 @@
 
 .outline {
   border: 1px solid var(--color-outline);
+  border-radius: var(--radius-medium);
+}
+
+.outline > :first-child {
+  border-radius: var(--radius-large) var(--radius-medium) 0 0;
+}
+
+.outline > :last-child {
+  border-radius: 0 0 var(--radius-large) var(--radius-medium);
 }
 
 .padding-xxxsmall {

package/src/layout/footer/footer.module.css

@@ -3,9 +3,10 @@
 }
 
 .inner {
-  border-top: 2px solid var(--color-outline);
+  border-top: 1px solid var(--color-outline);
 }
 
+
 .navItem {
   display: inline-block;
   margin: 0 var(--space-xxxsmall);

package/src/layout/index.js

@@ -1,6 +1,7 @@
 export { Box } from './box';
 export { Footer } from './footer';
 export { Header } from './header';
+export { Separator } from './separator';
 export { SubHeader } from './sub-header';
 export { Stack } from './stack';
 export { FlexStack } from './flex-stack';

package/src/layout/separator.jsx

@@ -0,0 +1,25 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import cx from 'classnames';
+
+import css from './separator.module.css';
+
+const propTypes = {
+  className: PropTypes.string,
+};
+
+const defaultProps = {
+  className: '',
+};
+
+/**
+ * @param {PropTypes.InferProps<typeof propTypes>} props
+ * @return {React.FunctionComponentElement}
+ */
+export const Separator = (props) => {
+  const { className, ...restProps } = props;
+  return <div className={cx(css.root, className)} {...restProps} />;
+};
+
+Separator.propTypes = propTypes;
+Separator.defaultProps = defaultProps;

package/src/layout/separator.module.css

@@ -0,0 +1,5 @@
+.root {
+  background: var(--color-outline);
+  width: 100%;
+  height: 1px;
+}

package/src/ui/dropdown/dropdown.jsx

@@ -11,27 +11,25 @@ export const Dropdown = (props) => {
   const { className, label, ariaLabel, glyph, activeLabel, children } = props;
   const rootClassName = cx(css.root, activeLabel && css.activeLabel, className);
 
-  const menu = useMenuState({baseId: process.env.NODE_ENV === 'test' && 'id-test', modal: true });
+  const menu = useMenuState({ baseId: process.env.NODE_ENV === 'test' && 'id-test', modal: true });
 
   return (
     <>
       <MenuButton {...menu} className={rootClassName} tabIndex={null}>
-        <FlexStack space="xxxsmall" className={css.label}>
+        <FlexStack space="xxxsmall">
           {glyph && <Icon className={css.labelIcon} glyph={glyph} />}
           {label}
         </FlexStack>
       </MenuButton>
       <Menu {...menu} aria-label={ariaLabel || label} className={css.dropdown}>
         {typeof children === 'function'
-          ?
-            children({
+          ? children({
               MenuItem,
               menu,
               menuItemClassName: css.menuItem,
               menuItemActiveClassName: css.menuItemActive,
             })
-          : children
-        }
+          : children}
       </Menu>
     </>
   );

package/src/ui/dropdown/dropdown.module.css

@@ -2,15 +2,9 @@
   background: none;
   border: none;
   padding: 0;
-}
 
-.root:focus {
-  outline: none;
-  box-shadow: 0 0 var(--space-xxxsmall) var(--color-shadow);
-}
-
-.label {
   border: 1px solid var(--color-outline);
+  border-radius: var(--radius-medium);
   padding: calc(var(--space-xxsmall) - 1px);
   display: flex;
   align-items: center;
@@ -24,7 +18,11 @@
   color: var(--color-text-light);
 }
 
-.label:hover {
+.root:hover,
+.root:focus,
+.root:active {
+  outline: none;
+  box-shadow: 0 0 var(--space-xxxsmall) var(--color-shadow);
   border-color: var(--color-outline-dark);
   color: var(--color-text);
 }
@@ -33,6 +31,7 @@
   position: absolute;
   left: 0;
   top: 100%;
+  border-radius: var(--radius-medium);
   box-shadow: 0 2px var(--space-xxsmall) var(--color-shadow);
   background: var(--color-background);
   min-width: 180px;

package/src/ui/filters/filters.jsx

@@ -18,25 +18,15 @@ const Filter = (props) => {
     <div className={rootClassName}>
       {/* eslint-disable */}
       <FlexStack space="xxxsmall" as="label" className={css.filterCheckbox}>
-      {/* eslint-enabled */}
-        <input
-          className={css.filterInput}
-          type="checkbox"
-          id={id}
-          name={name}
-          {...inputProps}
-        />
+        {/* eslint-enabled */}
+        <input className={css.filterInput} type="checkbox" id={id} name={name} {...inputProps} />
         <span className={css.filterLabel} title={label}>
           {label}
         </span>
       </FlexStack>
 
       {getOnOnlyClick && (
-        <button
-          className={css.filterOnlyButton}
-          type="button"
-          onClick={getOnOnlyClick(name)}
-        >
+        <button className={css.filterOnlyButton} type="button" onClick={getOnOnlyClick(name)}>
           only
         </button>
       )}
@@ -57,13 +47,17 @@ Filter.defaultProps = {
 };
 
 const FilterGroup = (props) => {
-  const { groupKey, data, values, onCheckboxChange, toggleFilters } = props;
+  const { className, groupKey, data, values, onCheckboxChange, toggleFilters } = props;
 
   const { label: groupLabel, ...groupData } = data;
 
   const groupItems = Object.entries(groupData);
-  const groupCheckboxes = groupItems.filter(([itemKey, item]) => typeof item?.defaultValue !== 'undefined');
-  const isGroupChecked = groupCheckboxes.map(([itemKey]) => get(values, `${groupKey}.${itemKey}`)).reduce((agg, val) => agg && val, true);
+  const groupCheckboxes = groupItems.filter(
+    ([itemKey, item]) => typeof item?.defaultValue !== 'undefined',
+  );
+  const isGroupChecked = groupCheckboxes
+    .map(([itemKey]) => get(values, `${groupKey}.${itemKey}`))
+    .reduce((agg, val) => agg && val, true);
 
   const filterSuffix = getGroupFiltersLabelSuffix(groupItems);
   const hasCustomFilterSuffix = !Object.values(LABELS).includes(filterSuffix);
@@ -71,26 +65,33 @@ const FilterGroup = (props) => {
     <>
       {`${groupLabel}:`}
       &nbsp;
-      <span className={cx(hasCustomFilterSuffix && css.labelSuffixCustom)}>
-        {filterSuffix}
-      </span>
+      <span className={cx(hasCustomFilterSuffix && css.labelSuffixCustom)}>{filterSuffix}</span>
     </>
   );
 
-  const getOnGroupCheck = (value, overrides = {}) => () => {
-    const newFilters = groupCheckboxes.reduce((agg, [itemKey]) => ({
-      ...agg,
-      [`${groupKey}.${itemKey}`]: value,
-    }), {});
-
-    toggleFilters({
-      ...newFilters,
-      ...overrides,
-    });
-  };
+  const getOnGroupCheck =
+    (value, overrides = {}) =>
+    () => {
+      const newFilters = groupCheckboxes.reduce(
+        (agg, [itemKey]) => ({
+          ...agg,
+          [`${groupKey}.${itemKey}`]: value,
+        }),
+        {},
+      );
+
+      toggleFilters({
+        ...newFilters,
+        ...overrides,
+      });
+    };
 
   return (
-    <Dropdown label={dropdownLabel} ariaLabel={`${groupLabel}: ${filterSuffix}`}>
+    <Dropdown
+      className={className}
+      label={dropdownLabel}
+      ariaLabel={`${groupLabel}: ${filterSuffix}`}
+    >
       {({ MenuItem, menu, menuItemClassName }) => {
         return (
           <>
@@ -100,7 +101,11 @@ const FilterGroup = (props) => {
                 const getOnOnlyClick = () => getOnGroupCheck(false, { [id]: true });
 
                 return (
-                  <MenuItem key={id} {...menu} className={cx(menuItemClassName, css.filterGroupItem)}>
+                  <MenuItem
+                    key={id}
+                    {...menu}
+                    className={cx(menuItemClassName, css.filterGroupItem)}
+                  >
                     <Filter
                       name={id}
                       label={itemData.label}
@@ -138,12 +143,13 @@ const FilterGroup = (props) => {
             </div>
           </>
         );
-    }}
+      }}
     </Dropdown>
   );
 };
 
 FilterGroup.propTypes = {
+  className: PropTypes.string,
   groupKey: PropTypes.string.isRequired,
   data: PropTypes.object.isRequired, // eslint-disable-line react/forbid-prop-types
   values: PropTypes.object.isRequired, // eslint-disable-line react/forbid-prop-types
@@ -151,14 +157,12 @@ FilterGroup.propTypes = {
   toggleFilters: PropTypes.func.isRequired,
 };
 
+FilterGroup.defaultProps = {
+  className: '',
+};
+
 export const Filters = (props) => {
-  const {
-    className,
-    values,
-    filters,
-    toggleFilter,
-    toggleFilters,
-  } = props;
+  const { className, values, filters, toggleFilter, toggleFilters } = props;
 
   const onCheckboxChange = ({ target }) => toggleFilter(target.name, target.checked);
   const rootClassName = cx(css.root, className);
@@ -182,16 +186,15 @@ export const Filters = (props) => {
           }
 
           return (
-            <div className={css.item}>
-              <FilterGroup
-                key={name}
-                groupKey={name}
-                data={data}
-                values={values}
-                onCheckboxChange={onCheckboxChange}
-                toggleFilters={toggleFilters}
-              />
-            </div>
+            <FilterGroup
+              className={css.item}
+              key={name}
+              groupKey={name}
+              data={data}
+              values={values}
+              onCheckboxChange={onCheckboxChange}
+              toggleFilters={toggleFilters}
+            />
           );
         })}
       </FlexStack>

package/src/ui/filters/filters.module.css

@@ -11,6 +11,7 @@
 }
 
 .filterInput {
+  border-radius: var(--radius-medium) 0 0 var(--radius-medium);
   flex: 0 0 auto;
   height: 1em;
   cursor: pointer;
@@ -129,11 +130,20 @@
 }
 
 .item {
+  border-radius: 0;
   position: relative;
   flex: 0 0 auto;
   margin-right: -1px;
 }
 
+.item:first-child {
+  border-radius: var(--radius-medium) 0 0 var(--radius-medium);
+}
+
+.item:last-child {
+  border-radius: 0 var(--radius-medium) var(--radius-medium) 0;
+}
+
 .item:hover {
   z-index: var(--layer-low); /* show collapsed border */
 }

package/src/ui/popover/popover.module.css

@@ -24,7 +24,7 @@
 
 .popover {
   padding: var(--space-small) var(--space-large) var(--space-small) var(--space-small);
-  border-radius: var(--radius-small);
+  border-radius: var(--radius-large);
   max-width: 360px;
   outline: none;
   filter: drop-shadow(0 0 var(--space-xxxsmall) var(--color-shadow));

package/src/ui/table/table.module.css

@@ -13,12 +13,8 @@
   text-align: left;
 }
 
-tr:last-child .cell {
-  border-bottom-width: 1px;
-}
-
 .root thead .cell {
-  border-width: 0 0 2px;
+  border-width: 0 0 1px;
   text-transform: uppercase;
   vertical-align: top;
 }
@@ -44,10 +40,6 @@ tr:last-child .cell {
   text-decoration: underline;
 }
 
-.root tbody tr:nth-child(odd) .cell {
-  background-color: var(--color-highlight);
-}
-
 .root tbody tr:hover .cell {
   background-color: var(--color-highlight-warning);
 }

package/src/ui/tabs/tabs.module.css

@@ -7,8 +7,7 @@
   position: relative;
   display: inline-block;
   padding: var(--space-xsmall) 0;
-  color: var(--color-text-ultra-light);
-  font-weight: bold;
+  color: var(--color-text-light);
   text-decoration: none;
 }
 

package/src/ui/tooltip/tooltip.module.css

@@ -12,7 +12,7 @@
 
 .tooltip {
   padding: var(--space-xxxsmall) var(--space-xxsmall);
-  border-radius: var(--radius-small);
+  border-radius: var(--radius-large);
   background: var(--color-light);
   color: var(--color-text);
   filter: drop-shadow(0 0 var(--space-xxxsmall) var(--color-shadow));