@bundle-stats/ui 4.0.5-beta.0 → 4.0.6

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (33) hide show
  1. package/lib/components/bundle-modules/bundle-modules.module.css +1 -0
  2. package/lib/components/summary-item/summary-item.js +15 -4
  3. package/lib/query-state.js +9 -7
  4. package/lib/ui/dropdown/dropdown.js +5 -2
  5. package/lib/ui/dropdown/dropdown.module.css +4 -2
  6. package/lib/ui/hover-card/hover-card.js +1 -7
  7. package/lib/ui/skeleton/skeleton.js +1 -1
  8. package/lib/ui/skeleton/skeleton.stories.js +4 -0
  9. package/lib/ui/tooltip/tooltip.module.css +1 -0
  10. package/lib/ui/tooltip/tooltip.stories.js +34 -17
  11. package/lib-esm/components/bundle-assets/bundle-assets.utils.js +0 -1
  12. package/lib-esm/components/bundle-modules/bundle-modules.module.css +1 -0
  13. package/lib-esm/components/bundle-packages/bundle-packages.utils.js +0 -1
  14. package/lib-esm/components/summary/summary.js +0 -1
  15. package/lib-esm/components/summary-item/summary-item.js +9 -2
  16. package/lib-esm/query-state.js +9 -10
  17. package/lib-esm/ui/dropdown/dropdown.js +5 -2
  18. package/lib-esm/ui/dropdown/dropdown.module.css +4 -2
  19. package/lib-esm/ui/filters/filters.js +0 -2
  20. package/lib-esm/ui/hover-card/hover-card.js +1 -7
  21. package/lib-esm/ui/skeleton/skeleton.js +1 -1
  22. package/lib-esm/ui/skeleton/skeleton.stories.js +4 -0
  23. package/lib-esm/ui/tooltip/tooltip.module.css +1 -0
  24. package/lib-esm/ui/tooltip/tooltip.stories.js +22 -11
  25. package/package.json +17 -17
  26. package/src/components/bundle-modules/bundle-modules.module.css +1 -0
  27. package/src/components/summary-item/summary-item.jsx +23 -13
  28. package/src/query-state.jsx +12 -9
  29. package/src/ui/dropdown/dropdown.jsx +2 -2
  30. package/src/ui/dropdown/dropdown.module.css +4 -2
  31. package/src/ui/hover-card/hover-card.jsx +0 -5
  32. package/src/ui/skeleton/skeleton.jsx +1 -1
  33. package/src/ui/tooltip/tooltip.module.css +1 -0
@@ -22,5 +22,6 @@
22
22
  }
23
23
 
24
24
  .nameText {
25
+ display: inline;
25
26
  vertical-align: middle;
26
27
  }
@@ -1,11 +1,13 @@
1
1
  "use strict";
2
2
 
3
+ 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); }
4
+
3
5
  Object.defineProperty(exports, "__esModule", {
4
6
  value: true
5
7
  });
6
8
  exports.SummaryItem = void 0;
7
9
 
8
- var _react = _interopRequireDefault(require("react"));
10
+ var _react = _interopRequireWildcard(require("react"));
9
11
 
10
12
  var _propTypes = _interopRequireDefault(require("prop-types"));
11
13
 
@@ -29,6 +31,10 @@ var _excluded = ["className", "as", "size", "id", "data", "loading", "showDelta"
29
31
 
30
32
  function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { "default": obj }; }
31
33
 
34
+ 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); }
35
+
36
+ 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; }
37
+
32
38
  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); }
33
39
 
34
40
  function _objectWithoutProperties(source, excluded) { if (source == null) return {}; var target = _objectWithoutPropertiesLoose(source, excluded); var key, i; if (Object.getOwnPropertySymbols) { var sourceSymbolKeys = Object.getOwnPropertySymbols(source); for (i = 0; i < sourceSymbolKeys.length; i++) { key = sourceSymbolKeys[i]; if (excluded.indexOf(key) >= 0) continue; if (!Object.prototype.propertyIsEnumerable.call(source, key)) continue; target[key] = source[key]; } } return target; }
@@ -38,13 +44,18 @@ function _objectWithoutPropertiesLoose(source, excluded) { if (source == null) r
38
44
  var MetricInfo = function MetricInfo(_ref) {
39
45
  var description = _ref.description,
40
46
  url = _ref.url;
47
+ // The component parent can be rendered inside an anchor tag, using button
48
+ // to avoid using nesting links
49
+ var onClick = (0, _react.useCallback)(function () {
50
+ if (url) {
51
+ window.open(url);
52
+ }
53
+ }, [url]);
41
54
  return /*#__PURE__*/_react["default"].createElement(_stack.Stack, {
42
55
  space: "xxxsmall"
43
56
  }, /*#__PURE__*/_react["default"].createElement("p", null, description), url && /*#__PURE__*/_react["default"].createElement("p", null, /*#__PURE__*/_react["default"].createElement("button", {
44
57
  type: "button",
45
- onClick: function onClick() {
46
- return window.open(url);
47
- },
58
+ onClick: onClick,
48
59
  className: _summaryItemModule["default"].readMoreLink
49
60
  }, "Read more")));
50
61
  };
@@ -13,10 +13,12 @@ var _isEqual2 = _interopRequireDefault(require("lodash/isEqual"));
13
13
 
14
14
  var _react = _interopRequireWildcard(require("react"));
15
15
 
16
- var _reactRouterDom = require("react-router-dom");
17
-
18
16
  var _useQueryParams4 = require("use-query-params");
19
17
 
18
+ var _reactRouter = require("use-query-params/adapters/react-router-5");
19
+
20
+ var _queryString = require("query-string");
21
+
20
22
  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); }
21
23
 
22
24
  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; }
@@ -40,12 +42,12 @@ function _arrayWithHoles(arr) { if (Array.isArray(arr)) return arr; }
40
42
  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); }
41
43
 
42
44
  var QueryStateProvider = function QueryStateProvider(props) {
43
- var history = (0, _reactRouterDom.useHistory)();
44
- var location = (0, _reactRouterDom.useLocation)();
45
45
  return /*#__PURE__*/_react["default"].createElement(_useQueryParams4.QueryParamProvider, _extends({
46
- ReactRouterRoute: _reactRouterDom.Route,
47
- location: location,
48
- history: history
46
+ adapter: _reactRouter.ReactRouter5Adapter,
47
+ options: {
48
+ searchStringToObject: _queryString.parse,
49
+ objectToSearchString: _queryString.stringify
50
+ }
49
51
  }, props));
50
52
  };
51
53
 
@@ -37,11 +37,14 @@ var Dropdown = function Dropdown(props) {
37
37
  className: (0, _classnames["default"])(_dropdownModule["default"].button, buttonClassName),
38
38
  tabIndex: null
39
39
  }, /*#__PURE__*/_react["default"].createElement(_flexStack.FlexStack, {
40
- space: "xxxsmall"
40
+ space: "xxxsmall",
41
+ className: _dropdownModule["default"].label
41
42
  }, glyph && /*#__PURE__*/_react["default"].createElement(_icon.Icon, {
42
43
  className: _dropdownModule["default"].labelIcon,
43
44
  glyph: glyph
44
- }), label)), /*#__PURE__*/_react["default"].createElement(_menu.Menu, {
45
+ }), label && /*#__PURE__*/_react["default"].createElement("span", {
46
+ className: _dropdownModule["default"].labelText
47
+ }, label))), /*#__PURE__*/_react["default"].createElement(_menu.Menu, {
45
48
  state: menuState,
46
49
  "aria-label": ariaLabel || label,
47
50
  className: _dropdownModule["default"].dropdown
@@ -10,8 +10,6 @@
10
10
  border: 1px solid var(--color-outline);
11
11
  border-radius: var(--radius-small);
12
12
  padding: calc(var(--space-xxsmall) - 1px);
13
- display: flex;
14
- align-items: center;
15
13
 
16
14
  background: none;
17
15
  font-size: var(--size-small);
@@ -31,6 +29,10 @@
31
29
  color: var(--color-text);
32
30
  }
33
31
 
32
+ .label {
33
+ align-items: center;
34
+ }
35
+
34
36
  .dropdown {
35
37
  position: absolute;
36
38
  left: 0;
@@ -55,11 +55,6 @@ var HoverCard = function HoverCard(props) {
55
55
  }); // Fallback to span if no href
56
56
 
57
57
  var Component = resolveComponent(as, href);
58
-
59
- var handleOnClick = function handleOnClick(event) {
60
- event.preventDefault();
61
- };
62
-
63
58
  return /*#__PURE__*/_react["default"].createElement("div", {
64
59
  className: (0, _classnames["default"])(_hoverCardModule["default"].root, className)
65
60
  }, /*#__PURE__*/_react["default"].createElement(_hovercard.HovercardAnchor, {
@@ -72,8 +67,7 @@ var HoverCard = function HoverCard(props) {
72
67
  className: (0, _classnames["default"])(_hoverCardModule["default"].hoverCard, hoverCardClassName),
73
68
  style: {
74
69
  zIndex: 10000
75
- },
76
- onClick: handleOnClick
70
+ }
77
71
  }), /*#__PURE__*/_react["default"].createElement(_hovercard.HovercardArrow, {
78
72
  size: 24
79
73
  }), typeof children === 'function' ? children({
@@ -43,6 +43,6 @@ Skeleton.propTypes = {
43
43
  };
44
44
  Skeleton.defaultProps = {
45
45
  className: '',
46
- as: 'span',
46
+ as: 'p',
47
47
  block: false
48
48
  };
@@ -33,6 +33,10 @@ var MultipleTextLines = function MultipleTextLines() {
33
33
  style: {
34
34
  outline: '1px solid magenta'
35
35
  }
36
+ }), /*#__PURE__*/_react["default"].createElement(_.Skeleton, {
37
+ style: {
38
+ outline: '1px solid pink'
39
+ }
36
40
  }));
37
41
  };
38
42
 
@@ -14,6 +14,7 @@
14
14
  padding: var(--space-xxxsmall) var(--space-xxsmall);
15
15
  border-radius: var(--radius-medium);
16
16
  background: var(--color-light);
17
+ max-width: 24em;
17
18
  color: var(--color-text);
18
19
  filter: drop-shadow(0 0 var(--space-xxxsmall) var(--color-shadow));
19
20
  will-change: filter;
@@ -1,8 +1,11 @@
1
1
  "use strict";
2
2
 
3
- var _react = _interopRequireDefault(require("react"));
3
+ Object.defineProperty(exports, "__esModule", {
4
+ value: true
5
+ });
6
+ exports["default"] = exports.LongTooltip = exports.Default = exports.DarkModeFalse = void 0;
4
7
 
5
- var _react2 = require("@storybook/react");
8
+ var _react = _interopRequireDefault(require("react"));
6
9
 
7
10
  var _stories = require("../../stories");
8
11
 
@@ -10,18 +13,32 @@ var _ = require(".");
10
13
 
11
14
  function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { "default": obj }; }
12
15
 
13
- var stories = (0, _react2.storiesOf)('UI/Tooltip', module);
14
- stories.addDecorator((0, _stories.getWrapperDecorator)({
15
- padding: '64px'
16
- }));
17
- stories.add('default', function () {
18
- return /*#__PURE__*/_react["default"].createElement(_.Tooltip, {
19
- title: "View job #100"
20
- }, "Job #100");
21
- });
22
- stories.add('darkMode:false', function () {
23
- return /*#__PURE__*/_react["default"].createElement(_.Tooltip, {
24
- title: "View job #100",
25
- darkMode: false
26
- }, "Job #100");
27
- });
16
+ var _default = {
17
+ title: 'UI/Tooltip',
18
+ component: _.Tooltip,
19
+ decorators: [(0, _stories.getWrapperDecorator)({
20
+ padding: '100px',
21
+ textAlign: 'center'
22
+ })],
23
+ args: {
24
+ title: 'View job #123'
25
+ }
26
+ };
27
+ exports["default"] = _default;
28
+
29
+ var Template = function Template(props) {
30
+ return /*#__PURE__*/_react["default"].createElement(_.Tooltip, props, "Job #123");
31
+ };
32
+
33
+ var Default = Template.bind();
34
+ exports.Default = Default;
35
+ var LongTooltip = Template.bind();
36
+ exports.LongTooltip = LongTooltip;
37
+ LongTooltip.args = {
38
+ title: 'Lorem ipsum dolor sit amet, consectetur adipiscing elit. Nam molestie neque id lectus mollis, et imperdiet libero porta.'
39
+ };
40
+ var DarkModeFalse = Template.bind();
41
+ exports.DarkModeFalse = DarkModeFalse;
42
+ DarkModeFalse.args = {
43
+ darkMode: false
44
+ };
@@ -1,5 +1,4 @@
1
1
  import _get from "lodash/get";
2
- import "core-js/modules/es.array.includes.js";
3
2
  import { ASSET_ENTRY_TYPE, ASSET_FILE_TYPE, ASSET_FILTERS, getFileType } from '@bundle-stats/utils';
4
3
  import { SORT_BY_NAME, SORT_BY_DELTA, SORT_BY_SIZE } from './bundle-assets.constants';
5
4
  export const addRowAssetFlags = row => {
@@ -22,5 +22,6 @@
22
22
  }
23
23
 
24
24
  .nameText {
25
+ display: inline;
25
26
  vertical-align: middle;
26
27
  }
@@ -1,7 +1,6 @@
1
1
  import _uniq from "lodash/uniq";
2
2
  import _flatten from "lodash/flatten";
3
3
  import _get from "lodash/get";
4
- import "core-js/modules/es.array.includes.js";
5
4
  import { PACKAGE_FILTERS } from '@bundle-stats/utils';
6
5
  import { SORT_BY_NAME, SORT_BY_DELTA, SORT_BY_SIZE } from './bundle-packages.constants'; // Get a list of duplicate packages across jobs
7
6
 
@@ -2,7 +2,6 @@ import _get from "lodash/get";
2
2
 
3
3
  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); }
4
4
 
5
- import "core-js/modules/es.array.includes.js";
6
5
  import React from 'react';
7
6
  import PropTypes from 'prop-types';
8
7
  import cx from 'classnames';
@@ -1,6 +1,6 @@
1
1
  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); }
2
2
 
3
- import React from 'react';
3
+ import React, { useCallback } from 'react';
4
4
  import PropTypes from 'prop-types';
5
5
  import cx from 'classnames';
6
6
  import { getGlobalMetricType, getMetricRunInfo } from '@bundle-stats/utils';
@@ -16,11 +16,18 @@ const MetricInfo = _ref => {
16
16
  description,
17
17
  url
18
18
  } = _ref;
19
+ // The component parent can be rendered inside an anchor tag, using button
20
+ // to avoid using nesting links
21
+ const onClick = useCallback(() => {
22
+ if (url) {
23
+ window.open(url);
24
+ }
25
+ }, [url]);
19
26
  return /*#__PURE__*/React.createElement(Stack, {
20
27
  space: "xxxsmall"
21
28
  }, /*#__PURE__*/React.createElement("p", null, description), url && /*#__PURE__*/React.createElement("p", null, /*#__PURE__*/React.createElement("button", {
22
29
  type: "button",
23
- onClick: () => window.open(url),
30
+ onClick: onClick,
24
31
  className: css.readMoreLink
25
32
  }, "Read more")));
26
33
  };
@@ -4,17 +4,16 @@ import _isEqual from "lodash/isEqual";
4
4
  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); }
5
5
 
6
6
  import React, { useCallback } from 'react';
7
- import { Route, useHistory, useLocation } from 'react-router-dom';
8
7
  import { JsonParam, QueryParamProvider, useQueryParams } from 'use-query-params';
9
- export const QueryStateProvider = props => {
10
- const history = useHistory();
11
- const location = useLocation();
12
- return /*#__PURE__*/React.createElement(QueryParamProvider, _extends({
13
- ReactRouterRoute: Route,
14
- location: location,
15
- history: history
16
- }, props));
17
- };
8
+ import { ReactRouter5Adapter } from 'use-query-params/adapters/react-router-5';
9
+ import { parse, stringify } from 'query-string';
10
+ export const QueryStateProvider = props => /*#__PURE__*/React.createElement(QueryParamProvider, _extends({
11
+ adapter: ReactRouter5Adapter,
12
+ options: {
13
+ searchStringToObject: parse,
14
+ objectToSearchString: stringify
15
+ }
16
+ }, props));
18
17
  export const useComponentQueryState = componentName => {
19
18
  const [search, setSearch] = useQueryParams({
20
19
  [componentName]: JsonParam
@@ -23,11 +23,14 @@ export const Dropdown = props => {
23
23
  className: cx(css.button, buttonClassName),
24
24
  tabIndex: null
25
25
  }, /*#__PURE__*/React.createElement(FlexStack, {
26
- space: "xxxsmall"
26
+ space: "xxxsmall",
27
+ className: css.label
27
28
  }, glyph && /*#__PURE__*/React.createElement(Icon, {
28
29
  className: css.labelIcon,
29
30
  glyph: glyph
30
- }), label)), /*#__PURE__*/React.createElement(Menu, {
31
+ }), label && /*#__PURE__*/React.createElement("span", {
32
+ className: css.labelText
33
+ }, label))), /*#__PURE__*/React.createElement(Menu, {
31
34
  state: menuState,
32
35
  "aria-label": ariaLabel || label,
33
36
  className: css.dropdown
@@ -10,8 +10,6 @@
10
10
  border: 1px solid var(--color-outline);
11
11
  border-radius: var(--radius-small);
12
12
  padding: calc(var(--space-xxsmall) - 1px);
13
- display: flex;
14
- align-items: center;
15
13
 
16
14
  background: none;
17
15
  font-size: var(--size-small);
@@ -31,6 +29,10 @@
31
29
  color: var(--color-text);
32
30
  }
33
31
 
32
+ .label {
33
+ align-items: center;
34
+ }
35
+
34
36
  .dropdown {
35
37
  position: absolute;
36
38
  left: 0;
@@ -1,5 +1,3 @@
1
- import "core-js/modules/es.array.includes.js";
2
-
3
1
  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); }
4
2
 
5
3
  import React, { useCallback } from 'react';
@@ -38,11 +38,6 @@ export const HoverCard = props => {
38
38
  }); // Fallback to span if no href
39
39
 
40
40
  const Component = resolveComponent(as, href);
41
-
42
- const handleOnClick = event => {
43
- event.preventDefault();
44
- };
45
-
46
41
  return /*#__PURE__*/React.createElement("div", {
47
42
  className: cx(css.root, className)
48
43
  }, /*#__PURE__*/React.createElement(HovercardAnchor, {
@@ -55,8 +50,7 @@ export const HoverCard = props => {
55
50
  className: cx(css.hoverCard, hoverCardClassName),
56
51
  style: {
57
52
  zIndex: 10000
58
- },
59
- onClick: handleOnClick
53
+ }
60
54
  }), /*#__PURE__*/React.createElement(HovercardArrow, {
61
55
  size: 24
62
56
  }), typeof children === 'function' ? children({
@@ -23,6 +23,6 @@ Skeleton.propTypes = {
23
23
  };
24
24
  Skeleton.defaultProps = {
25
25
  className: '',
26
- as: 'span',
26
+ as: 'p',
27
27
  block: false
28
28
  };
@@ -19,6 +19,10 @@ export const MultipleTextLines = () => /*#__PURE__*/React.createElement(React.Fr
19
19
  style: {
20
20
  outline: '1px solid magenta'
21
21
  }
22
+ }), /*#__PURE__*/React.createElement(Skeleton, {
23
+ style: {
24
+ outline: '1px solid pink'
25
+ }
22
26
  }));
23
27
  export const Block = () => /*#__PURE__*/React.createElement("div", {
24
28
  style: {
@@ -14,6 +14,7 @@
14
14
  padding: var(--space-xxxsmall) var(--space-xxsmall);
15
15
  border-radius: var(--radius-medium);
16
16
  background: var(--color-light);
17
+ max-width: 24em;
17
18
  color: var(--color-text);
18
19
  filter: drop-shadow(0 0 var(--space-xxxsmall) var(--color-shadow));
19
20
  will-change: filter;
@@ -1,15 +1,26 @@
1
1
  import React from 'react';
2
- import { storiesOf } from '@storybook/react';
3
2
  import { getWrapperDecorator } from '../../stories';
4
3
  import { Tooltip } from '.';
5
- const stories = storiesOf('UI/Tooltip', module);
6
- stories.addDecorator(getWrapperDecorator({
7
- padding: '64px'
8
- }));
9
- stories.add('default', () => /*#__PURE__*/React.createElement(Tooltip, {
10
- title: "View job #100"
11
- }, "Job #100"));
12
- stories.add('darkMode:false', () => /*#__PURE__*/React.createElement(Tooltip, {
13
- title: "View job #100",
4
+ export default {
5
+ title: 'UI/Tooltip',
6
+ component: Tooltip,
7
+ decorators: [getWrapperDecorator({
8
+ padding: '100px',
9
+ textAlign: 'center'
10
+ })],
11
+ args: {
12
+ title: 'View job #123'
13
+ }
14
+ };
15
+
16
+ const Template = props => /*#__PURE__*/React.createElement(Tooltip, props, "Job #123");
17
+
18
+ export const Default = Template.bind();
19
+ export const LongTooltip = Template.bind();
20
+ LongTooltip.args = {
21
+ title: 'Lorem ipsum dolor sit amet, consectetur adipiscing elit. Nam molestie neque id lectus mollis, et imperdiet libero porta.'
22
+ };
23
+ export const DarkModeFalse = Template.bind();
24
+ DarkModeFalse.args = {
14
25
  darkMode: false
15
- }, "Job #100"));
26
+ };
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@bundle-stats/ui",
3
- "version": "4.0.5-beta.0",
3
+ "version": "4.0.6",
4
4
  "description": "UI components for BundleStats projects.",
5
5
  "main": "lib/index.js",
6
6
  "module": "lib-esm/index.js",
@@ -36,28 +36,28 @@
36
36
  },
37
37
  "homepage": "https://github.com/relative-ci/bundle-stats/blob/master/packages/ui#readme",
38
38
  "devDependencies": {
39
- "@babel/cli": "7.18.9",
40
- "@babel/core": "7.18.9",
39
+ "@babel/cli": "7.18.10",
40
+ "@babel/core": "7.18.13",
41
41
  "@babel/plugin-proposal-class-properties": "7.18.6",
42
- "@babel/preset-env": "7.18.9",
42
+ "@babel/preset-env": "7.18.10",
43
43
  "@babel/preset-react": "7.18.6",
44
44
  "@simplyianm/babel-plugin-inline-react-svg": "1.2.0",
45
- "@storybook/addon-essentials": "6.5.9",
45
+ "@storybook/addon-essentials": "6.5.10",
46
46
  "@storybook/addon-knobs": "6.4.0",
47
47
  "@storybook/addon-postcss": "2.0.0",
48
- "@storybook/addon-storyshots": "6.5.9",
49
- "@storybook/addons": "6.5.9",
50
- "@storybook/builder-webpack5": "6.5.9",
51
- "@storybook/manager-webpack5": "6.5.9",
52
- "@storybook/react": "6.5.9",
48
+ "@storybook/addon-storyshots": "6.5.10",
49
+ "@storybook/addons": "6.5.10",
50
+ "@storybook/builder-webpack5": "6.5.10",
51
+ "@storybook/manager-webpack5": "6.5.10",
52
+ "@storybook/react": "6.5.10",
53
53
  "babel-core": "7.0.0-bridge.0",
54
- "babel-jest": "28.1.3",
54
+ "babel-jest": "29.0.0",
55
55
  "babel-plugin-lodash": "3.3.4",
56
56
  "babel-plugin-require-context-hook": "1.0.0",
57
- "chromatic": "6.7.2",
57
+ "chromatic": "6.7.4",
58
58
  "classnames": "2.3.1",
59
59
  "colormap": "2.3.2",
60
- "core-js": "3.24.1",
60
+ "core-js": "3.25.0",
61
61
  "cross-env": "7.0.3",
62
62
  "enzyme": "3.11.0",
63
63
  "enzyme-adapter-react-16": "1.15.6",
@@ -67,7 +67,7 @@
67
67
  "jest": "27.5.1",
68
68
  "jest-date-mock": "1.0.8",
69
69
  "lodash": "4.17.21",
70
- "postcss": "8.4.14",
70
+ "postcss": "8.4.16",
71
71
  "prop-types": "15.8.1",
72
72
  "react": "16.14.0",
73
73
  "react-dom": "16.14.0",
@@ -85,12 +85,12 @@
85
85
  "react-router-dom": "^5.0.0"
86
86
  },
87
87
  "dependencies": {
88
- "@bundle-stats/utils": "^4.0.5-beta.0",
88
+ "@bundle-stats/utils": "^4.0.6",
89
89
  "ariakit": "^2.0.0-next.33",
90
90
  "date-fns": "^2.28.0",
91
91
  "modern-css-reset": "^1.4.0",
92
92
  "query-string": "^7.1.0",
93
- "use-query-params": "^1.2.3"
93
+ "use-query-params": "^2.0.0"
94
94
  },
95
- "gitHead": "21c3e6e8dcf686e4aaa8205a4a0d3d9d2154cf30"
95
+ "gitHead": "0afe0d22d8cd44a9617a07d0a395f1b1a23dba2a"
96
96
  }
@@ -22,5 +22,6 @@
22
22
  }
23
23
 
24
24
  .nameText {
25
+ display: inline;
25
26
  vertical-align: middle;
26
27
  }
@@ -1,4 +1,4 @@
1
- import React from 'react';
1
+ import React, { useCallback } from 'react';
2
2
  import PropTypes from 'prop-types';
3
3
  import cx from 'classnames';
4
4
  import { getGlobalMetricType, getMetricRunInfo } from '@bundle-stats/utils';
@@ -10,18 +10,28 @@ import { Metric } from '../metric';
10
10
  import { Delta } from '../delta';
11
11
  import css from './summary-item.module.css';
12
12
 
13
- const MetricInfo = ({ description, url }) => (
14
- <Stack space="xxxsmall">
15
- <p>{description}</p>
16
- {url && (
17
- <p>
18
- <button type="button" onClick={() => window.open(url)} className={css.readMoreLink}>
19
- Read more
20
- </button>
21
- </p>
22
- )}
23
- </Stack>
24
- );
13
+ const MetricInfo = ({ description, url }) => {
14
+ // The component parent can be rendered inside an anchor tag, using button
15
+ // to avoid using nesting links
16
+ const onClick = useCallback(() => {
17
+ if (url) {
18
+ window.open(url);
19
+ }
20
+ }, [url]);
21
+
22
+ return (
23
+ <Stack space="xxxsmall">
24
+ <p>{description}</p>
25
+ {url && (
26
+ <p>
27
+ <button type="button" onClick={onClick} className={css.readMoreLink}>
28
+ Read more
29
+ </button>
30
+ </p>
31
+ )}
32
+ </Stack>
33
+ );
34
+ };
25
35
 
26
36
  MetricInfo.propTypes = {
27
37
  description: PropTypes.string.isRequired,
@@ -1,16 +1,19 @@
1
1
  import React, { useCallback } from 'react';
2
- import { Route, useHistory, useLocation } from 'react-router-dom';
3
2
  import { JsonParam, QueryParamProvider, useQueryParams } from 'use-query-params';
3
+ import { ReactRouter5Adapter } from 'use-query-params/adapters/react-router-5'
4
+ import { parse, stringify } from 'query-string'
4
5
  import { isEqual, merge } from 'lodash';
5
6
 
6
- export const QueryStateProvider = (props) => {
7
- const history = useHistory();
8
- const location = useLocation();
9
-
10
- return (
11
- <QueryParamProvider ReactRouterRoute={Route} location={location} history={history} {...props} />
12
- );
13
- };
7
+ export const QueryStateProvider = (props) => (
8
+ <QueryParamProvider
9
+ adapter={ReactRouter5Adapter}
10
+ options={{
11
+ searchStringToObject: parse,
12
+ objectToSearchString: stringify,
13
+ }}
14
+ {...props}
15
+ />
16
+ );
14
17
 
15
18
  export const useComponentQueryState = (componentName) => {
16
19
  const [search, setSearch] = useQueryParams({
@@ -15,9 +15,9 @@ export const Dropdown = (props) => {
15
15
  return (
16
16
  <div className={rootClassName}>
17
17
  <MenuButton state={menuState} className={cx(css.button, buttonClassName)} tabIndex={null}>
18
- <FlexStack space="xxxsmall">
18
+ <FlexStack space="xxxsmall" className={css.label}>
19
19
  {glyph && <Icon className={css.labelIcon} glyph={glyph} />}
20
- {label}
20
+ {label && <span className={css.labelText}>{label}</span>}
21
21
  </FlexStack>
22
22
  </MenuButton>
23
23
  <Menu state={menuState} aria-label={ariaLabel || label} className={css.dropdown}>
@@ -10,8 +10,6 @@
10
10
  border: 1px solid var(--color-outline);
11
11
  border-radius: var(--radius-small);
12
12
  padding: calc(var(--space-xxsmall) - 1px);
13
- display: flex;
14
- align-items: center;
15
13
 
16
14
  background: none;
17
15
  font-size: var(--size-small);
@@ -31,6 +29,10 @@
31
29
  color: var(--color-text);
32
30
  }
33
31
 
32
+ .label {
33
+ align-items: center;
34
+ }
35
+
34
36
  .dropdown {
35
37
  position: absolute;
36
38
  left: 0;
@@ -31,10 +31,6 @@ export const HoverCard = (props) => {
31
31
  // Fallback to span if no href
32
32
  const Component = resolveComponent(as, href);
33
33
 
34
- const handleOnClick = (event) => {
35
- event.preventDefault();
36
- };
37
-
38
34
  return (
39
35
  <div className={cx(css.root, className)}>
40
36
  <HovercardAnchor
@@ -50,7 +46,6 @@ export const HoverCard = (props) => {
50
46
  state={state}
51
47
  className={cx(css.hoverCard, hoverCardClassName)}
52
48
  style={{ zIndex: 10000 }}
53
- onClick={handleOnClick}
54
49
  >
55
50
  <HovercardArrow size={24} />
56
51
  {typeof children === 'function' ? children({ close: state.hide }) : children}
@@ -19,6 +19,6 @@ Skeleton.propTypes = {
19
19
 
20
20
  Skeleton.defaultProps = {
21
21
  className: '',
22
- as: 'span',
22
+ as: 'p',
23
23
  block: false,
24
24
  };
@@ -14,6 +14,7 @@
14
14
  padding: var(--space-xxxsmall) var(--space-xxsmall);
15
15
  border-radius: var(--radius-medium);
16
16
  background: var(--color-light);
17
+ max-width: 24em;
17
18
  color: var(--color-text);
18
19
  filter: drop-shadow(0 0 var(--space-xxxsmall) var(--color-shadow));
19
20
  will-change: filter;