@justeattakeaway/pie-button 0.10.2 → 0.12.0

package/.turbo/turbo-build.log

@@ -3,9 +3,9 @@ transforming...
 ✓ 4 modules transformed.
 rendering chunks...
 computing gzip size...
-dist/index.js  3.85 kB │ gzip: 1.61 kB
+dist/index.js  4.32 kB │ gzip: 1.65 kB
 
 [vite:dts] Start generate declaration files...
-✓ built in 8.24s
-[vite:dts] Declaration files built in 7676ms.
+✓ built in 19.30s
+[vite:dts] Declaration files built in 18529ms.
 

package/CHANGELOG.md

@@ -1,5 +1,33 @@
 # Changelog
 
+## 0.12.0
+
+### Minor Changes
+
+- [Changed] - extended readme file to mention props, events and enum exports ([#434](https://github.com/justeattakeaway/pie/pull/434)) by [@jamieomaguire](https://github.com/jamieomaguire)
+
+- [Changed] - Prevent hover and active status on disabled btns and change outline to border for safari support ([#434](https://github.com/justeattakeaway/pie/pull/434)) by [@jamieomaguire](https://github.com/jamieomaguire)
+
+- [Removed] - custom event handler and updated tests accordingly ([#434](https://github.com/justeattakeaway/pie/pull/434)) by [@jamieomaguire](https://github.com/jamieomaguire)
+
+- [Added] - isFullWidth prop to button ([#434](https://github.com/justeattakeaway/pie/pull/434)) by [@jamieomaguire](https://github.com/jamieomaguire)
+
+## 0.11.0
+
+### Minor Changes
+
+- [Changed] - Improved readme files ([#418](https://github.com/justeattakeaway/pie/pull/418)) by [@jamieomaguire](https://github.com/jamieomaguire)
+
+- [Changed] - Refactor SCSS to reduce duplication ([#416](https://github.com/justeattakeaway/pie/pull/416)) by [@jamieomaguire](https://github.com/jamieomaguire)
+
+- [Changed] - Copy playwright projects from monorepo root ([#416](https://github.com/justeattakeaway/pie/pull/416)) by [@jamieomaguire](https://github.com/jamieomaguire)
+
+- [Changed] - Split small size into expressive and productive ([#416](https://github.com/justeattakeaway/pie/pull/416)) by [@jamieomaguire](https://github.com/jamieomaguire)
+
+### Patch Changes
+
+- [Removed] the custom decorator and uses the native `customElement` decorator instead ([#417](https://github.com/justeattakeaway/pie/pull/417)) by [@fernandofranca](https://github.com/fernandofranca)
+
 ## 0.10.2
 
 ### Patch Changes

package/README.md

@@ -1,6 +1,31 @@
-# pie-button
+<p align="center">
+  <img align="center" src="../../../readme_image.png" height="200" alt="">
+</p>
 
-This button is a Web Component built using Lit.
+<p align="center">
+  <a href="https://www.npmjs.com/@justeattakeaway/pie-button">
+    <img alt="GitHub Workflow Status" src="https://img.shields.io/npm/v/@justeattakeaway/pie-button.svg">
+  </a>
+</p>
+
+# Table of Contents
+
+1. [Introduction](#pie-button)
+2. [Local Development](#local-development)
+3. [Props](#props)
+4. [Events](#events)
+   - [HTML example](#html)
+   - [Vue example (using Nuxt 3)](#vue-templates-using-nuxt-3)
+   - [React example (using Next 13)](#react-templates-using-next-13)
+5. [TypeScript Enum Exports](#typescript-enum-exports)
+6. [Testing](#testing)
+   - [Browser Tests](#browser-tests)
+   - [Visual Tests](#visual-tests)
+
+
+## `pie-button`
+
+`pie-button` is a Web Component built using the Lit library. It offers a simple and accessible button component for web applications. This component can be easily integrated into various frontend frameworks and customized through a set of properties. For TypeScript projects, it also provides exported enums for type safety and autocompletion.
 
 ## Local development
 
@@ -24,3 +49,76 @@ Local dev server using Vite (with hot module reloading)
 ```
 yarn dev
 ```
+## Props
+
+| Property    | Type      | Default         | Description                                                          |
+|-------------|-----------|-----------------|----------------------------------------------------------------------|
+| size        | `String`  | `BUTTON_SIZE.MEDIUM`    | Size of the button, one of `BUTTON_SIZE` enum values (TypeScript Enum) or a raw string value such as `'large'` |
+| type        | `String`  | `BUTTON_TYPE.SUBMIT`    | Type of the button, one of `BUTTON_TYPE` enum values (TypeScript Enum) or a raw string value such as `'submit'` |
+| variant     | `String`  | `BUTTON_VARIANT.PRIMARY` | Variant of the button, one of `BUTTON_VARIANT` enum values (TypeScript Enum) or a raw string value such as `'primary'` |
+| disabled    | `Boolean` | `false`         | If `true`, disables the button.                                      |
+| isFullWidth | `Boolean` | `false`         | If `true`, sets the button width to 100% of it's container.                            |
+
+
+## Events
+
+This component does not use any custom event handlers. In order to add event listening to this component, you can treat it like a native HTML element in your application.
+
+For example, to add a click handler in various templates:
+
+### HTML
+```html
+<!-- Other attributes omitted for clarity -->
+<pie-button onclick="e => console.log(e)">Click me!</pie-button>
+```
+
+### Vue templates (using Nuxt 3)
+```html
+<!-- Other attributes omitted for clarity -->
+<pie-button @click="handleClick">Click me!</pie-button>
+```
+
+### React templates (using Next 13)
+```html
+<!-- Other attributes omitted for clarity -->
+<PieButton onClick={handleClick}>increment</PieButton>
+
+```
+
+## TypeScript Enum Exports
+
+For TypeScript projects, we export three enums related to button properties: `BUTTON_SIZE`, `BUTTON_TYPE`, and `BUTTON_VARIANT`. You can import and use these enums to set the corresponding property values for the `pie-button` component. This ensures better type safety and autocompletion in your project.
+
+Here's an example of how to import and use the enums in a TypeScript project:
+
+```typescript
+import { BUTTON_SIZE, BUTTON_TYPE, BUTTON_VARIANT } from '@justeattakeaway/pie-button';
+
+// Using the enums to set property values
+const myButtonSize = BUTTON_SIZE.LARGE;
+const myButtonType = BUTTON_TYPE.RESET;
+const myButtonVariant = BUTTON_VARIANT.SECONDARY;
+```
+
+In your markup or JSX, you can then use these variables to set the properties for the pie-button component:
+
+```html
+<PieButton size={myButtonSize} type={myButtonType} variant={myButtonVariant}>Click me!</PieButton>
+```
+
+
+## Testing
+
+### Browser tests
+
+Run at the root of the monorepo:
+```
+yarn test:browsers --filter=pie-button
+```
+
+### Visual tests
+
+Run at the root of the monorepo:
+```
+yarn test:visual --filter=pie-button
+```

package/dist/index.js

@@ -1,80 +1,81 @@
 import { unsafeCSS as f, LitElement as h, html as g } from "lit";
 import { classMap as m } from "lit/directives/class-map.js";
-import { property as b, customElement as v } from "lit/decorators.js";
-const x = `.o-btn{border-radius:50rem;border:none;font-family:JetSansDigital;font-weight:700;cursor:pointer;user-select:none;outline:none}.o-btn:focus-visible{outline:2px solid #4996fd}.o-btn--xsmall{padding:6px 8px;min-height:32px;font-size:14px;line-height:20px}.o-btn--small{padding:8px 16px;min-height:40px;font-size:19px;line-height:28px}.o-btn--medium{padding:10px 24px;min-height:48px;font-size:20px;line-height:28px}.o-btn--large{padding:14px 24px;min-height:56px;font-size:20px;line-height:28px}.o-btn--primary{background-color:#f36805;color:#fff}.o-btn--primary:hover{background-color:#df5f05}.o-btn--primary:active{background-color:#b74e04}.o-btn--secondary{background-color:#f5f3f1;color:#242e30}.o-btn--secondary:hover{background-color:#ede9e5}.o-btn--secondary:active{background-color:#dcd4cd}.o-btn--outline{outline:1px solid #dbd9d7;background-color:#fff;color:#303d3f}.o-btn--outline:hover{background-color:#f5f5f5}.o-btn--outline:active{background-color:#e0e0e0}.o-btn--ghost{background-color:#fff;color:#242e30}.o-btn--ghost:hover{background-color:#f5f5f5}.o-btn--ghost:active{background-color:#e0e0e0}.o-btn.o-btn--is-disabled{background-color:#efedea;color:#8c999b;outline:1px solid #efedea;cursor:default}.o-btn.o-btn--is-disabled.o-btn--ghost{background-color:#fff;outline:none}
-`, p = (e, n) => function(i, t) {
-  const o = `#${t}`;
-  Object.defineProperty(i, t, {
+import { property as a, customElement as x } from "lit/decorators.js";
+const v = `.o-btn{border-radius:50rem;border:none;font-family:JetSansDigital;cursor:pointer;user-select:none;outline:none}.o-btn:focus-visible{outline:2px solid #4996fd}.o-btn--xsmall{min-block-size:32px;padding:6px 8px;font-size:14px;font-weight:700;line-height:20px}.o-btn--small-productive{min-block-size:40px;padding:8px 16px;font-size:16px;font-weight:700;line-height:24px}.o-btn--small-expressive{min-block-size:40px;padding:6px 16px;font-size:20px;font-weight:700;line-height:28px}.o-btn--medium{min-block-size:48px;padding:10px 24px;font-size:20px;font-weight:700;line-height:28px}.o-btn--large{min-block-size:56px;padding:14px 24px;font-size:20px;font-weight:700;line-height:28px}.o-btn--primary{background-color:#f36805;color:#fff}.o-btn--primary:hover:not(:disabled){background-color:#df5f05}.o-btn--primary:active:not(:disabled){background-color:#b74e04}.o-btn--secondary{background-color:#f5f3f1;color:#242e30}.o-btn--secondary:hover:not(:disabled){background-color:#ede9e5}.o-btn--secondary:active:not(:disabled){background-color:#dcd4cd}.o-btn--outline{background-color:#fff;color:#303d3f;border:1px solid #dbd9d7}.o-btn--outline:hover:not(:disabled){background-color:#f5f5f5}.o-btn--outline:active:not(:disabled){background-color:#e0e0e0}.o-btn--ghost{background-color:#fff;color:#242e30}.o-btn--ghost:hover:not(:disabled){background-color:#f5f5f5}.o-btn--ghost:active:not(:disabled){background-color:#e0e0e0}.o-btn.o-btn--is-disabled{background-color:#efedea;color:#8c999b;border:1px solid #efedea;cursor:not-allowed}.o-btn.o-btn--is-disabled:hover:not(:disabled){background-color:#e6e3de}.o-btn.o-btn--is-disabled:active:not(:disabled){background-color:#d5cfc7}.o-btn.o-btn--is-disabled.o-btn--ghost{background-color:#fff;outline:none}.o-btn--fullWidth{inline-size:100%}
+`, u = (o, n) => function(r, t) {
+  const e = `#${t}`;
+  Object.defineProperty(r, t, {
     get() {
-      return this[o];
+      return this[e];
     },
-    set(r) {
-      const s = this[o];
-      e.includes(r) ? this[o] = r : (console.error(
-        `[pie-button] Invalid value "${r}" provided for property "${t}".`,
-        `Must be one of: ${e.join(" | ")}.`,
+    set(i) {
+      const d = this[e];
+      o.includes(i) ? this[e] = i : (console.error(
+        `[pie-button] Invalid value "${i}" provided for property "${t}".`,
+        `Must be one of: ${o.join(" | ")}.`,
         `Falling back to default value: "${n}"`
-      ), this[o] = n), this.requestUpdate(t, s);
+      ), this[e] = n), this.requestUpdate(t, d);
     }
   });
 };
-var a = /* @__PURE__ */ ((e) => (e.XSMALL = "xsmall", e.SMALL = "small", e.MEDIUM = "medium", e.LARGE = "large", e))(a || {}), d = /* @__PURE__ */ ((e) => (e.SUBMIT = "submit", e.BUTTON = "button", e.RESET = "reset", e.MENU = "menu", e))(d || {}), u = /* @__PURE__ */ ((e) => (e.PRIMARY = "primary", e.SECONDARY = "secondary", e.OUTLINE = "outline", e.GHOST = "ghost", e))(u || {}), y = Object.defineProperty, M = Object.getOwnPropertyDescriptor, c = (e, n, i, t) => {
-  for (var o = t > 1 ? void 0 : t ? M(n, i) : n, r = e.length - 1, s; r >= 0; r--)
-    (s = e[r]) && (o = (t ? s(n, i, o) : s(o)) || o);
-  return t && o && y(n, i, o), o;
+var b = /* @__PURE__ */ ((o) => (o.XSMALL = "xsmall", o.SMALL_EXPRESSIVE = "small-expressive", o.SMALL_PRODUCTIVE = "small-productive", o.MEDIUM = "medium", o.LARGE = "large", o))(b || {}), c = /* @__PURE__ */ ((o) => (o.SUBMIT = "submit", o.BUTTON = "button", o.RESET = "reset", o.MENU = "menu", o))(c || {}), p = /* @__PURE__ */ ((o) => (o.PRIMARY = "primary", o.SECONDARY = "secondary", o.OUTLINE = "outline", o.GHOST = "ghost", o))(p || {}), y = Object.defineProperty, M = Object.getOwnPropertyDescriptor, l = (o, n, r, t) => {
+  for (var e = t > 1 ? void 0 : t ? M(n, r) : n, i = o.length - 1, d; i >= 0; i--)
+    (d = o[i]) && (e = (t ? d(n, r, e) : d(e)) || e);
+  return t && e && y(n, r, e), e;
 };
-function k(e) {
-  return (n) => {
-    if (!customElements.get(e))
-      return v(e)(n);
-  };
-}
-let l = class extends h {
+let s = class extends h {
   constructor() {
-    super(...arguments), this.size = a.MEDIUM, this.type = d.SUBMIT, this.variant = u.PRIMARY, this.disabled = !1;
+    super(...arguments), this.size = b.MEDIUM, this.type = c.SUBMIT, this.variant = p.PRIMARY, this.disabled = !1, this.isFullWidth = !1;
   }
   render() {
-    const { size: e, type: n, variant: i, disabled: t } = this, o = {
+    const {
+      size: o,
+      type: n,
+      variant: r,
+      disabled: t,
+      isFullWidth: e
+    } = this, i = {
       "o-btn": !0,
-      [`o-btn--${e}`]: e,
-      [`o-btn--${i}`]: i,
-      "o-btn--is-disabled": t
-    }, r = () => {
-      const s = new CustomEvent("CustomEvent", { detail: "WC event dispatched" });
-      console.info("WC event dispatched"), this.dispatchEvent(s);
+      [`o-btn--${o}`]: o,
+      [`o-btn--${r}`]: r,
+      "o-btn--is-disabled": t,
+      "o-btn--fullWidth": e
     };
     return g`
             <button
-                class=${m(o)}
+                class=${m(i)}
                 type=${n}
                 ?disabled=${t}
-                @click="${r}">
+                ?isFullWidth=${e}>
                 <slot></slot>
             </button>`;
   }
 };
-l.styles = f(x);
-c([
-  b(),
-  p(Object.values(a), a.MEDIUM)
-], l.prototype, "size", 2);
-c([
-  b(),
-  p(Object.values(d), d.SUBMIT)
-], l.prototype, "type", 2);
-c([
-  b(),
-  p(Object.values(u), u.PRIMARY)
-], l.prototype, "variant", 2);
-c([
-  b({ type: Boolean, reflect: !0 })
-], l.prototype, "disabled", 2);
-l = c([
-  k("pie-button")
-], l);
+s.styles = f(v);
+l([
+  a(),
+  u(Object.values(b), b.MEDIUM)
+], s.prototype, "size", 2);
+l([
+  a(),
+  u(Object.values(c), c.SUBMIT)
+], s.prototype, "type", 2);
+l([
+  a(),
+  u(Object.values(p), p.PRIMARY)
+], s.prototype, "variant", 2);
+l([
+  a({ type: Boolean, reflect: !0 })
+], s.prototype, "disabled", 2);
+l([
+  a({ type: Boolean, reflect: !0 })
+], s.prototype, "isFullWidth", 2);
+s = l([
+  x("pie-button")
+], s);
 export {
-  a as BUTTON_SIZE,
-  d as BUTTON_TYPE,
-  u as BUTTON_VARIANT,
-  l as PieButton
+  b as BUTTON_SIZE,
+  c as BUTTON_TYPE,
+  p as BUTTON_VARIANT,
+  s as PieButton
 };

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

@@ -3,7 +3,8 @@
  */
 export declare enum BUTTON_SIZE {
     XSMALL = "xsmall",
-    SMALL = "small",
+    SMALL_EXPRESSIVE = "small-expressive",
+    SMALL_PRODUCTIVE = "small-productive",
     MEDIUM = "medium",
     LARGE = "large"
 }

package/dist/types/src/defs.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"defs.d.ts","sourceRoot":"","sources":["../../../src/defs.ts"],"names":[],"mappings":"AAAA;;GAEG;AACH,oBAAY,WAAW;IACnB,MAAM,WAAW;IACjB,KAAK,UAAU;IACf,MAAM,WAAW;IACjB,KAAK,UAAU;CAClB;AAED;;GAEG;AACH,oBAAY,WAAW;IACnB,MAAM,WAAW;IACjB,MAAM,WAAW;IACjB,KAAK,UAAU;IACf,IAAI,SAAS;CAChB;AAED;;GAEG;AACH,oBAAY,cAAc;IACtB,OAAO,YAAY;IACnB,SAAS,cAAc;IACvB,OAAO,YAAY;IACnB,KAAK,UAAU;CAClB"}
+{"version":3,"file":"defs.d.ts","sourceRoot":"","sources":["../../../src/defs.ts"],"names":[],"mappings":"AAAA;;GAEG;AACH,oBAAY,WAAW;IACnB,MAAM,WAAW;IACjB,gBAAgB,qBAAqB;IACrC,gBAAgB,qBAAqB;IACrC,MAAM,WAAW;IACjB,KAAK,UAAU;CAClB;AAED;;GAEG;AACH,oBAAY,WAAW;IACnB,MAAM,WAAW;IACjB,MAAM,WAAW;IACjB,KAAK,UAAU;IACf,IAAI,SAAS;CAChB;AAED;;GAEG;AACH,oBAAY,cAAc;IACtB,OAAO,YAAY;IACnB,SAAS,cAAc;IACvB,OAAO,YAAY;IACnB,KAAK,UAAU;CAClB"}

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

@@ -6,6 +6,7 @@ export declare class PieButton extends LitElement {
     type: BUTTON_TYPE;
     variant: BUTTON_VARIANT;
     disabled: boolean;
+    isFullWidth: boolean;
     render(): import("lit-html").TemplateResult<1>;
     static styles: import("lit").CSSResult;
 }

package/dist/types/src/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAmB,MAAM,KAAK,CAAC;AAMlD,OAAO,EAAE,WAAW,EAAE,WAAW,EAAE,cAAc,EAAE,MAAM,QAAQ,CAAC;AAGlE,OAAO,EAAE,WAAW,EAAE,WAAW,EAAE,cAAc,EAAE,CAAC;AAWpD,qBACa,SAAU,SAAQ,UAAU;IAGrC,IAAI,EAAG,WAAW,CAAsB;IAIxC,IAAI,EAAG,WAAW,CAAsB;IAIxC,OAAO,EAAG,cAAc,CAA0B;IAGlD,QAAQ,EAAG,OAAO,CAAS;IAE3B,MAAM;IA2BN,MAAM,CAAC,MAAM,0BAAqB;CACrC;AAED,OAAO,CAAC,MAAM,CAAC;IACX,UAAU,qBAAqB;QAC3B,YAAY,EAAE,SAAS,CAAC;KAC3B;CACJ"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAmB,MAAM,KAAK,CAAC;AAMlD,OAAO,EAAE,WAAW,EAAE,WAAW,EAAE,cAAc,EAAE,MAAM,QAAQ,CAAC;AAGlE,OAAO,EAAE,WAAW,EAAE,WAAW,EAAE,cAAc,EAAE,CAAC;AAEpD,qBACa,SAAU,SAAQ,UAAU;IAGjC,IAAI,EAAG,WAAW,CAAsB;IAIxC,IAAI,EAAG,WAAW,CAAsB;IAIxC,OAAO,EAAG,cAAc,CAA0B;IAGlD,QAAQ,UAAS;IAGjB,WAAW,UAAS;IAExB,MAAM;IAwBN,MAAM,CAAC,MAAM,0BAAqB;CACrC;AAED,OAAO,CAAC,MAAM,CAAC;IACX,UAAU,qBAAqB;QAC3B,YAAY,EAAE,SAAS,CAAC;KAC3B;CACJ"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@justeattakeaway/pie-button",
-  "version": "0.10.2",
+  "version": "0.12.0",
   "description": "PIE design system button built using web components",
   "type": "module",
   "main": "dist/index.js",

package/playwright/index.html

@@ -31,11 +31,24 @@
     body {
         font-feature-settings: "tnum"; /* Enable tabular numbers */
     }
+    /* basic styles to center align components and give them some spacing */
+    #root {
+        padding: 1em;
+    }
+
+    #root > * {
+        display: block;
+        margin-inline: auto;
+    }
+
+    #root > * + * {
+        margin-top: 1em;
+    }
 </style>
     <title>Testing Page</title>
-  </head>
-  <body>
+</head>
+<body>
     <div id="root"></div>
     <script type="module" src="./index.ts"></script>
-  </body>
-</html>
+</body>
+</html>

package/playwright-lit-visual.config.ts

@@ -33,11 +33,20 @@ export default defineConfig({
   /* Configure projects for major browsers */
   projects: [
     {
-      name: 'component:chrome',
+      name: 'visual:desktop',
+      grepInvert: /@mobile/,
       use: {
         ...devices['Desktop Chrome'],
       },
       testMatch: ['**/test/visual/*.spec.ts']
-    }
+    },
+    {
+      name: 'visual:mobile',
+      grep: /@mobile/,
+      use: {
+        ...devices['Pixel 5'],
+      },
+      testMatch: ['**/test/visual/*.spec.ts']
+    },
   ],
-});
+});

package/src/button.scss

@@ -1,110 +1,160 @@
 @use '@justeat/pie-design-tokens/dist/jet.scss' as dt;
 
+$button-height-xs: 32px;
+$button-height-s: 40px;
+$button-height-m: 48px;
+$button-height-l: 56px;
+
+$button-border-radius: dt.$radius-rounded-e;
+$button-font: dt.$font-interactive-m-family;
+$button-focus: dt.$color-focus;
+
+$button-bg-color-primary: dt.$color-interactive-brand;
+$button-text-color-primary: dt.$color-content-interactive-primary;
+
+$button-bg-color-secondary: dt.$color-interactive-secondary;
+$button-text-color-secondary: dt.$color-content-interactive-secondary;
+
+$button-bg-color-outline: dt.$color-container-default;
+$button-text-color-outline: dt.$color-content-interactive-tertiary;
+$button-outline-color-outline: dt.$color-border-strong;
+
+$button-bg-color-ghost: dt.$color-container-default;
+$button-text-color-ghost: dt.$color-content-link;
+$button-disabled-bg-color-ghost: dt.$color-container-default;
+
+$button-bg-color-disabled: dt.$color-disabled-01;
+$button-text-color-disabled: dt.$color-content-disabled;
+$button-outline-color-disabled: dt.$color-disabled-01;
+
+// Sets up the size of a button such as xsmall or large
+@mixin button-sizing($min-block-size, $padding-block, $padding-inline, $font-size, $font-weight, $line-height) {
+    min-block-size: $min-block-size;
+    padding: $padding-block $padding-inline;
+    font-size: $font-size * 1px;
+    font-weight: $font-weight;
+    line-height: $line-height * 1px;
+}
+
+// 'Themes' a button for it's particular variant, such as primary, secondary, outline, ghost
+@mixin button-variant($bg-color, $text-color, $outline-color: null) {
+    background-color: $bg-color;
+    color: $text-color;
+
+    &:hover:not(:disabled) {
+        background-color: darken($bg-color, dt.$color-hover-01);
+    }
+
+    &:active:not(:disabled) {
+        background-color: darken($bg-color, dt.$color-active-01);
+    }
+
+    @if $outline-color != null {
+        border: 1px solid $outline-color;
+    }
+}
+
+// Base button styles
 .o-btn {
-    border-radius: dt.$radius-rounded-e;
+    border-radius: $button-border-radius;
     border: none;
-    font-family: dt.$font-interactive-m-family;
-    font-weight: dt.$font-interactive-m-weight;
+    font-family: $button-font;
 
     cursor: pointer;
     user-select: none;
     outline: none;
 
     &:focus-visible {
-        outline: 2px solid dt.$color-focus;
+        outline: 2px solid $button-focus;
     }
 }
 
-.o-btn--xsmall {
-    padding: 6px dt.$spacing-b;
-    min-height: 32px;
-    font-size: dt.$font-size-14 * 1px;
-    line-height: dt.$font-size-14-line-height * 1px;
-}
-
-.o-btn--small {
-    padding: 8px dt.$spacing-d;
-    min-height: 40px;
-    font-size: dt.$font-size-19 * 1px;
-    line-height: dt.$font-size-19-line-height * 1px;
-}
-
-.o-btn--medium {
-    padding: 10px dt.$spacing-e;
-    min-height: 48px;
-    font-size: dt.$font-size-20 * 1px;
-    line-height: dt.$font-size-20-line-height * 1px;
-}
-
-.o-btn--large {
-    padding: 14px dt.$spacing-e;
-    min-height: 56px;
-    font-size: dt.$font-size-20 * 1px;
-    line-height: dt.$font-size-20-line-height * 1px;
+// Sizes
+$button-sizes: (
+    'xsmall': (
+        'min-block-size': $button-height-xs,
+        'padding-block': 6px,
+        'padding-inline': dt.$spacing-b,
+        'font-size': dt.$font-interactive-xs-size,
+        'font-weight': dt.$font-interactive-xs-weight,
+        'line-height': dt.$font-interactive-xs-line-height
+    ),
+    'small-productive': (
+        'min-block-size': $button-height-s,
+        'padding-block': dt.$spacing-b,
+        'padding-inline': dt.$spacing-d,
+        'font-size': dt.$font-interactive-s-size,
+        'font-weight': dt.$font-interactive-s-weight,
+        'line-height': dt.$font-interactive-s-line-height
+    ),
+    'small-expressive': (
+        'min-block-size': $button-height-s,
+        'padding-block': 6px,
+        'padding-inline': dt.$spacing-d,
+        'font-size': dt.$font-interactive-l-size,
+        'font-weight': dt.$font-interactive-l-weight,
+        'line-height': dt.$font-interactive-l-line-height
+    ),
+    'medium': (
+        'min-block-size': $button-height-m,
+        'padding-block': 10px,
+        'padding-inline': dt.$spacing-e,
+        'font-size': dt.$font-interactive-l-size,
+        'font-weight': dt.$font-interactive-l-weight,
+        'line-height': dt.$font-interactive-l-line-height
+    ),
+    'large': (
+        'min-block-size': $button-height-l,
+        'padding-block': 14px,
+        'padding-inline': dt.$spacing-e,
+        'font-size': dt.$font-interactive-l-size,
+        'font-weight': dt.$font-interactive-l-weight,
+        'line-height': dt.$font-interactive-l-line-height
+    )
+);
+
+@each $size, $values in $button-sizes {
+    .o-btn--#{$size} {
+        @include button-sizing(
+            map-get($values, 'min-block-size'),
+            map-get($values, 'padding-block'),
+            map-get($values, 'padding-inline'),
+            map-get($values, 'font-size'),
+            map-get($values, 'font-weight'),
+            map-get($values, 'line-height')
+        );
+    }
 }
 
+// Variants
 .o-btn--primary {
-    background-color: dt.$color-interactive-brand;
-    color: dt.$color-content-interactive-primary;
-
-    &:hover {
-        background-color: darken(dt.$color-interactive-brand, dt.$color-hover-01);
-    }
-
-    &:active {
-        background-color: darken(dt.$color-interactive-brand, dt.$color-active-01);
-    }
+    @include button-variant($button-bg-color-primary, $button-text-color-primary);
 }
 
 .o-btn--secondary {
-    background-color: dt.$color-interactive-secondary;
-    color: dt.$color-content-interactive-secondary;
-
-    &:hover {
-        background-color: darken(dt.$color-interactive-secondary, dt.$color-hover-01);
-    }
-
-    &:active {
-        background-color: darken(dt.$color-interactive-secondary, dt.$color-active-01);
-    }
+    @include button-variant($button-bg-color-secondary, $button-text-color-secondary);
 }
 
 .o-btn--outline {
-    outline: 1px solid dt.$color-border-strong;
-    background-color: dt.$color-container-default;
-    color: dt.$color-content-interactive-tertiary;
-
-    &:hover {
-        background-color: darken(dt.$color-container-default, dt.$color-hover-01);
-    }
-
-    &:active {
-        background-color: darken(dt.$color-container-default, dt.$color-active-01);
-    }
+    @include button-variant($button-bg-color-outline, $button-text-color-outline, $button-outline-color-outline);
 }
 
 .o-btn--ghost {
-    background-color: dt.$color-container-default;
-    color: dt.$color-content-link;
-
-    &:hover {
-        background-color: darken(dt.$color-container-default, dt.$color-hover-01);
-    }
-
-    &:active {
-        background-color: darken(dt.$color-container-default, dt.$color-active-01);
-    }
+    @include button-variant($button-bg-color-ghost, $button-text-color-ghost);
 }
 
 .o-btn.o-btn--is-disabled {
-    background-color: dt.$color-disabled-01;
-    color: dt.$color-content-disabled;
-    outline: 1px solid dt.$color-disabled-01;
+    @include button-variant($button-bg-color-disabled, $button-text-color-disabled, $button-outline-color-disabled);
 
-    cursor: default;
+    cursor: not-allowed;
 
     &.o-btn--ghost {
-        background-color: dt.$color-container-default;
+        background-color: $button-disabled-bg-color-ghost;
         outline: none;
     }
-}
+}
+
+// Additional modifiers
+.o-btn--fullWidth {
+    inline-size: 100%;
+}

package/src/defs.ts

@@ -3,7 +3,8 @@
  */
 export enum BUTTON_SIZE {
     XSMALL = 'xsmall',
-    SMALL = 'small',
+    SMALL_EXPRESSIVE = 'small-expressive',
+    SMALL_PRODUCTIVE = 'small-productive',
     MEDIUM = 'medium',
     LARGE = 'large'
 }

package/src/index.ts

@@ -9,54 +9,45 @@ import { BUTTON_SIZE, BUTTON_TYPE, BUTTON_VARIANT } from './defs';
 // Valid values available to consumers
 export { BUTTON_SIZE, BUTTON_TYPE, BUTTON_VARIANT };
 
-// TODO: Extract as a utility function in a shared package
-function defineCustomElement(elementName:string) {
-    return (elementClass:typeof LitElement) => {
-        if(customElements.get(elementName)) return;
-
-        return customElement(elementName)(elementClass);
-    }
-}
-
-@defineCustomElement('pie-button')
+@customElement('pie-button')
 export class PieButton extends LitElement {
     @property()
     @validPropertyValues(Object.values(BUTTON_SIZE), BUTTON_SIZE.MEDIUM)
-    size : BUTTON_SIZE = BUTTON_SIZE.MEDIUM;
+        size : BUTTON_SIZE = BUTTON_SIZE.MEDIUM;
 
     @property()
     @validPropertyValues(Object.values(BUTTON_TYPE), BUTTON_TYPE.SUBMIT)
-    type : BUTTON_TYPE = BUTTON_TYPE.SUBMIT;
+        type : BUTTON_TYPE = BUTTON_TYPE.SUBMIT;
 
     @property()
     @validPropertyValues(Object.values(BUTTON_VARIANT), BUTTON_VARIANT.PRIMARY)
-    variant : BUTTON_VARIANT = BUTTON_VARIANT.PRIMARY;
+        variant : BUTTON_VARIANT = BUTTON_VARIANT.PRIMARY;
+
+    @property({ type: Boolean, reflect: true })
+        disabled = false;
 
-    @property({type: Boolean, reflect: true})
-    disabled : boolean = false;
+    @property({ type: Boolean, reflect: true })
+        isFullWidth = false;
 
     render () {
-        const { size, type, variant, disabled } = this;
+        const {
+            size, type, variant, disabled, isFullWidth,
+        } = this;
 
         const classes = {
             'o-btn': true,
             [`o-btn--${size}`]: size,
             [`o-btn--${variant}`]: variant,
             'o-btn--is-disabled': disabled,
+            'o-btn--fullWidth': isFullWidth,
         };
 
-        const raiseWCEvent = () => {
-            const event = new CustomEvent('CustomEvent', { detail: 'WC event dispatched' })
-            console.info('WC event dispatched')
-            this.dispatchEvent(event)
-        }
-
         return html`
             <button
                 class=${classMap(classes)}
                 type=${type}
                 ?disabled=${disabled}
-                @click="${raiseWCEvent}">
+                ?isFullWidth=${isFullWidth}>
                 <slot></slot>
             </button>`;
     }

package/test/component/pie-button.spec.ts

@@ -6,46 +6,52 @@ const sizes = Object.values(BUTTON_SIZE);
 const variants = Object.values(BUTTON_VARIANT);
 const disabledStates = [true, false];
 
-variants.forEach(variant => {
-  test(`should render - ${variant}`, async ({ mount }) => {
+variants.forEach((variant) => {
+    test(`should render - ${variant}`, async ({ mount }) => {
+        for (const size of sizes) {
+            for (const isDisabled of disabledStates) {
+                const component = await mount(
+                    PieButton,
+                    {
+                        props: {
+                            size,
+                            variant,
+                            disabled: isDisabled,
+                        },
+                        slots: {
+                            default: `Hello, ${size} ${variant} Button!`,
+                        },
+                    },
+                );
 
-    for (const size of sizes) {
-      for (const isDisabled of disabledStates) {
-        const component = await mount(PieButton,
-          {
+                await expect(component).toContainText(`Hello, ${size} ${variant} Button!`);
+            }
+        }
+    });
+});
+
+test('should correctly work with native click events', async ({ mount }) => {
+    const messages: string[] = [];
+    const expectedEventMessage = 'Native event dispatched';
+    const component = await mount(
+        PieButton,
+        {
             props: {
-              size,
-              variant,
-              disabled: isDisabled
+                size: BUTTON_SIZE.LARGE,
+                variant: BUTTON_VARIANT.PRIMARY,
             },
             slots: {
-              default: `Hello, ${size} ${variant} Button!`
+                default: 'Click me!',
             },
-          });
-
-        await expect(component).toContainText(`Hello, ${size} ${variant} Button!`);
-      }
-    }
-  });
-})
-
-test('should emit an event when clicked', async ({ mount }) => {
-  const messages: string[] = [];
-  const component = await mount(PieButton,
-    {
-      props: {
-        size: BUTTON_SIZE.LARGE,
-        variant: BUTTON_VARIANT.PRIMARY
-      },
-      slots: {
-        default: 'Click me!'
-      },
-      on: {
-        CustomEvent: (data: string) => messages.push(data),
-      },
-    });
+            on: {
+                click: () => {
+                    messages.push(expectedEventMessage);
+                },
+            },
+        },
+    );
 
-  await component.click();
+    await component.click();
 
-  expect(messages).toEqual(['WC event dispatched'])
-});
+    expect(messages).toEqual([expectedEventMessage]);
+});

package/test/visual/pie-button.spec.ts

@@ -1,31 +1,52 @@
 import { test } from '@sand4rt/experimental-ct-web';
+import percySnapshot from '@percy/playwright';
 import { PieButton } from '@/index';
-import percySnapshot from '@percy/playwright'
 import { BUTTON_SIZE, BUTTON_TYPE, BUTTON_VARIANT } from '@/defs';
 
 const sizes = Object.values(BUTTON_SIZE);
 const variants = Object.values(BUTTON_VARIANT);
 const disabledStates = [true, false];
 
-variants.forEach(variant => {
-  test(`should render - ${variant}`, async ({ page, mount }) => {
+variants.forEach((variant) => {
+    test(`should render - ${variant}`, async ({ page, mount }) => {
+        for (const size of sizes) {
+            for (const disabledState of disabledStates) {
+                await mount(
+                    PieButton,
+                    {
+                        props: {
+                            type: BUTTON_TYPE.BUTTON,
+                            size,
+                            variant,
+                            disabled: disabledState,
+                            isFullWidth: false,
+                        },
+                        slots: {
+                            default: `Hello, ${size} ${variant} Button!`,
+                        },
+                    },
+                );
+            }
 
-    for (const size of sizes) {
-      for (const isDisabled of disabledStates) {
-        await mount(PieButton,
-          {
-            props: {
-              type: BUTTON_TYPE.BUTTON,
-              size,
-              variant,
-              disabled: isDisabled
-            },
-            slots: {
-              default: `Hello, ${size} ${variant} Button!`
-            },
-          });
-      }
-    }
-    await percySnapshot(page, `PIE Button - ${variant}`);
-  });
-})
+            for (const disabledState of disabledStates) {
+                await mount(
+                    PieButton,
+                    {
+                        props: {
+                            type: BUTTON_TYPE.BUTTON,
+                            size,
+                            variant,
+                            disabled: disabledState,
+                            isFullWidth: true,
+                        },
+                        slots: {
+                            default: `Hello, ${size} ${variant} Button!`,
+                        },
+                    },
+                );
+            }
+        }
+
+        await percySnapshot(page, `PIE Button - ${variant}`);
+    });
+});