@bundle-stats/ui 4.0.0-rc.1 → 4.0.0-rc.5

package/lib/components/bundle-modules/bundle-modules.utils.js

@@ -125,15 +125,18 @@ var useModuleFilterByChunk = function useModuleFilterByChunk(_ref) {
     }
 
     var jobsWithFilteredData = jobs.map(function (job) {
-      var _job$metrics;
+      var _job$metrics, _job$metrics$webpack;
 
-      var _ref2 = (job === null || job === void 0 ? void 0 : (_job$metrics = job.metrics) === null || _job$metrics === void 0 ? void 0 : _job$metrics.webpack) || {},
-          modules = _ref2.modules;
+      var modules = job === null || job === void 0 ? void 0 : (_job$metrics = job.metrics) === null || _job$metrics === void 0 ? void 0 : (_job$metrics$webpack = _job$metrics.webpack) === null || _job$metrics$webpack === void 0 ? void 0 : _job$metrics$webpack.modules;
 
-      var filteredModules = Object.entries(modules).reduce(function (agg, _ref3) {
-        var _ref4 = _slicedToArray(_ref3, 2),
-            moduleId = _ref4[0],
-            moduleEntry = _ref4[1];
+      if (!modules) {
+        return job;
+      }
+
+      var filteredModules = Object.entries(modules).reduce(function (agg, _ref2) {
+        var _ref3 = _slicedToArray(_ref2, 2),
+            moduleId = _ref3[0],
+            moduleEntry = _ref3[1];
 
         var match = (0, _intersection2["default"])(moduleEntry.chunkIds, includedChunkIds);
 

package/lib/components/metrics-table-title/metrics-table-title.js

@@ -43,10 +43,12 @@ var MetricsTableTitle = function MetricsTableTitle(props) {
     })
   }, /*#__PURE__*/_react["default"].createElement(_stack.Stack, {
     space: "xxxsmall"
-  }, popoverInfo && /*#__PURE__*/_react["default"].createElement("p", null, popoverInfo), popoverHref && /*#__PURE__*/_react["default"].createElement("p", null, /*#__PURE__*/_react["default"].createElement("a", {
-    href: popoverHref,
-    target: "_blank",
-    rel: "noreferrer"
+  }, popoverInfo && /*#__PURE__*/_react["default"].createElement("p", null, popoverInfo), popoverHref && /*#__PURE__*/_react["default"].createElement("p", null, /*#__PURE__*/_react["default"].createElement("button", {
+    type: "button",
+    onClick: function onClick() {
+      return window.open(popoverHref);
+    },
+    className: _metricsTableTitleModule["default"].readMoreLink
   }, _i18n["default"].READ_MORE)))));
 };
 

package/lib/components/metrics-table-title/metrics-table-title.module.css

@@ -11,3 +11,20 @@
   text-transform: none;
 }
 
+.readMoreLink {
+  appearance: none;
+  background: none;
+  margin: 0;
+  border: 0;
+  padding: 0;
+  color: var(--color-primary);
+  text-decoration: underline;
+  cursor: pointer;
+}
+
+.readMoreLink:hover,
+.readMoreLink:active,
+.readMoreLink:focus {
+  color: var(--color-primary-dark);
+  text-decoration: none;
+}

package/lib/components/total-size-type-title/total-size-type-title.js

@@ -38,10 +38,12 @@ var TotalSizeTypeTitle = function TotalSizeTypeTitle(_ref) {
     })
   }, /*#__PURE__*/_react["default"].createElement(_stack.Stack, {
     space: "xxxsmall"
-  }, /*#__PURE__*/_react["default"].createElement("p", null, _i18n["default"].ASSET_TOTALS_INFO), /*#__PURE__*/_react["default"].createElement("p", null, /*#__PURE__*/_react["default"].createElement("a", {
-    href: _config["default"].documentation.assets,
-    target: "_blank",
-    rel: "noreferrer"
+  }, /*#__PURE__*/_react["default"].createElement("p", null, _i18n["default"].ASSET_TOTALS_INFO), /*#__PURE__*/_react["default"].createElement("p", null, /*#__PURE__*/_react["default"].createElement("button", {
+    type: "button",
+    onClick: function onClick() {
+      return window.open(_config["default"].documentation.assets);
+    },
+    className: _totalSizeTypeTitleModule["default"].readMoreLink
   }, _i18n["default"].READ_MORE)))));
 };
 

package/lib/components/total-size-type-title/total-size-type-title.module.css

@@ -5,3 +5,21 @@
   line-height: var(--space-medium);
   text-transform: uppercase;
 }
+
+.readMoreLink {
+  appearance: none;
+  background: none;
+  margin: 0;
+  border: 0;
+  padding: 0;
+  color: var(--color-primary);
+  text-decoration: underline;
+  cursor: pointer;
+}
+
+.readMoreLink:hover,
+.readMoreLink:active,
+.readMoreLink:focus {
+  color: var(--color-primary-dark);
+  text-decoration: none;
+}

package/lib/ui/filters/filters.js

@@ -1,5 +1,7 @@
 "use strict";
 
+function _typeof(obj) { "@babel/helpers - typeof"; return _typeof = "function" == typeof Symbol && "symbol" == typeof Symbol.iterator ? function (obj) { return typeof obj; } : function (obj) { return obj && "function" == typeof Symbol && obj.constructor === Symbol && obj !== Symbol.prototype ? "symbol" : typeof obj; }, _typeof(obj); }
+
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
@@ -7,7 +9,7 @@ exports.Filters = void 0;
 
 var _get2 = _interopRequireDefault(require("lodash/get"));
 
-var _react = _interopRequireDefault(require("react"));
+var _react = _interopRequireWildcard(require("react"));
 
 var _propTypes = _interopRequireDefault(require("prop-types"));
 
@@ -24,6 +26,10 @@ var _filtersModule = _interopRequireDefault(require("./filters.module.css"));
 var _excluded = ["className", "buttonClassName", "label", "name", "getOnOnlyClick"],
     _excluded2 = ["label"];
 
+function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
+
+function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || _typeof(obj) !== "object" && typeof obj !== "function") { return { "default": obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj["default"] = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
+
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { "default": obj }; }
 
 function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (Object.getOwnPropertySymbols) { var symbols = Object.getOwnPropertySymbols(object); enumerableOnly && (symbols = symbols.filter(function (sym) { return Object.getOwnPropertyDescriptor(object, sym).enumerable; })), keys.push.apply(keys, symbols); } return keys; }
@@ -216,22 +222,20 @@ var Filters = function Filters(props) {
       filters = props.filters,
       toggleFilter = props.toggleFilter,
       toggleFilters = props.toggleFilters;
-
-  var onCheckboxChange = function onCheckboxChange(_ref10) {
-    var target = _ref10.target;
-    return toggleFilter(target.name, target.checked);
-  };
-
+  var onCheckboxChange = (0, _react.useCallback)(function (event) {
+    var name = event.target.name;
+    toggleFilter(name, !values[name]);
+  }, [toggleFilter, values]);
   var rootClassName = (0, _classnames["default"])(_filtersModule["default"].root, className);
   return /*#__PURE__*/_react["default"].createElement("form", {
     className: rootClassName
   }, /*#__PURE__*/_react["default"].createElement(_flexStack.FlexStack, {
     space: "xxsmall",
     className: _filtersModule["default"].items
-  }, Object.entries(filters).map(function (_ref11) {
-    var _ref12 = _slicedToArray(_ref11, 2),
-        name = _ref12[0],
-        data = _ref12[1];
+  }, Object.entries(filters).map(function (_ref10) {
+    var _ref11 = _slicedToArray(_ref10, 2),
+        name = _ref11[0],
+        data = _ref11[1];
 
     if (typeof (data === null || data === void 0 ? void 0 : data.defaultValue) !== 'undefined') {
       return /*#__PURE__*/_react["default"].createElement(Filter, {

package/lib/ui/filters/filters.module.css

@@ -91,7 +91,7 @@
 }
 
 .filterGroupItems {
-  max-height: calc(20 * (var(--space-small) + 2 * var(--space-xxxsmall)));
+  max-height: calc(10 * (var(--space-small) + 2 * var(--space-xxxsmall)));
   overflow: auto;
 }
 

package/lib-esm/components/bundle-modules/bundle-modules.utils.js

@@ -78,11 +78,14 @@ export const useModuleFilterByChunk = _ref => {
     }
 
     const jobsWithFilteredData = jobs.map(job => {
-      var _job$metrics;
+      var _job$metrics, _job$metrics$webpack;
+
+      const modules = job == null ? void 0 : (_job$metrics = job.metrics) == null ? void 0 : (_job$metrics$webpack = _job$metrics.webpack) == null ? void 0 : _job$metrics$webpack.modules;
+
+      if (!modules) {
+        return job;
+      }
 
-      const {
-        modules
-      } = (job == null ? void 0 : (_job$metrics = job.metrics) == null ? void 0 : _job$metrics.webpack) || {};
       const filteredModules = Object.entries(modules).reduce((agg, _ref2) => {
         let [moduleId, moduleEntry] = _ref2;
 

package/lib-esm/components/metrics-table-title/metrics-table-title.js

@@ -27,10 +27,10 @@ export const MetricsTableTitle = props => {
     })
   }, /*#__PURE__*/React.createElement(Stack, {
     space: "xxxsmall"
-  }, popoverInfo && /*#__PURE__*/React.createElement("p", null, popoverInfo), popoverHref && /*#__PURE__*/React.createElement("p", null, /*#__PURE__*/React.createElement("a", {
-    href: popoverHref,
-    target: "_blank",
-    rel: "noreferrer"
+  }, popoverInfo && /*#__PURE__*/React.createElement("p", null, popoverInfo), popoverHref && /*#__PURE__*/React.createElement("p", null, /*#__PURE__*/React.createElement("button", {
+    type: "button",
+    onClick: () => window.open(popoverHref),
+    className: css.readMoreLink
   }, I18N.READ_MORE)))));
 };
 MetricsTableTitle.propTypes = {

package/lib-esm/components/metrics-table-title/metrics-table-title.module.css

@@ -11,3 +11,20 @@
   text-transform: none;
 }
 
+.readMoreLink {
+  appearance: none;
+  background: none;
+  margin: 0;
+  border: 0;
+  padding: 0;
+  color: var(--color-primary);
+  text-decoration: underline;
+  cursor: pointer;
+}
+
+.readMoreLink:hover,
+.readMoreLink:active,
+.readMoreLink:focus {
+  color: var(--color-primary-dark);
+  text-decoration: none;
+}

package/lib-esm/components/total-size-type-title/total-size-type-title.js

@@ -21,10 +21,10 @@ export const TotalSizeTypeTitle = _ref => {
     })
   }, /*#__PURE__*/React.createElement(Stack, {
     space: "xxxsmall"
-  }, /*#__PURE__*/React.createElement("p", null, I18N.ASSET_TOTALS_INFO), /*#__PURE__*/React.createElement("p", null, /*#__PURE__*/React.createElement("a", {
-    href: config.documentation.assets,
-    target: "_blank",
-    rel: "noreferrer"
+  }, /*#__PURE__*/React.createElement("p", null, I18N.ASSET_TOTALS_INFO), /*#__PURE__*/React.createElement("p", null, /*#__PURE__*/React.createElement("button", {
+    type: "button",
+    onClick: () => window.open(config.documentation.assets),
+    className: css.readMoreLink
   }, I18N.READ_MORE)))));
 };
 TotalSizeTypeTitle.propTypes = {

package/lib-esm/components/total-size-type-title/total-size-type-title.module.css

@@ -5,3 +5,21 @@
   line-height: var(--space-medium);
   text-transform: uppercase;
 }
+
+.readMoreLink {
+  appearance: none;
+  background: none;
+  margin: 0;
+  border: 0;
+  padding: 0;
+  color: var(--color-primary);
+  text-decoration: underline;
+  cursor: pointer;
+}
+
+.readMoreLink:hover,
+.readMoreLink:active,
+.readMoreLink:focus {
+  color: var(--color-primary-dark);
+  text-decoration: none;
+}

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

@@ -3,7 +3,7 @@ import "core-js/modules/es.array.includes.js";
 
 function _extends() { _extends = Object.assign ? Object.assign.bind() : function (target) { for (var i = 1; i < arguments.length; i++) { var source = arguments[i]; for (var key in source) { if (Object.prototype.hasOwnProperty.call(source, key)) { target[key] = source[key]; } } } return target; }; return _extends.apply(this, arguments); }
 
-import React from 'react';
+import React, { useCallback } from 'react';
 import PropTypes from 'prop-types';
 import cx from 'classnames';
 import { FlexStack } from '../../layout/flex-stack';
@@ -177,22 +177,20 @@ export const Filters = props => {
     toggleFilter,
     toggleFilters
   } = props;
-
-  const onCheckboxChange = _ref6 => {
-    let {
-      target
-    } = _ref6;
-    return toggleFilter(target.name, target.checked);
-  };
-
+  const onCheckboxChange = useCallback(event => {
+    const {
+      name
+    } = event.target;
+    toggleFilter(name, !values[name]);
+  }, [toggleFilter, values]);
   const rootClassName = cx(css.root, className);
   return /*#__PURE__*/React.createElement("form", {
     className: rootClassName
   }, /*#__PURE__*/React.createElement(FlexStack, {
     space: "xxsmall",
     className: css.items
-  }, Object.entries(filters).map(_ref7 => {
-    let [name, data] = _ref7;
+  }, Object.entries(filters).map(_ref6 => {
+    let [name, data] = _ref6;
 
     if (typeof (data == null ? void 0 : data.defaultValue) !== 'undefined') {
       return /*#__PURE__*/React.createElement(Filter, {

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

@@ -91,7 +91,7 @@
 }
 
 .filterGroupItems {
-  max-height: calc(20 * (var(--space-small) + 2 * var(--space-xxxsmall)));
+  max-height: calc(10 * (var(--space-small) + 2 * var(--space-xxxsmall)));
   overflow: auto;
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bundle-stats/ui",
-  "version": "4.0.0-rc.1",
+  "version": "4.0.0-rc.5",
   "description": "UI components for BundleStats projects.",
   "main": "lib/index.js",
   "module": "lib-esm/index.js",
@@ -57,7 +57,7 @@
     "chromatic": "6.7.0",
     "classnames": "2.3.1",
     "colormap": "2.3.2",
-    "core-js": "3.23.3",
+    "core-js": "3.23.4",
     "cross-env": "7.0.3",
     "enzyme": "3.11.0",
     "enzyme-adapter-react-16": "1.15.6",
@@ -85,12 +85,12 @@
     "react-router-dom": "^5.0.0"
   },
   "dependencies": {
-    "@bundle-stats/utils": "^4.0.0-rc.1",
-    "ariakit": "2.0.0-next.31",
+    "@bundle-stats/utils": "^4.0.0-rc.5",
+    "ariakit": "2.0.0-next.33",
     "date-fns": "^2.28.0",
     "modern-css-reset": "^1.4.0",
     "query-string": "^7.1.0",
     "use-query-params": "^1.2.3"
   },
-  "gitHead": "8f28ce5d4c570ac11670f5f0f34a3c9c3ea6a162"
+  "gitHead": "68ae6f2656d4c1572efbc0f16dc04dfeec73f6a5"
 }

package/src/components/bundle-modules/bundle-modules.utils.js

@@ -87,7 +87,11 @@ export const useModuleFilterByChunk = ({ jobs, filters, chunkIds }) => {
     }
 
     const jobsWithFilteredData = jobs.map((job) => {
-      const { modules } = job?.metrics?.webpack || {};
+      const modules = job?.metrics?.webpack?.modules;
+
+      if (!modules) {
+        return job;
+      }
 
       const filteredModules = Object.entries(modules).reduce((agg, [moduleId, moduleEntry]) => {
         const match = intersection(moduleEntry.chunkIds, includedChunkIds);

package/src/components/metrics-table-title/metrics-table-title.jsx

@@ -23,9 +23,9 @@ export const MetricsTableTitle = (props) => {
             {popoverInfo && <p>{popoverInfo}</p>}
             {popoverHref && (
               <p>
-                <a href={popoverHref} target="_blank" rel="noreferrer">
+                <button type="button" onClick={() => window.open(popoverHref)} className={css.readMoreLink}>
                   {I18N.READ_MORE}
-                </a>
+                </button>
               </p>
             )}
           </Stack>

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

@@ -11,3 +11,20 @@
   text-transform: none;
 }
 
+.readMoreLink {
+  appearance: none;
+  background: none;
+  margin: 0;
+  border: 0;
+  padding: 0;
+  color: var(--color-primary);
+  text-decoration: underline;
+  cursor: pointer;
+}
+
+.readMoreLink:hover,
+.readMoreLink:active,
+.readMoreLink:focus {
+  color: var(--color-primary-dark);
+  text-decoration: none;
+}

package/src/components/total-size-type-title/total-size-type-title.jsx

@@ -17,9 +17,13 @@ export const TotalSizeTypeTitle = ({ className }) => (
       <Stack space="xxxsmall">
         <p>{I18N.ASSET_TOTALS_INFO}</p>
         <p>
-          <a href={config.documentation.assets} target="_blank" rel="noreferrer">
+          <button
+            type="button"
+            onClick={() => window.open(config.documentation.assets)}
+            className={css.readMoreLink}
+          >
             {I18N.READ_MORE}
-          </a>
+          </button>
         </p>
       </Stack>
     </HoverCard>

package/src/components/total-size-type-title/total-size-type-title.module.css

@@ -5,3 +5,21 @@
   line-height: var(--space-medium);
   text-transform: uppercase;
 }
+
+.readMoreLink {
+  appearance: none;
+  background: none;
+  margin: 0;
+  border: 0;
+  padding: 0;
+  color: var(--color-primary);
+  text-decoration: underline;
+  cursor: pointer;
+}
+
+.readMoreLink:hover,
+.readMoreLink:active,
+.readMoreLink:focus {
+  color: var(--color-primary-dark);
+  text-decoration: none;
+}

package/src/ui/filters/filters.jsx

@@ -1,7 +1,7 @@
-import React from 'react';
+import React, { useCallback } from 'react';
 import PropTypes from 'prop-types';
 import cx from 'classnames';
-import { get } from 'lodash';
+import { get, map } from 'lodash';
 
 import { FlexStack } from '../../layout/flex-stack';
 import { Dropdown } from '../dropdown';
@@ -170,7 +170,13 @@ FilterGroup.defaultProps = {
 export const Filters = (props) => {
   const { className, values, filters, toggleFilter, toggleFilters } = props;
 
-  const onCheckboxChange = ({ target }) => toggleFilter(target.name, target.checked);
+  const onCheckboxChange = useCallback(
+    (event) => {
+      const { name } = event.target;
+      toggleFilter(name, !values[name]);
+    },
+    [toggleFilter, values]
+  );
   const rootClassName = cx(css.root, className);
 
   return (

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

@@ -91,7 +91,7 @@
 }
 
 .filterGroupItems {
-  max-height: calc(20 * (var(--space-small) + 2 * var(--space-xxxsmall)));
+  max-height: calc(10 * (var(--space-small) + 2 * var(--space-xxxsmall)));
   overflow: auto;
 }