@justeattakeaway/pie-select 0.7.0 → 0.7.1

package/dist/index.js

@@ -11,7 +11,7 @@ const h = class h extends b {
     this.getAttribute("v") || this.setAttribute("v", h.v);
   }
 };
-h.v = "0.7.0";
+h.v = "0.7.1";
 let g = h;
 const T = "*,*:after,*:before{box-sizing:inherit}:host{display:block}.c-select{--select-padding-block: var(--dt-spacing-c);--select-padding-inline-start: var(--dt-spacing-d);--select-padding-inline-end: 52px;--select-background-color: var(--dt-color-container-default);--select-text-color: var(--dt-color-content-default);--select-border-color: var(--dt-color-border-form);--select-font-size: calc(var(--dt-font-body-l-size) * 1px);--select-line-height: calc(var(--dt-font-body-l-line-height) * 1px);--select-height: 48px;--select-cursor: pointer;position:relative;color:var(--select-text-color);font-size:var(--select-font-size);line-height:var(--select-line-height)}.c-select select{height:var(--select-height);width:100%;border:1px solid var(--select-border-color);border-radius:var(--dt-radius-rounded-c);padding-inline-start:var(--select-padding-inline-start);padding-inline-end:var(--select-padding-inline-end);padding-block-start:var(--select-padding-block);padding-block-end:var(--select-padding-block);background-color:var(--select-background-color);font-family:inherit;font-size:inherit;line-height:inherit;color:inherit;outline:0;-webkit-appearance:none;-moz-appearance:none;appearance:none;cursor:var(--select-cursor);text-overflow:ellipsis;white-space:nowrap;overflow:hidden}.c-select select:focus-within{box-shadow:0 0 0 2px var(--dt-color-focus-inner),0 0 0 4px var(--dt-color-focus-outer);outline:none}.c-select.c-select--small{--select-padding-block: var(--dt-spacing-b);--select-height: 40px}.c-select.c-select--large{--select-padding-block: var(--dt-spacing-d);--select-height: 56px}.c-select.c-select--error{--select-border-color: var(--dt-color-support-error)}.c-select.c-select--withLeadingIcon{--select-padding-inline-start: calc(var(--dt-spacing-h) - var(--dt-spacing-a))}.c-select ::slotted([slot=leadingIcon]),.c-select .c-select-trailingIcon{position:absolute;top:50%;transform:translateY(-50%);pointer-events:none}.c-select:not(.is-disabled) ::slotted([slot=leadingIcon]),.c-select:not(.is-disabled) .c-select-trailingIcon{color:var(--dt-color-content-subdued)}.c-select ::slotted([slot=leadingIcon]){--icon-display-override: block;--icon-size-override: 24px;inset-inline-start:var(--dt-spacing-d)}.c-select .c-select-trailingIcon{inset-inline-end:var(--dt-spacing-d)}@media (hover: hover){.c-select:hover{--select-background-color: hsl(var(--dt-color-container-default-h), var(--dt-color-container-default-s), calc(var(--dt-color-container-default-l) + calc(-1 * var(--dt-color-hover-01))))}@supports (background-color: color-mix(in srgb,black,white)){.c-select:hover{--select-background-color: color-mix(in srgb, var(--dt-color-hover-01-bg) var(--dt-color-hover-01), var(--dt-color-container-default))}}}.c-select.is-disabled{--select-background-color: var(--dt-color-disabled-01);--select-border-color: var(--dt-color-disabled-01);--select-text-color: var(--dt-color-content-disabled);--select-cursor: auto}", O = ["small", "medium", "large"], A = ["default", "error"], c = {
   size: "medium",
@@ -20,17 +20,19 @@ const T = "*,*:after,*:before{box-sizing:inherit}:host{display:block}.c-select{-
   options: [],
   value: ""
 };
-var F = Object.defineProperty, P = Object.getOwnPropertyDescriptor, i = (t, e, o, r) => {
-  for (var s = r > 1 ? void 0 : r ? P(e, o) : e, a = t.length - 1, d; a >= 0; a--)
-    (d = t[a]) && (s = (r ? d(e, o, s) : d(s)) || s);
-  return r && s && F(e, o, s), s;
+var F = Object.defineProperty, P = Object.getOwnPropertyDescriptor, o = (t, e, r, i) => {
+  for (var s = i > 1 ? void 0 : i ? P(e, r) : e, a = t.length - 1, d; a >= 0; a--)
+    (d = t[a]) && (s = (i ? d(e, r, s) : d(s)) || s);
+  return i && s && F(e, r, s), s;
 };
 const x = "pie-select", v = "assistive-text";
 let l = class extends I(w(g)) {
   constructor() {
     super(...arguments), this.size = c.size, this.disabled = c.disabled, this.status = c.status, this.options = c.options, this._value = c.value, this._hasLeadingIcon = !1, this._handleChange = (t) => {
-      const e = z(t);
-      this.dispatchEvent(e), this._internals.setFormValue(this._select.value);
+      const { value: e } = this._select;
+      this._value = e, this._internals.setFormValue(e);
+      const r = z(t);
+      this.dispatchEvent(r);
     };
   }
   firstUpdated() {
@@ -64,8 +66,8 @@ let l = class extends I(w(g)) {
    * Resets the value to the default select value.
    */
   formResetCallback() {
-    const t = this.options.reduce((s, a) => a.tag === "optgroup" ? s.concat(a.options) : (s.push(a), s), []), e = t.length > 0 ? t[0].value : "", o = t.find((s) => s.selected === !0), r = o ? o.value : e;
-    this._select.value = r || "", this._internals.setFormValue(r || null), this._value = r || "", this.requestUpdate();
+    const t = this.options.reduce((s, a) => a.tag === "optgroup" ? s.concat(a.options) : (s.push(a), s), []), e = t.length > 0 ? t[0].value : "", r = t.find((s) => s.selected === !0), i = r ? r.value : e;
+    this._select.value = i || "", this._internals.setFormValue(i || null), this._value = i || "", this.requestUpdate();
   }
   _handleLeadingIconSlotchange() {
     this._hasLeadingIcon = !!this._leadingIconSlot.length;
@@ -86,12 +88,12 @@ let l = class extends I(w(g)) {
                     ${this.renderChildren(e.options)}
                 </optgroup>
             `;
-      const r = this._value !== "" ? this._value === e.value : e.selected;
+      const i = this._value !== "" ? this._value === e.value : e.selected;
       return p`
             <option
                 .value="${L(e.value || "")}"
                 ?disabled="${e.disabled}"
-                ?selected="${r}">
+                ?selected="${i}">
                 ${e.text}
             </option>
         `;
@@ -116,15 +118,15 @@ let l = class extends I(w(g)) {
     const {
       assistiveText: t,
       disabled: e,
-      status: o,
-      size: r,
+      status: r,
+      size: i,
       name: s,
       options: a,
       _hasLeadingIcon: d
     } = this, y = {
       "c-select": !0,
+      [`c-select--${i}`]: !0,
       [`c-select--${r}`]: !0,
-      [`c-select--${o}`]: !0,
       "c-select--withLeadingIcon": d,
       "is-disabled": e
     };
@@ -138,8 +140,8 @@ let l = class extends I(w(g)) {
                     name="${u(s)}"
                     ?disabled="${e}"
                     aria-describedby="${u(t ? v : void 0)}"
-                    aria-invalid="${o === "error" ? "true" : "false"}"
-                    aria-errormessage="${u(o === "error" ? v : void 0)}"
+                    aria-invalid="${r === "error" ? "true" : "false"}"
+                    aria-errormessage="${u(r === "error" ? v : void 0)}"
                     @change=${this._handleChange}>
                     ${this.renderChildren(a)}
                 </select>
@@ -151,42 +153,42 @@ let l = class extends I(w(g)) {
 };
 l.shadowRootOptions = { ...b.shadowRootOptions, delegatesFocus: !0 };
 l.styles = $(T);
-i([
+o([
   n({ type: String }),
   f(x, O, c.size)
 ], l.prototype, "size", 2);
-i([
+o([
   n({ type: Boolean })
 ], l.prototype, "disabled", 2);
-i([
+o([
   n({ type: String }),
   f(x, A, c.status)
 ], l.prototype, "status", 2);
-i([
+o([
   n({ type: String })
 ], l.prototype, "assistiveText", 2);
-i([
+o([
   n({ type: String, reflect: !0 })
 ], l.prototype, "name", 2);
-i([
+o([
   n({ type: Array })
 ], l.prototype, "options", 2);
-i([
+o([
   m("select")
 ], l.prototype, "focusTarget", 2);
-i([
+o([
   m("select")
 ], l.prototype, "_select", 2);
-i([
+o([
   S({ slot: "leadingIcon", flatten: !0 })
 ], l.prototype, "_leadingIconSlot", 2);
-i([
+o([
   C()
 ], l.prototype, "_hasLeadingIcon", 2);
-i([
+o([
   n()
 ], l.prototype, "value", 1);
-l = i([
+l = o([
   k("pie-select")
 ], l);
 export {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@justeattakeaway/pie-select",
   "description": "PIE Design System Select built using Web Components",
-  "version": "0.7.0",
+  "version": "0.7.1",
   "repository": {
     "type": "git",
     "url": "https://github.com/justeattakeaway/pie",

package/src/index.ts

@@ -159,6 +159,11 @@ export class PieSelect extends FormControlMixin(RtlMixin(PieElement)) implements
      * @param event - The change event.
      */
     private _handleChange = (event: Event) => {
+        // Update value state
+        const { value } = this._select;
+        this._value = value;
+        this._internals.setFormValue(value);
+
         // We have to create our own change event because the native one
         // does not penetrate the shadow boundary.
 
@@ -166,8 +171,6 @@ export class PieSelect extends FormControlMixin(RtlMixin(PieElement)) implements
         // Reference: https://javascript.info/shadow-dom-events#event-composed
         const customChangeEvent = wrapNativeEvent(event);
         this.dispatchEvent(customChangeEvent);
-
-        this._internals.setFormValue(this._select.value);
     };
 
     private _handleLeadingIconSlotchange () {