@spark-web/button 5.3.2 → 5.4.0

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # @spark-web/button
 
+## 5.4.0
+
+### Minor Changes
+
+- [#647](https://github.com/brighte-labs/spark-web/pull/647)
+  [`b6825e0`](https://github.com/brighte-labs/spark-web/commit/b6825e0fdffb297cecb033fd70e15b13a65d27a7)
+  Thanks [@michtntbrighte](https://github.com/michtntbrighte)! - Improve low
+  prominence and support filled props for button
+
+## 5.3.3
+
+### Patch Changes
+
+- [#625](https://github.com/brighte-labs/spark-web/pull/625)
+  [`b844b9c`](https://github.com/brighte-labs/spark-web/commit/b844b9c2891b92fe739015add83569efd272d223)
+  Thanks [@michtntbrighte](https://github.com/michtntbrighte)! - Upgrade button
+  spinner
+
 ## 5.3.2
 
 ### Patch Changes

package/dist/declarations/src/button-link.d.ts

@@ -5,5 +5,5 @@ export declare type ButtonLinkProps = LinkComponentProps & CommonButtonProps;
 /** The appearance of a `Button`, with the semantics of a link. */
 export declare const ButtonLink: <Comp extends import("react").ElementType<any, keyof import("react").JSX.IntrinsicElements> = "a">(props: {
     as?: Comp | undefined;
-    ref?: import("react").Ref<Comp extends "symbol" | "clipPath" | "filter" | "mask" | "marker" | "svg" | "animate" | "animateMotion" | "animateTransform" | "circle" | "defs" | "desc" | "ellipse" | "feBlend" | "feColorMatrix" | "feComponentTransfer" | "feComposite" | "feConvolveMatrix" | "feDiffuseLighting" | "feDisplacementMap" | "feDistantLight" | "feDropShadow" | "feFlood" | "feFuncA" | "feFuncB" | "feFuncG" | "feFuncR" | "feGaussianBlur" | "feImage" | "feMerge" | "feMergeNode" | "feMorphology" | "feOffset" | "fePointLight" | "feSpecularLighting" | "feSpotLight" | "feTile" | "feTurbulence" | "foreignObject" | "g" | "image" | "line" | "linearGradient" | "metadata" | "mpath" | "path" | "pattern" | "polygon" | "polyline" | "radialGradient" | "rect" | "set" | "stop" | "switch" | "text" | "textPath" | "tspan" | "use" | "view" | keyof HTMLElementTagNameMap ? (HTMLElementTagNameMap & Pick<SVGElementTagNameMap, "symbol" | "clipPath" | "filter" | "mask" | "marker" | "svg" | "animate" | "animateMotion" | "animateTransform" | "circle" | "defs" | "desc" | "ellipse" | "feBlend" | "feColorMatrix" | "feComponentTransfer" | "feComposite" | "feConvolveMatrix" | "feDiffuseLighting" | "feDisplacementMap" | "feDistantLight" | "feDropShadow" | "feFlood" | "feFuncA" | "feFuncB" | "feFuncG" | "feFuncR" | "feGaussianBlur" | "feImage" | "feMerge" | "feMergeNode" | "feMorphology" | "feOffset" | "fePointLight" | "feSpecularLighting" | "feSpotLight" | "feTile" | "feTurbulence" | "foreignObject" | "g" | "image" | "line" | "linearGradient" | "metadata" | "mpath" | "path" | "pattern" | "polygon" | "polyline" | "radialGradient" | "rect" | "set" | "stop" | "switch" | "text" | "textPath" | "tspan" | "use" | "view">)[Comp] : Comp extends new (...args: any) => any ? InstanceType<Comp> : undefined> | undefined;
+    ref?: import("react").Ref<Comp extends "symbol" | "text" | "switch" | "clipPath" | "filter" | "mask" | "marker" | "svg" | "animate" | "animateMotion" | "animateTransform" | "circle" | "defs" | "desc" | "ellipse" | "feBlend" | "feColorMatrix" | "feComponentTransfer" | "feComposite" | "feConvolveMatrix" | "feDiffuseLighting" | "feDisplacementMap" | "feDistantLight" | "feDropShadow" | "feFlood" | "feFuncA" | "feFuncB" | "feFuncG" | "feFuncR" | "feGaussianBlur" | "feImage" | "feMerge" | "feMergeNode" | "feMorphology" | "feOffset" | "fePointLight" | "feSpecularLighting" | "feSpotLight" | "feTile" | "feTurbulence" | "foreignObject" | "g" | "image" | "line" | "linearGradient" | "metadata" | "mpath" | "path" | "pattern" | "polygon" | "polyline" | "radialGradient" | "rect" | "set" | "stop" | "textPath" | "tspan" | "use" | "view" | keyof HTMLElementTagNameMap ? (HTMLElementTagNameMap & Pick<SVGElementTagNameMap, "symbol" | "text" | "switch" | "clipPath" | "filter" | "mask" | "marker" | "svg" | "animate" | "animateMotion" | "animateTransform" | "circle" | "defs" | "desc" | "ellipse" | "feBlend" | "feColorMatrix" | "feComponentTransfer" | "feComposite" | "feConvolveMatrix" | "feDiffuseLighting" | "feDisplacementMap" | "feDistantLight" | "feDropShadow" | "feFlood" | "feFuncA" | "feFuncB" | "feFuncG" | "feFuncR" | "feGaussianBlur" | "feImage" | "feMerge" | "feMergeNode" | "feMorphology" | "feOffset" | "fePointLight" | "feSpecularLighting" | "feSpotLight" | "feTile" | "feTurbulence" | "foreignObject" | "g" | "image" | "line" | "linearGradient" | "metadata" | "mpath" | "path" | "pattern" | "polygon" | "polyline" | "radialGradient" | "rect" | "set" | "stop" | "textPath" | "tspan" | "use" | "view">)[Comp] : Comp extends new (...args: any) => any ? InstanceType<Comp> : undefined> | undefined;
 } & Omit<import("react").PropsWithoutRef<import("react").ComponentProps<Comp>>, "as"> & ButtonLinkProps) => import("react").ReactElement<any, string | import("react").JSXElementConstructor<any>>;

package/dist/declarations/src/button.d.ts

@@ -22,6 +22,8 @@ export declare type ButtonProps = CommonButtonProps & {
     size?: CommonButtonProps['size'];
     /** Provide an alternate type if the button is within a form. */
     type?: 'button' | 'submit' | 'reset';
+    /** When true, the button will be filled depending on the variant. */
+    filled?: boolean;
 };
 /**
  * Buttons are used to initialize an action, their label should express what

package/dist/declarations/src/types.d.ts

@@ -38,5 +38,7 @@ export declare type ButtonStyleProps = {
     size?: ButtonSize;
     /** Sets the tone of the button. */
     tone?: ButtonTone;
+    /** Sets the background color of the button. */
+    filled?: boolean;
 };
 export {};

package/dist/declarations/src/use-button-styles.d.ts

@@ -7,7 +7,7 @@ import type { ButtonStyleProps } from "./types.js";
  * underlying `Box` component, and the second item is a CSS object that can be
  * passed to Emotion's `css` function.
  */
-export declare function useButtonStyles({ iconOnly, prominence, rounded, size, tone, }: UseButtonStylesProps): readonly [{
+export declare function useButtonStyles({ iconOnly, prominence, rounded, size, tone, filled, }: UseButtonStylesProps): readonly [{
     readonly alignItems: "center";
     readonly background: string | number | undefined;
     readonly border: import("@spark-web/theme").ResponsiveProp<string> | undefined;
@@ -95,7 +95,7 @@ export declare function useButtonStyles({ iconOnly, prominence, rounded, size, t
 }, {
     readonly fontWeight: "medium" | "light" | "bold" | "thin" | "extralight" | "regular" | "semibold" | "extrabold" | "black" | undefined;
 }];
-export declare type UseButtonStylesProps = Omit<Required<ButtonStyleProps>, 'css' | 'rounded'> & Partial<Pick<ButtonStyleProps, 'rounded'>> & {
+export declare type UseButtonStylesProps = Omit<Required<ButtonStyleProps>, 'css' | 'rounded' | 'filled'> & Partial<Pick<ButtonStyleProps, 'rounded' | 'filled'>> & {
     /** Whether the children of the button is a single icon or not. */
     iconOnly: boolean;
 };

package/dist/declarations/src/utils.d.ts

@@ -4,6 +4,7 @@ import type { SparkTheme } from '@spark-web/theme';
 import type { ButtonProminence, ButtonTone } from "./types.js";
 declare type BaseButtonStyles = {
     background: BoxProps['background'];
+    backgroundFill?: BoxProps['background'];
     border?: BoxProps['border'];
     borderWidth?: BoxProps['borderWidth'];
     textTone?: ForegroundTone;

package/dist/spark-web-button.cjs.dev.js

@@ -10,9 +10,9 @@ var react = require('react');
 var jsxRuntime = require('@emotion/react/jsx-runtime');
 var _slicedToArray = require('@babel/runtime/helpers/slicedToArray');
 var react$1 = require('@emotion/react');
-var a11y = require('@spark-web/a11y');
 var spinner = require('@spark-web/spinner');
 var text = require('@spark-web/text');
+var a11y = require('@spark-web/a11y');
 var theme = require('@spark-web/theme');
 var link = require('@spark-web/link');
 var ts = require('@spark-web/utils/ts');
@@ -127,9 +127,10 @@ var variants = {
   low: {
     primary: _objectSpread({
       background: 'surface',
+      backgroundFill: 'primarySoft',
       border: 'primary',
       borderWidth: 'large',
-      textTone: 'primary',
+      textTone: 'neutral',
       backgroundHover: 'none',
       borderHover: 'primaryHover',
       textToneHover: 'primaryHover',
@@ -317,13 +318,15 @@ function HiddenWhenLoading(_ref2) {
  * passed to Emotion's `css` function.
  */
 function useButtonStyles(_ref) {
-  var _theme$components$but, _theme$components, _theme$components2;
+  var _variant$backgroundFi, _theme$components$but, _theme$components, _theme$components2;
   var iconOnly = _ref.iconOnly,
     prominence = _ref.prominence,
     _ref$rounded = _ref.rounded,
     rounded = _ref$rounded === void 0 ? false : _ref$rounded,
     size = _ref.size,
-    tone = _ref.tone;
+    tone = _ref.tone,
+    _ref$filled = _ref.filled,
+    filled = _ref$filled === void 0 ? false : _ref$filled;
   var theme$1 = theme.useTheme();
   var focusRingStyles = a11y.useFocusRing({
     tone: tone,
@@ -341,7 +344,7 @@ function useButtonStyles(_ref) {
   };
   return [{
     alignItems: 'center',
-    background: variant === null || variant === void 0 ? void 0 : variant.background,
+    background: filled ? (_variant$backgroundFi = variant === null || variant === void 0 ? void 0 : variant.backgroundFill) !== null && _variant$backgroundFi !== void 0 ? _variant$backgroundFi : variant === null || variant === void 0 ? void 0 : variant.background : variant === null || variant === void 0 ? void 0 : variant.background,
     border: variant === null || variant === void 0 ? void 0 : variant.border,
     borderWidth: variant === null || variant === void 0 ? void 0 : variant.borderWidth,
     borderRadius: rounded ? 'full' : isLarge ? 'medium' : 'small',
@@ -394,7 +397,7 @@ function useButtonStyles(_ref) {
   }];
 }
 
-var _excluded$1 = ["aria-controls", "aria-describedby", "aria-expanded", "aria-label", "data", "disabled", "id", "loading", "onClick", "css", "rounded", "prominence", "size", "tone", "type"];
+var _excluded$1 = ["aria-controls", "aria-describedby", "aria-expanded", "aria-label", "data", "disabled", "id", "loading", "onClick", "css", "rounded", "prominence", "size", "tone", "type", "filled"];
 /**
  * Buttons are used to initialize an action, their label should express what
  * action will occur when the user interacts with it.
@@ -420,6 +423,8 @@ var Button = /*#__PURE__*/react.forwardRef(function (_ref, forwardedRef) {
     _ref$tone = _ref.tone,
     tone = _ref$tone === void 0 ? 'primary' : _ref$tone,
     type = _ref.type,
+    _ref$filled = _ref.filled,
+    filled = _ref$filled === void 0 ? false : _ref$filled,
     props = _objectWithoutProperties(_ref, _excluded$1);
   var iconOnly = Boolean(props.label);
   var _useButtonStyles = useButtonStyles({
@@ -427,7 +432,8 @@ var Button = /*#__PURE__*/react.forwardRef(function (_ref, forwardedRef) {
       rounded: rounded,
       size: size,
       tone: tone,
-      prominence: prominence
+      prominence: prominence,
+      filled: filled
     }),
     _useButtonStyles2 = _slicedToArray(_useButtonStyles, 3),
     boxProps = _useButtonStyles2[0],
@@ -460,7 +466,7 @@ var Button = /*#__PURE__*/react.forwardRef(function (_ref, forwardedRef) {
 Button.displayName = 'Button';
 function Loading(_ref2) {
   var tone = _ref2.tone;
-  return jsxRuntime.jsxs(box.Box, {
+  return jsxRuntime.jsx(box.Box, {
     as: "span",
     position: "absolute",
     top: 0,
@@ -470,12 +476,10 @@ function Loading(_ref2) {
     display: "flex",
     alignItems: "center",
     justifyContent: "center",
-    children: [jsxRuntime.jsx(a11y.VisuallyHidden, {
-      children: "button loading indicator"
-    }), jsxRuntime.jsx(spinner.Spinner, {
+    children: jsxRuntime.jsx(spinner.Spinner, {
       size: "xsmall",
       tone: tone
-    })]
+    })
   });
 }
 

package/dist/spark-web-button.cjs.prod.js

@@ -10,9 +10,9 @@ var react = require('react');
 var jsxRuntime = require('@emotion/react/jsx-runtime');
 var _slicedToArray = require('@babel/runtime/helpers/slicedToArray');
 var react$1 = require('@emotion/react');
-var a11y = require('@spark-web/a11y');
 var spinner = require('@spark-web/spinner');
 var text = require('@spark-web/text');
+var a11y = require('@spark-web/a11y');
 var theme = require('@spark-web/theme');
 var link = require('@spark-web/link');
 var ts = require('@spark-web/utils/ts');
@@ -127,9 +127,10 @@ var variants = {
   low: {
     primary: _objectSpread({
       background: 'surface',
+      backgroundFill: 'primarySoft',
       border: 'primary',
       borderWidth: 'large',
-      textTone: 'primary',
+      textTone: 'neutral',
       backgroundHover: 'none',
       borderHover: 'primaryHover',
       textToneHover: 'primaryHover',
@@ -317,13 +318,15 @@ function HiddenWhenLoading(_ref2) {
  * passed to Emotion's `css` function.
  */
 function useButtonStyles(_ref) {
-  var _theme$components$but, _theme$components, _theme$components2;
+  var _variant$backgroundFi, _theme$components$but, _theme$components, _theme$components2;
   var iconOnly = _ref.iconOnly,
     prominence = _ref.prominence,
     _ref$rounded = _ref.rounded,
     rounded = _ref$rounded === void 0 ? false : _ref$rounded,
     size = _ref.size,
-    tone = _ref.tone;
+    tone = _ref.tone,
+    _ref$filled = _ref.filled,
+    filled = _ref$filled === void 0 ? false : _ref$filled;
   var theme$1 = theme.useTheme();
   var focusRingStyles = a11y.useFocusRing({
     tone: tone,
@@ -341,7 +344,7 @@ function useButtonStyles(_ref) {
   };
   return [{
     alignItems: 'center',
-    background: variant === null || variant === void 0 ? void 0 : variant.background,
+    background: filled ? (_variant$backgroundFi = variant === null || variant === void 0 ? void 0 : variant.backgroundFill) !== null && _variant$backgroundFi !== void 0 ? _variant$backgroundFi : variant === null || variant === void 0 ? void 0 : variant.background : variant === null || variant === void 0 ? void 0 : variant.background,
     border: variant === null || variant === void 0 ? void 0 : variant.border,
     borderWidth: variant === null || variant === void 0 ? void 0 : variant.borderWidth,
     borderRadius: rounded ? 'full' : isLarge ? 'medium' : 'small',
@@ -394,7 +397,7 @@ function useButtonStyles(_ref) {
   }];
 }
 
-var _excluded$1 = ["aria-controls", "aria-describedby", "aria-expanded", "aria-label", "data", "disabled", "id", "loading", "onClick", "css", "rounded", "prominence", "size", "tone", "type"];
+var _excluded$1 = ["aria-controls", "aria-describedby", "aria-expanded", "aria-label", "data", "disabled", "id", "loading", "onClick", "css", "rounded", "prominence", "size", "tone", "type", "filled"];
 /**
  * Buttons are used to initialize an action, their label should express what
  * action will occur when the user interacts with it.
@@ -420,6 +423,8 @@ var Button = /*#__PURE__*/react.forwardRef(function (_ref, forwardedRef) {
     _ref$tone = _ref.tone,
     tone = _ref$tone === void 0 ? 'primary' : _ref$tone,
     type = _ref.type,
+    _ref$filled = _ref.filled,
+    filled = _ref$filled === void 0 ? false : _ref$filled,
     props = _objectWithoutProperties(_ref, _excluded$1);
   var iconOnly = Boolean(props.label);
   var _useButtonStyles = useButtonStyles({
@@ -427,7 +432,8 @@ var Button = /*#__PURE__*/react.forwardRef(function (_ref, forwardedRef) {
       rounded: rounded,
       size: size,
       tone: tone,
-      prominence: prominence
+      prominence: prominence,
+      filled: filled
     }),
     _useButtonStyles2 = _slicedToArray(_useButtonStyles, 3),
     boxProps = _useButtonStyles2[0],
@@ -460,7 +466,7 @@ var Button = /*#__PURE__*/react.forwardRef(function (_ref, forwardedRef) {
 Button.displayName = 'Button';
 function Loading(_ref2) {
   var tone = _ref2.tone;
-  return jsxRuntime.jsxs(box.Box, {
+  return jsxRuntime.jsx(box.Box, {
     as: "span",
     position: "absolute",
     top: 0,
@@ -470,12 +476,10 @@ function Loading(_ref2) {
     display: "flex",
     alignItems: "center",
     justifyContent: "center",
-    children: [jsxRuntime.jsx(a11y.VisuallyHidden, {
-      children: "button loading indicator"
-    }), jsxRuntime.jsx(spinner.Spinner, {
+    children: jsxRuntime.jsx(spinner.Spinner, {
       size: "xsmall",
       tone: tone
-    })]
+    })
   });
 }
 

package/dist/spark-web-button.esm.js

@@ -6,9 +6,9 @@ import { forwardRef, useRef, useCallback, Children, isValidElement, cloneElement
 import { jsx, jsxs } from '@emotion/react/jsx-runtime';
 import _slicedToArray from '@babel/runtime/helpers/esm/slicedToArray';
 import { css } from '@emotion/react';
-import { useFocusRing, VisuallyHidden } from '@spark-web/a11y';
 import { Spinner } from '@spark-web/spinner';
 import { Text } from '@spark-web/text';
+import { useFocusRing } from '@spark-web/a11y';
 import { useTheme } from '@spark-web/theme';
 import { useLinkComponent } from '@spark-web/link';
 import { forwardRefWithAs } from '@spark-web/utils/ts';
@@ -123,9 +123,10 @@ var variants = {
   low: {
     primary: _objectSpread({
       background: 'surface',
+      backgroundFill: 'primarySoft',
       border: 'primary',
       borderWidth: 'large',
-      textTone: 'primary',
+      textTone: 'neutral',
       backgroundHover: 'none',
       borderHover: 'primaryHover',
       textToneHover: 'primaryHover',
@@ -313,13 +314,15 @@ function HiddenWhenLoading(_ref2) {
  * passed to Emotion's `css` function.
  */
 function useButtonStyles(_ref) {
-  var _theme$components$but, _theme$components, _theme$components2;
+  var _variant$backgroundFi, _theme$components$but, _theme$components, _theme$components2;
   var iconOnly = _ref.iconOnly,
     prominence = _ref.prominence,
     _ref$rounded = _ref.rounded,
     rounded = _ref$rounded === void 0 ? false : _ref$rounded,
     size = _ref.size,
-    tone = _ref.tone;
+    tone = _ref.tone,
+    _ref$filled = _ref.filled,
+    filled = _ref$filled === void 0 ? false : _ref$filled;
   var theme = useTheme();
   var focusRingStyles = useFocusRing({
     tone: tone,
@@ -337,7 +340,7 @@ function useButtonStyles(_ref) {
   };
   return [{
     alignItems: 'center',
-    background: variant === null || variant === void 0 ? void 0 : variant.background,
+    background: filled ? (_variant$backgroundFi = variant === null || variant === void 0 ? void 0 : variant.backgroundFill) !== null && _variant$backgroundFi !== void 0 ? _variant$backgroundFi : variant === null || variant === void 0 ? void 0 : variant.background : variant === null || variant === void 0 ? void 0 : variant.background,
     border: variant === null || variant === void 0 ? void 0 : variant.border,
     borderWidth: variant === null || variant === void 0 ? void 0 : variant.borderWidth,
     borderRadius: rounded ? 'full' : isLarge ? 'medium' : 'small',
@@ -390,7 +393,7 @@ function useButtonStyles(_ref) {
   }];
 }
 
-var _excluded$1 = ["aria-controls", "aria-describedby", "aria-expanded", "aria-label", "data", "disabled", "id", "loading", "onClick", "css", "rounded", "prominence", "size", "tone", "type"];
+var _excluded$1 = ["aria-controls", "aria-describedby", "aria-expanded", "aria-label", "data", "disabled", "id", "loading", "onClick", "css", "rounded", "prominence", "size", "tone", "type", "filled"];
 /**
  * Buttons are used to initialize an action, their label should express what
  * action will occur when the user interacts with it.
@@ -416,6 +419,8 @@ var Button = /*#__PURE__*/forwardRef(function (_ref, forwardedRef) {
     _ref$tone = _ref.tone,
     tone = _ref$tone === void 0 ? 'primary' : _ref$tone,
     type = _ref.type,
+    _ref$filled = _ref.filled,
+    filled = _ref$filled === void 0 ? false : _ref$filled,
     props = _objectWithoutProperties(_ref, _excluded$1);
   var iconOnly = Boolean(props.label);
   var _useButtonStyles = useButtonStyles({
@@ -423,7 +428,8 @@ var Button = /*#__PURE__*/forwardRef(function (_ref, forwardedRef) {
       rounded: rounded,
       size: size,
       tone: tone,
-      prominence: prominence
+      prominence: prominence,
+      filled: filled
     }),
     _useButtonStyles2 = _slicedToArray(_useButtonStyles, 3),
     boxProps = _useButtonStyles2[0],
@@ -456,7 +462,7 @@ var Button = /*#__PURE__*/forwardRef(function (_ref, forwardedRef) {
 Button.displayName = 'Button';
 function Loading(_ref2) {
   var tone = _ref2.tone;
-  return jsxs(Box, {
+  return jsx(Box, {
     as: "span",
     position: "absolute",
     top: 0,
@@ -466,12 +472,10 @@ function Loading(_ref2) {
     display: "flex",
     alignItems: "center",
     justifyContent: "center",
-    children: [jsx(VisuallyHidden, {
-      children: "button loading indicator"
-    }), jsx(Spinner, {
+    children: jsx(Spinner, {
       size: "xsmall",
       tone: tone
-    })]
+    })
   });
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@spark-web/button",
-  "version": "5.3.2",
+  "version": "5.4.0",
   "homepage": "https://github.com/brighte-labs/spark-web#readme",
   "repository": {
     "type": "git",
@@ -21,9 +21,9 @@
     "@spark-web/box": "^5.0.0",
     "@spark-web/icon": "^5.0.0",
     "@spark-web/link": "^5.0.0",
-    "@spark-web/spinner": "^5.0.0",
+    "@spark-web/spinner": "^5.0.1",
     "@spark-web/text": "^5.1.0",
-    "@spark-web/theme": "^5.9.0",
+    "@spark-web/theme": "^5.11.0",
     "@spark-web/utils": "^5.0.0"
   },
   "devDependencies": {