@bundle-stats/ui 3.2.1 → 3.2.3

package/lib-esm/css/variables.css

@@ -27,20 +27,20 @@
 
   /*
    * Color scheme:
-   * http://mcg.mbitson.com/#!?mcgpalette0=%23dd555e&mcgpalette1=%2363bf5e&mcgpalette2=%23f7e37d&mcgpalette3=%23484a4d&mcgpalette4=%233385d6&themename=mcgtheme
+   * http://mcg.mbitson.com/#!?mcgpalette0=%23dd555e&mcgpalette1=%2363bf5e&mcgpalette2=%23f7e37d&mcgpalette3=%23636568&mcgpalette4=%233385d6&themename=mcgtheme
    * https://coolors.co/3385d6-dd555e-63bf5e-f7e37d-192339
    */
   --color-gray-25: #fafafa;
-  --color-gray-50: #e9e9ea;
-  --color-gray-100: #c8c9ca;
-  --color-gray-200: #a4a5a6;
-  --color-gray-300: #7f8082;
-  --color-gray-400: #636568;
-  --color-gray-500: #484a4d;
-  --color-gray-600: #414346;
-  --color-gray-700: #383a3d;
-  --color-gray-800: #303234;
-  --color-gray-900: #212225;
+  --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-ultra-light: var(--color-gray-200);
   --color-gray-light: var(--color-gray-300);
@@ -181,6 +181,10 @@
   --layer-high: 1000;
   --layer-xhigh: 10000;
 
+  --radius-small: var(--space-xxxsmall);
+  --radius-medium: var(--space-xxsmall);
+  --radius-large: var(--space-small);
+
   /** Layout */
   --max-width: 80rem;
 }

package/lib-esm/ui/filters/filters.container.js

@@ -28,45 +28,6 @@ export const filtersDropdownContainer = BaseComponent => {
   class FiltersContainer extends React.Component {
     constructor(props) {
       super(props);
-
-      this.toggleFilter = (key, value) => this.setState(_ref2 => {
-        let {
-          values,
-          ...state
-        } = _ref2;
-        const newState = { ...state,
-          values: { ...values,
-            [key]: value
-          }
-        };
-        const {
-          onChange
-        } = this.props;
-
-        if (onChange) {
-          onChange(newState.values);
-        }
-
-        return newState;
-      });
-
-      this.toggleFilters = newFilters => this.setState(prevState => {
-        const newState = { ...prevState,
-          values: { ...prevState.values,
-            ...newFilters
-          }
-        };
-        const {
-          onChange
-        } = this.props;
-
-        if (onChange) {
-          onChange(newState.values);
-        }
-
-        return newState;
-      });
-
       const {
         filters
       } = this.props;
@@ -75,10 +36,10 @@ export const filtersDropdownContainer = BaseComponent => {
       };
     }
 
-    static getDerivedStateFromProps(_ref3, state) {
+    static getDerivedStateFromProps(_ref2, state) {
       let {
         filters
-      } = _ref3;
+      } = _ref2;
 
       if (_isEqual(filters, state.values)) {
         return null;
@@ -89,6 +50,43 @@ export const filtersDropdownContainer = BaseComponent => {
       };
     }
 
+    toggleFilter = (key, value) => this.setState(_ref3 => {
+      let {
+        values,
+        ...state
+      } = _ref3;
+      const newState = { ...state,
+        values: { ...values,
+          [key]: value
+        }
+      };
+      const {
+        onChange
+      } = this.props;
+
+      if (onChange) {
+        onChange(newState.values);
+      }
+
+      return newState;
+    });
+    toggleFilters = newFilters => this.setState(prevState => {
+      const newState = { ...prevState,
+        values: { ...prevState.values,
+          ...newFilters
+        }
+      };
+      const {
+        onChange
+      } = this.props;
+
+      if (onChange) {
+        onChange(newState.values);
+      }
+
+      return newState;
+    });
+
     render() {
       return /*#__PURE__*/React.createElement(BaseComponent, _extends({}, this.props, this.state, {
         toggleFilter: this.toggleFilter,

package/lib-esm/ui/icon/icon.js

@@ -6,6 +6,7 @@ import cx from 'classnames';
 import css from './icon.module.css';
 const ICONS = {
   ARROW: 'arrow',
+  ARROW_RIGHT_CIRLCE: 'arrow-right-circle',
   CANCEL: 'cancel',
   CLOSE: 'close',
   CLOCK: 'clock',

package/lib-esm/ui/icon/icon.stories.js

@@ -1,21 +1,32 @@
 import React from 'react';
-import { storiesOf } from '@storybook/react';
-import { getWrapperDecorator } from '../../stories';
 import { Icon } from '.';
-const stories = storiesOf('UI/Icon', module);
-stories.addDecorator(getWrapperDecorator());
-stories.add('default', () => /*#__PURE__*/React.createElement(Icon, {
+export default {
+  title: 'UI/Icon',
+  component: Icon
+};
+
+const Template = args => /*#__PURE__*/React.createElement(Icon, args);
+
+export const Standard = Template.bind();
+Standard.args = {
   glyph: Icon.ICONS.ARROW
-}));
-stories.add('with size', () => /*#__PURE__*/React.createElement(Icon, {
+};
+export const CustomSize = Template.bind();
+CustomSize.args = {
   glyph: Icon.ICONS.ARROW,
   size: Icon.SIZE_LARGE
-}));
-stories.add('all', () => /*#__PURE__*/React.createElement(React.Fragment, null, Object.values(Icon.ICONS).map(glyph => /*#__PURE__*/React.createElement("span", {
+};
+export const All = () => /*#__PURE__*/React.createElement(React.Fragment, null, Object.values(Icon.ICONS).map(glyph => /*#__PURE__*/React.createElement("div", {
   style: {
-    margin: '0.3rem'
+    display: 'inline-block',
+    padding: '1rem',
+    textAlign: 'center'
   }
 }, /*#__PURE__*/React.createElement(Icon, {
   glyph: glyph,
   key: glyph
-})))));
+}), /*#__PURE__*/React.createElement("code", {
+  style: {
+    display: 'block'
+  }
+}, glyph))));

package/lib-esm/ui/popover/popover.js

@@ -16,7 +16,7 @@ export const Popover = _ref => {
   } = _ref;
   const popover = usePopoverState({
     baseId: process.env.NODE_ENV === 'test' && 'id-test',
-    gutter: 24,
+    gutter: 12,
     modal: true,
     placement: 'top'
   });
@@ -45,7 +45,7 @@ export const Popover = _ref => {
   }), /*#__PURE__*/React.createElement(UIPopoverArrow, _extends({
     className: css.arrow
   }, popover, {
-    size: 24
+    size: 16
   })), typeof children === 'function' ? children({
     popoverToggle: popover.toggle
   }) : children, /*#__PURE__*/React.createElement(Icon, {

package/lib-esm/ui/popover/popover.module.css

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

package/lib-esm/ui/tooltip/tooltip.module.css

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bundle-stats/ui",
-  "version": "3.2.1",
+  "version": "3.2.3",
   "description": "UI components for BundleStats projects.",
   "main": "lib/index.js",
   "module": "lib-esm/index.js",
@@ -12,7 +12,7 @@
     "build-esm": "babel -D src -d lib-esm --ignore **/__tests__/**/* --ignore /prototypes/**",
     "build-storybook": "npx build-storybook -c ./build/storybook",
     "generate-chart-colors": "node scripts/generate-chart-colors.js",
-    "generate-svg-sprite": "npx svg-sprite -s --si src/assets/icons/*.svg && mv symbol/svg/sprite.symbol.svg src/assets/icons.svg",
+    "generate-svg-sprite": "npx svg-sprite -s --si --no-svg-namespace-classnames src/assets/icons/*.svg && mv symbol/svg/sprite.symbol.svg src/assets/icons.svg",
     "lint": "../../node_modules/.bin/eslint . --ext .jsx,.js",
     "start": "cross-env NODE_ENV=development start-storybook -c ./build/storybook -p 8080",
     "test": "npm run test:unit && npm run test:storyshots",
@@ -37,36 +37,36 @@
   "homepage": "https://github.com/relative-ci/bundle-stats/blob/master/packages/ui#readme",
   "devDependencies": {
     "@babel/cli": "7.16.0",
-    "@babel/core": "7.16.0",
-    "@babel/plugin-proposal-class-properties": "7.16.0",
-    "@babel/preset-env": "7.16.4",
-    "@babel/preset-react": "7.16.0",
+    "@babel/core": "7.16.5",
+    "@babel/plugin-proposal-class-properties": "7.16.5",
+    "@babel/preset-env": "7.16.5",
+    "@babel/preset-react": "7.16.5",
     "@simplyianm/babel-plugin-inline-react-svg": "1.2.0",
-    "@storybook/addon-knobs": "6.3.1",
+    "@storybook/addon-knobs": "6.4.0",
     "@storybook/addon-postcss": "2.0.0",
-    "@storybook/addon-storyshots": "6.3.12",
-    "@storybook/addons": "6.3.12",
-    "@storybook/builder-webpack5": "6.3.12",
-    "@storybook/manager-webpack5": "6.3.12",
-    "@storybook/react": "6.3.12",
+    "@storybook/addon-storyshots": "6.4.9",
+    "@storybook/addons": "6.4.9",
+    "@storybook/builder-webpack5": "6.4.9",
+    "@storybook/manager-webpack5": "6.4.9",
+    "@storybook/react": "6.4.9",
     "babel-core": "7.0.0-bridge.0",
-    "babel-jest": "27.3.1",
+    "babel-jest": "27.4.5",
     "babel-plugin-lodash": "3.3.4",
     "babel-plugin-require-context-hook": "1.0.0",
-    "chromatic": "6.0.6",
+    "chromatic": "6.2.3",
     "classnames": "2.3.1",
     "colormap": "2.3.2",
-    "core-js": "3.19.1",
+    "core-js": "3.20.0",
     "cross-env": "7.0.3",
     "enzyme": "3.11.0",
     "enzyme-adapter-react-16": "1.15.6",
     "enzyme-to-json": "3.6.2",
     "hex-rgb": "5.0.0",
     "identity-obj-proxy": "3.0.0",
-    "jest": "27.3.1",
+    "jest": "27.4.5",
     "jest-date-mock": "1.0.8",
     "lodash": "4.17.21",
-    "postcss": "8.3.11",
+    "postcss": "8.4.5",
     "prop-types": "15.7.2",
     "react": "16.14.0",
     "react-dom": "16.14.0",
@@ -75,20 +75,20 @@
     "rimraf": "3.0.2"
   },
   "peerDependencies": {
-    "classnames": "^2.2.6",
-    "core-js": "^3.16.2",
-    "lodash": "^4.17.21",
-    "prop-types": "^15.7.2",
-    "react": "^16.14.0",
-    "react-dom": "^16.14.0",
-    "react-router-dom": "^5.3.0"
+    "classnames": "2.2.6",
+    "core-js": "3.16.2",
+    "lodash": "4.17.21",
+    "prop-types": "15.7.2",
+    "react": "16.14.0",
+    "react-dom": "16.14.0",
+    "react-router-dom": "5.3.0"
   },
   "dependencies": {
-    "@bundle-stats/utils": "^3.2.1",
-    "date-fns": "^2.6.0",
-    "query-string": "^7.0.0",
-    "reakit": "^1.3.6",
-    "use-query-params": "^1.2.2"
+    "@bundle-stats/utils": "^3.2.3",
+    "date-fns": "2.6.0",
+    "query-string": "7.0.0",
+    "reakit": "1.3.6",
+    "use-query-params": "1.2.2"
   },
-  "gitHead": "0fdfecf0860efdf80e19fa1bb16bb1fddbe070a4"
+  "gitHead": "cc3f39324ba9def9fe0814c45983234c7beb941e"
 }

package/src/assets/icons/arrow-right-circle.svg

@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" width="24" height="24" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="feather feather-arrow-right-circle"><circle cx="12" cy="12" r="10"></circle><polyline points="12 16 16 12 12 8"></polyline><line x1="8" y1="12" x2="16" y2="12"></line></svg>

package/src/assets/icons/close.svg

@@ -1 +1 @@
-<svg xmlns="http://www.w3.org/2000/svg" width="64" height="64" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="feather feather-x"><line x1="18" y1="6" x2="6" y2="18"></line><line x1="6" y1="6" x2="18" y2="18"></line></svg>
+<svg xmlns="http://www.w3.org/2000/svg" width="24" height="24" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="feather feather-x-circle"><circle cx="12" cy="12" r="10"></circle><line x1="15" y1="9" x2="9" y2="15"></line><line x1="9" y1="9" x2="15" y2="15"></line></svg>

package/src/assets/icons.svg

@@ -1 +1,42 @@
-<svg width="16" height="16"><symbol viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="afeather afeather-arrow-down" id="arrow" xmlns="http://www.w3.org/2000/svg"><path d="M12 5v14M19 12l-7 7-7-7"/></symbol><symbol viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="bfeather bfeather-x-circle" id="cancel" xmlns="http://www.w3.org/2000/svg"><circle cx="12" cy="12" r="10"/><path d="M15 9l-6 6M9 9l6 6"/></symbol><symbol viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="cfeather cfeather-clock" id="clock" xmlns="http://www.w3.org/2000/svg"><circle cx="12" cy="12" r="10"/><path d="M12 6v6l4 2"/></symbol><symbol viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="dfeather dfeather-x" id="close" xmlns="http://www.w3.org/2000/svg"><path d="M18 6L6 18M6 6l12 12"/></symbol><symbol viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="efeather efeather-git-commit" id="commit" xmlns="http://www.w3.org/2000/svg"><circle cx="12" cy="12" r="4"/><path d="M1.05 12H7M17.01 12h5.95"/></symbol><symbol viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="ffeather ffeather-bar-chart" id="filter" xmlns="http://www.w3.org/2000/svg"><path d="M12 20V10M18 20V4M6 20v-4"/></symbol><symbol viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="gfeather gfeather-help-circle" id="help" xmlns="http://www.w3.org/2000/svg"><circle cx="12" cy="12" r="10"/><path d="M9.09 9a3 3 0 015.83 1c0 2-3 3-3 3M12 17h.01"/></symbol><symbol viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="hfeather hfeather-menu" id="menu" xmlns="http://www.w3.org/2000/svg"><path d="M3 12h18M3 6h18M3 18h18"/></symbol><symbol viewBox="0 0 24 24" fill="currentColor" id="sort" xmlns="http://www.w3.org/2000/svg"><path d="M3 18h6v-2H3v2zM3 6v2h18V6H3zm0 7h12v-2H3v2z"/><path d="M0 0h24v24H0z" fill="none"/></symbol><symbol viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="jfeather jfeather-alert-triangle" id="warning" xmlns="http://www.w3.org/2000/svg"><path d="M10.29 3.86L1.82 18a2 2 0 001.71 3h16.94a2 2 0 001.71-3L13.71 3.86a2 2 0 00-3.42 0zM12 9v4M12 17h.01"/></symbol></svg>
+<svg width="16" height="16">
+<symbol viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="feather feather-arrow-down" id="arrow" xmlns="http://www.w3.org/2000/svg">
+<path d="M12 5v14M19 12l-7 7-7-7"/>
+</symbol>
+<symbol fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="feather feather-arrow-right-circle" viewBox="0 0 24 24" id="arrow-right-circle" xmlns="http://www.w3.org/2000/svg">
+<circle cx="12" cy="12" r="10"/>
+<path d="M12 16l4-4-4-4M8 12h8"/>
+</symbol>
+<symbol viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="feather feather-x-circle" id="cancel" xmlns="http://www.w3.org/2000/svg">
+<circle cx="12" cy="12" r="10"/>
+<path d="M15 9l-6 6M9 9l6 6"/>
+</symbol>
+<symbol viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="feather feather-clock" id="clock" xmlns="http://www.w3.org/2000/svg">
+<circle cx="12" cy="12" r="10"/>
+<path d="M12 6v6l4 2"/>
+</symbol>
+<symbol fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="feather feather-x-circle" viewBox="0 0 24 24" id="close" xmlns="http://www.w3.org/2000/svg">
+<circle cx="12" cy="12" r="10"/>
+<path d="M15 9l-6 6M9 9l6 6"/>
+</symbol>
+<symbol viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="feather feather-git-commit" id="commit" xmlns="http://www.w3.org/2000/svg">
+<circle cx="12" cy="12" r="4"/>
+<path d="M1.05 12H7M17.01 12h5.95"/>
+</symbol>
+<symbol viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="feather feather-bar-chart" id="filter" xmlns="http://www.w3.org/2000/svg">
+<path d="M12 20V10M18 20V4M6 20v-4"/>
+</symbol>
+<symbol viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="feather feather-help-circle" id="help" xmlns="http://www.w3.org/2000/svg">
+<circle cx="12" cy="12" r="10"/>
+<path d="M9.09 9a3 3 0 015.83 1c0 2-3 3-3 3M12 17h.01"/>
+</symbol>
+<symbol viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="feather feather-menu" id="menu" xmlns="http://www.w3.org/2000/svg">
+<path d="M3 12h18M3 6h18M3 18h18"/>
+</symbol>
+<symbol viewBox="0 0 24 24" fill="currentColor" id="sort" xmlns="http://www.w3.org/2000/svg">
+<path d="M3 18h6v-2H3v2zM3 6v2h18V6H3zm0 7h12v-2H3v2z"/>
+<path d="M0 0h24v24H0z" fill="none"/>
+</symbol>
+<symbol viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="feather feather-alert-triangle" id="warning" xmlns="http://www.w3.org/2000/svg">
+<path d="M10.29 3.86L1.82 18a2 2 0 001.71 3h16.94a2 2 0 001.71-3L13.71 3.86a2 2 0 00-3.42 0zM12 9v4M12 17h.01"/>
+</symbol>
+</svg>

package/src/components/summary/summary.jsx

@@ -5,34 +5,13 @@ import { get } from 'lodash';
 import { METRIC_COMPONENT_LINKS } from '@bundle-stats/utils';
 
 import { METRICS_WEBPACK_GENERAL } from '../../constants';
+import { Icon } from '../../ui/icon';
 import { Box } from '../../layout/box';
 import { FlexStack } from '../../layout/flex-stack';
 import { ComponentLink } from '../component-link';
 import { SummaryItem } from '../summary-item';
 import css from './summary.module.css';
 
-const CustomSummaryItem = ({ className, metricOptions, SummaryItemCustomLink, ...props }) => (
-  <Box
-    padding={['xsmall', 'small']}
-    className={cx(className, css.summaryItemLink)}
-    {...props}
-    as={SummaryItemCustomLink}
-    {...metricOptions.link}
-  />
-);
-
-CustomSummaryItem.propTypes = {
-  className: PropTypes.string,
-  metricOptions: PropTypes.shape({
-    link: PropTypes.any, // eslint-disable-line react/forbid-prop-types
-  }).isRequired,
-  SummaryItemCustomLink: PropTypes.elementType.isRequired,
-};
-
-CustomSummaryItem.defaultProps = {
-  className: '',
-};
-
 export const Summary = ({
   className,
   size,
@@ -44,23 +23,25 @@ export const Summary = ({
 }) => (
   <Box outline className={cx(css.root, className)}>
     <FlexStack className={css.items}>
-      {Array.from(METRIC_COMPONENT_LINKS).filter(([metricId]) => keys.includes(metricId)).map(
-        ([metricId, metricOptions]) => (
-          <SummaryItem
-            as={CustomSummaryItem}
-            className={css.item}
-            metricOptions={metricOptions}
-            SummaryItemCustomLink={SummaryItemCustomLink}
-            size={size}
-            key={metricId}
-            id={metricId}
-            data={get(data, metricId)}
-            loading={loading}
-            showMetricDescription
-            showDelta={showSummaryItemDelta && metricOptions.showDelta !== false}
-          />
-        )
-      )}
+      {Array.from(METRIC_COMPONENT_LINKS)
+        .filter(([metricId]) => keys.includes(metricId))
+        .map(([metricId, metricOptions]) => (
+          <div key={metricId} className={css.item}>
+            <SummaryItem
+              size={size}
+              id={metricId}
+              data={get(data, metricId)}
+              loading={loading}
+              showMetricDescription
+              showDelta={showSummaryItemDelta && metricOptions.showDelta !== false}
+            />
+            {!loading && (
+              <SummaryItemCustomLink {...metricOptions.link} className={css.readMore}>
+                <Icon className={css.readMoreIcon} glyph={Icon.ICONS.ARROW_RIGHT_CIRLCE} />
+              </SummaryItemCustomLink>
+            )}
+          </div>
+        ))}
     </FlexStack>
   </Box>
 );

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

@@ -3,35 +3,52 @@
   overflow: auto;
 }
 
-.items {}
-
 .item {
+  padding: var(--space-small) var(--space-xxlarge) var(--space-small) var(--space-small);
   flex: 1 0 25%; /* force equal sizes */
   position: relative;
   z-index: 1;
 }
 
-.item:focus-visible {
-  background: var(--color-highlight);
-  outline: none;
-}
-
 .item::before {
   content: '';
   display: block;
   width: 1px;
   position: absolute;
   left: -1px;
-  top: var(--space-xsmall);
-  bottom: var(--space-xsmall);
+  top: var(--space-small);
+  bottom: var(--space-small);
   background: var(--color-outline);
 }
 
-.item:hover {
-  background: var(--color-highlight);
+.readMore {
+  display: block;
+  position: absolute;
+  right: 0;
+  bottom: 0;
+  padding: var(--space-small);
+  color: var(--color-text-ultra-light);
+  transition: var(--transition-out);
+}
+
+.readMore:hover,
+.readMore:focus {
+  color: var(--color-primary);
+  transition: var(--transition-in);
+}
+
+.readMoreIcon {
+  display: block;
+}
+
+/** hover state */
+.readMore {
+  opacity: 0;
 }
 
-.summaryItemLink {
-  color: inherit;
-  text-decoration: none;
+.item:hover .readMore,
+.item:active .readMore,
+.item:focus .readMore {
+  opacity: 1;
+  transition: var(--transition-in);
 }

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

@@ -11,7 +11,7 @@ import { Metric } from '../metric';
 import { Delta } from '../delta';
 import css from './summary-item.module.css';
 
-const MetricInfo = ({ description, url }) => {
+const MetricInfoPopover = ({ description, url }) => {
   // Workaround cases where the parent is a link
   const readMoreOnClick = () => {
     window.open(url);
@@ -31,12 +31,12 @@ const MetricInfo = ({ description, url }) => {
   );
 };
 
-MetricInfo.propTypes = {
+MetricInfoPopover.propTypes = {
   description: PropTypes.string.isRequired,
   url: PropTypes.string,
 };
 
-MetricInfo.defaultProps = {
+MetricInfoPopover.defaultProps = {
   url: '',
 };
 
@@ -57,14 +57,7 @@ export const SummaryItem = ({
   const runInfo = getMetricRunInfo(metric, current, baseline);
   const showMetricDescriptionTooltip = showMetricDescription && metric?.description;
 
-  const rootClassName = cx(
-    css.root,
-    className,
-    css[size],
-    showMetricDescription && css.showMetricDescription,
-    showMetricDescriptionTooltip && css.showMetricDescription,
-    showDelta && css.showDelta,
-  );
+  const rootClassName = cx(css.root, className, css[size], showDelta && css.showDelta);
 
   return (
     <Stack space="xxsmall" as={Component} className={rootClassName} {...props}>
@@ -73,13 +66,13 @@ export const SummaryItem = ({
 
         {showMetricDescriptionTooltip && (
           <Popover className={css.icon} icon="help">
-            <MetricInfo {...metric} />
+            <MetricInfoPopover {...metric} />
           </Popover>
         )}
       </FlexStack>
 
-      <Stack>
-        {!loading ? (
+      {!loading ? (
+        <Stack>
           <Metric
             className={css.currentMetric}
             value={current}
@@ -95,16 +88,14 @@ export const SummaryItem = ({
               />
             )}
           </Metric>
-        ) : (
-          <Skeleton className={cx(css.currentMetric, css.loading)} />
-        )}
-
-        {!loading ? (
           <Metric className={css.baselineMetric} value={baseline} formatter={metric.formatter} />
-        ) : (
+        </Stack>
+      ) : (
+        <Stack>
+          <Skeleton className={cx(css.currentMetric, css.loading)} />
           <Skeleton className={cx(css.baselineMetric, css.loading)} />
-        )}
-      </Stack>
+        </Stack>
+      )}
     </Stack>
   );
 };