@sondor-technology/sondor-components 1.0.3 → 1.0.5

package/dist/styles/index.css

@@ -69,6 +69,8 @@
 :root[data-theme=light] {
   --st-color-bg: var(--st-white);
   --st-color-surface: var(--st-gray-50);
+  --st-color-surface-muted: color-mix(in srgb, var(--st-color-surface), black 15%);
+  --st-color-surface-hover: color-mix(in srgb, var(--st-color-surface), white 10%);
   --st-color-border: var(--st-gray-200);
   --st-color-text: var(--st-gray-900);
   --st-color-text-muted: var(--st-gray-700);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sondor-technology/sondor-components",
-  "version": "1.0.3",
+  "version": "1.0.5",
   "description": "",
   "main": "src/index.js",
   "types": "dist/index.d.ts",

package/src/components/button-group/button-group.ts

@@ -65,3 +65,9 @@ export class StButtonGroup extends LitElement {
     `;
   }
 }
+
+declare global {
+  interface HTMLElementTagNameMap {
+    "st-button-group": StButtonGroup;
+  }
+}

package/src/components/checkbox/checkbox.ts

@@ -11,7 +11,6 @@ export class StCheckbox extends LitElement {
 
   @property({type: String, reflect: true}) size: "sm" | "md" | "lg" = "md";
   @property({type: String, reflect: true}) name = "";
-  // Checkboxes default to 'on' natively if no value is provided
   @property({type: String}) value = "on";
 
   @property({type: Boolean, reflect: true}) checked = false;
@@ -20,21 +19,20 @@ export class StCheckbox extends LitElement {
   @property({type: Boolean, reflect: true}) disabled = false;
   @property({type: Boolean, reflect: true}) invalid = false;
   @property({type: Boolean, reflect: true}) required = false;
+  @property({type: String}) errorMessage = "";
 
   @property({attribute: "aria-label"}) ariaLabel = "";
 
-constructor() {
+  constructor() {
     super();
     this._internals = this.attachInternals();
 
-    // 1. Trigger error state if the user clicks away and the field is invalid
     this.addEventListener('focusout', () => {
-      this.invalid = !this._inputEl.validity.valid;
+      this.invalid = !this.checkValidity();
     });
 
-    // 2. Trigger error state if the st-form tries to submit and fails
     this.addEventListener('invalid', () => {
-      this.invalid = true;
+      this.invalid = !this.checkValidity();
     });
   }
 
@@ -67,7 +65,6 @@ constructor() {
         font-size: var(--st-font-size-base);
       }
 
-      /* Visually hide the native input but keep it in the DOM for focus/screen readers */
       input {
         position: absolute;
         width: var(--checkbox-size);
@@ -80,7 +77,6 @@ constructor() {
         border-width: 0;
       }
 
-      /* Custom Checkbox Box */
       .control {
         display: flex;
         align-items: center;
@@ -92,30 +88,25 @@ constructor() {
         background-color: var(--st-color-bg);
         transition: all 0.2s ease;
         flex-shrink: 0;
-        /* Aligns the box nicely with standard text line-height */
         margin-top: 0.125rem;
       }
 
-      /* Hover State */
       label:hover .control {
         border-color: var(--st-color-primary-hover);
       }
 
-      /* Focus State */
       input:focus-visible + .control {
         box-shadow:
           0 0 0 2px var(--st-color-bg),
           0 0 0 4px var(--st-color-primary);
       }
 
-      /* Checked & Indeterminate States */
       :host([checked]) .control,
       :host([indeterminate]) .control {
         border-color: var(--st-color-primary);
         background-color: var(--st-color-primary);
       }
 
-      /* SVG Icons */
       svg {
         width: var(--checkbox-icon-size);
         height: var(--checkbox-icon-size);
@@ -131,14 +122,12 @@ constructor() {
           opacity 0.2s ease;
       }
 
-      /* Reveal Icons */
       :host([checked]) svg.icon-check,
       :host([indeterminate]) svg.icon-minus {
         opacity: 1;
         transform: scale(1);
       }
 
-      /* Disabled State */
       :host([disabled]) {
         pointer-events: none;
       }
@@ -147,7 +136,6 @@ constructor() {
         cursor: not-allowed;
       }
 
-      /* Invalid State */
       :host([invalid]) .control {
         border-color: var(--st-color-error);
       }
@@ -162,8 +150,7 @@ constructor() {
   override updated(changedProperties: PropertyValues<this>) {
     super.updated(changedProperties);
 
-    if (changedProperties.has("checked") || changedProperties.has("value")) {
-      // Form API: If checked, submit the value. If unchecked, submit null to remove it from FormData.
+    if (changedProperties.has("checked") || changedProperties.has("value") || changedProperties.has("errorMessage")) {
       this._internals.setFormValue(this.checked ? this.value : null);
       this._syncValidity();
     }
@@ -175,16 +162,54 @@ constructor() {
 
   private _syncValidity() {
     if (!this._inputEl) return;
-    this._internals.setValidity(
-      this._inputEl.validity,
-      this._inputEl.validationMessage,
-      this._inputEl,
-    );
+
+    if (this.required && !this.checked) {
+      const message = this.errorMessage || this._inputEl.validationMessage || "Please check this box if you want to proceed.";
+      this._internals.setValidity(
+        {valueMissing: true},
+        message,
+        this._inputEl,
+      );
+    } else {
+      this._internals.setValidity(
+        this._inputEl.validity,
+        this._inputEl.validationMessage,
+        this._inputEl,
+      );
+    }
+  }
+
+  public checkValidity(): boolean {
+    if (!this._inputEl) return true;
+
+    if (this.required && !this.checked) {
+      const message = this.errorMessage || this._inputEl.validationMessage || "Please check this box if you want to proceed.";
+      this._internals.setValidity(
+        {valueMissing: true},
+        message,
+        this._inputEl,
+      );
+      this.invalid = true;
+      return false;
+    }
+
+    const isValid = this._inputEl.checkValidity();
+    this._syncValidity();
+    this.invalid = !isValid;
+    return isValid;
+  }
+
+  public reportValidity(): boolean {
+    this._syncValidity();
+    const isValid = this._internals.reportValidity();
+    this.invalid = !isValid;
+    return isValid;
   }
 
   formResetCallback() {
     this.checked = false;
     this.indeterminate = false;
+    this.invalid = false;
     this._internals.setFormValue(null);
     this.updateComplete.then(() => this._syncValidity());
   }
@@ -194,12 +219,16 @@ constructor() {
 
     if (this.disabled) return;
 
-    // Standard native behavior: clicking an indeterminate checkbox unsets the indeterminate state
     if (this.indeterminate) {
       this.indeterminate = false;
     }
 
     this.checked = this._inputEl.checked;
+    
+    if (this.checkValidity()) {
+      this.invalid = false;
+    }
+
     this.dispatchEvent(new Event("change", {bubbles: true, composed: true}));
   }
 
@@ -218,7 +247,6 @@ constructor() {
           @change=${this._handleChange}
         />
         <div class="control">
-          <!-- Render both SVGs and use CSS to conditionally reveal them based on host attributes -->
           ${this.indeterminate
             ? html`<svg class="icon-minus" viewBox="0 0 24 24"><path d="M5 12h14"></path></svg>`
             : html`<svg class="icon-check" viewBox="0 0 24 24">
@@ -235,4 +263,4 @@ declare global {
   interface HTMLElementTagNameMap {
     "st-checkbox": StCheckbox;
   }
-}
+}

package/src/components/code-block/code-block.ts

@@ -254,4 +254,10 @@ export class StCodeBlock extends LitElement {
       </div>
     `;
   }
+}
+
+declare global {
+  interface HTMLElementTagNameMap {
+    "st-code-block": StCodeBlock;
+  }
 }

package/src/components/form/form-step.ts

@@ -6,6 +6,10 @@ import {resetStyles} from "../../styles/reset.js";
 export class StFormStep extends LitElement {
   @property({type: String}) title = "";
   @property({type: Boolean, reflect: true}) active = false;
+  
+  // Declare the validate function property (attribute: false prevents reflection of functions to DOM attributes)
+  @property({attribute: false})
+  validate?: () => Promise<boolean> | boolean;
 
   static styles = [
     resetStyles,
@@ -29,4 +33,4 @@ declare global {
   interface HTMLElementTagNameMap {
     "st-form-step": StFormStep;
   }
-}
+}

package/src/components/form/form.ts

@@ -1,5 +1,5 @@
-import {LitElement, html, css} from "lit";
-import {customElement, property, state, queryAssignedElements, query} from "lit/decorators.js";
+import {LitElement, html, css, PropertyValues} from "lit";
+import {customElement, property, state, query} from "lit/decorators.js";
 import {resetStyles} from "../../styles/reset.js";
 import type {StFormStep} from "./form-step.js";
 
@@ -13,11 +13,14 @@ export class StForm extends LitElement {
 
   @state() private _currentStepIndex = 0;
 
-  @queryAssignedElements({selector: "st-form-step"})
-  private _stepElements!: StFormStep[];
-
   @query("form") private _formEl!: HTMLFormElement;
 
+  private _modalFooterEl: HTMLElement | null = null;
+
+  private get _steps(): StFormStep[] {
+    return Array.from(this.querySelectorAll("st-form-step"));
+  }
+
   static styles = [
     resetStyles,
     css`
@@ -31,9 +34,6 @@ export class StForm extends LitElement {
         gap: var(--st-spacing-md);
       }
 
-      /* ==========================================================
-         Layouts
-         ========================================================== */
       :host([layout="vertical"]) form {
         flex-direction: column;
       }
@@ -55,9 +55,6 @@ export class StForm extends LitElement {
         flex-direction: column;
       }
 
-      /* ==========================================================
-         Progress Navigation
-         ========================================================== */
       .progress-nav {
         display: flex;
         align-items: center;
@@ -121,9 +118,6 @@ export class StForm extends LitElement {
         background-color: var(--st-color-success);
       }
 
-      /* ==========================================================
-         Actions & States
-         ========================================================== */
       .form-actions {
         display: flex;
         gap: var(--st-spacing-md);
@@ -162,42 +156,165 @@ export class StForm extends LitElement {
     `,
   ];
 
-  private _handleSlotChange() {
-    if (this._stepElements.length > 0) {
+  override connectedCallback() {
+    super.connectedCallback();
+    this.updateComplete.then(() => {
       this._updateStepVisibility();
+      this._updateModalFooter();
+    });
+  }
+
+  override disconnectedCallback() {
+    super.disconnectedCallback();
+    if (this._modalFooterEl) {
+      this._modalFooterEl.remove();
+      this._modalFooterEl = null;
+    }
+  }
+
+  override updated(changedProperties: PropertyValues<this>) {
+    super.updated(changedProperties);
+    this._updateModalFooter();
+  }
+
+  private _updateModalFooter() {
+    const modal = this.closest("st-modal");
+    if (modal) {
+      if (!this._modalFooterEl) {
+        this._modalFooterEl = document.createElement("div");
+        this._modalFooterEl.setAttribute("slot", "footer");
+        this._modalFooterEl.style.display = "flex";
+        this._modalFooterEl.style.gap = "var(--st-spacing-md)";
+        this._modalFooterEl.style.width = "100%";
+        this._modalFooterEl.style.justifyContent = "flex-end";
+        this._modalFooterEl.style.alignItems = "center";
+        modal.appendChild(this._modalFooterEl);
+      }
+
+      const steps = this._steps;
+      const hasSteps = steps.length > 0;
+      const isLastStep = hasSteps ? this._currentStepIndex === steps.length - 1 : true;
+
+      // Imperative DOM management prevents event listener loss and re-rendering conflicts
+      this._modalFooterEl.innerHTML = "";
+
+      if (hasSteps && this._currentStepIndex > 0) {
+        const backBtn = document.createElement("st-button");
+        backBtn.setAttribute("variant", "secondary");
+        backBtn.setAttribute("type", "button");
+        if (this.loading) backBtn.setAttribute("disabled", "");
+        backBtn.textContent = "Back";
+        backBtn.addEventListener("click", (e) => {
+          e.preventDefault();
+          this._prevStep();
+        });
+        this._modalFooterEl.appendChild(backBtn);
+      }
+
+      const submitBtn = document.createElement("st-button");
+      submitBtn.setAttribute("color", "primary");
+      submitBtn.setAttribute("type", "button");
+      if (this.loading) submitBtn.setAttribute("disabled", "");
+      submitBtn.textContent = this.loading ? "Submitting..." : (isLastStep ? "Submit" : "Continue");
+      submitBtn.addEventListener("click", (e) => {
+        e.preventDefault();
+        this._triggerSubmission();
+      });
+      this._modalFooterEl.appendChild(submitBtn);
+
+    } else if (this._modalFooterEl) {
+      this._modalFooterEl.remove();
+      this._modalFooterEl = null;
     }
   }
 
+  private _handleSlotChange() {
+    this._updateStepVisibility();
+    this._updateModalFooter();
+  }
+
   private _updateStepVisibility() {
-    this._stepElements.forEach((step, index) => {
+    const steps = this._steps;
+    steps.forEach((step, index) => {
       step.active = index === this._currentStepIndex;
     });
     this.requestUpdate();
   }
 
-  private _validateCurrentContext(): boolean {
-    if (this._stepElements.length === 0) {
-      return this._formEl.checkValidity();
-    }
+  private async _validateCurrentContext(): Promise<boolean> {
+    const steps = this._steps;
+    const scope: ParentNode = steps.length > 0
+      ? steps[this._currentStepIndex]
+      : this;
 
-    const currentStep = this._stepElements[this._currentStepIndex];
     let isValid = true;
 
-    // Find all custom and native form inputs assigned to the current step
     const inputs = Array.from(
-      currentStep.querySelectorAll<
-        HTMLElement & {checkValidity?: () => boolean; reportValidity?: () => void}
-      >("*"),
+      scope.querySelectorAll<
+        HTMLElement & {
+          checkValidity?: () => boolean;
+          reportValidity?: () => boolean;
+          required?: boolean;
+          value?: any;
+          checked?: boolean;
+          invalid?: boolean;
+        }
+      >("st-input, st-select, st-textarea, st-checkbox, st-radio, input, select, textarea"),
     );
-
+    
     for (const input of inputs) {
+      if (input.hasAttribute("disabled") || (input as any).disabled) continue;
+
+      let fieldValid = true;
+
       if (typeof input.checkValidity === "function") {
-        if (!input.checkValidity()) {
-          // Focus and report on the first invalid element encountered
-          if (isValid && typeof input.reportValidity === "function") {
-            input.reportValidity();
+        fieldValid = input.checkValidity();
+      } else {
+        const isRequired = input.hasAttribute("required") || input.required;
+        if (isRequired) {
+          const isToggle = input.tagName === "ST-CHECKBOX" || input.tagName === "ST-RADIO";
+          const val = isToggle ? input.checked : input.value;
+          if (val === undefined || val === null || val === "" || val === false) {
+            fieldValid = false;
           }
+        }
+      }
+
+      if (!fieldValid) {
+        if (typeof input.reportValidity === "function") {
+          input.reportValidity();
+        } else if ("invalid" in input) {
+          input.invalid = true;
+        }
+        if (isValid && typeof (input as any).focus === "function") {
+          (input as any).focus();
+        }
+        isValid = false;
+      }
+    }
+
+    if (steps.length === 0 && this._formEl) {
+      if (!this._formEl.checkValidity()) {
+        isValid = false;
+      }
+    }
+
+    if (isValid && steps.length > 0) {
+      const currentStep = steps[this._currentStepIndex] as StFormStep & {
+        validate?: () => Promise<boolean> | boolean;
+      };
+
+      if (typeof currentStep.validate === "function") {
+        try {
+          this.loading = true;
+          const stepPassed = await currentStep.validate();
+          if (!stepPassed) {
+            isValid = false;
+          }
+        } catch {
           isValid = false;
+        } finally {
+          this.loading = false;
         }
       }
     }
@@ -206,7 +323,6 @@ export class StForm extends LitElement {
   }
 
   private _triggerSubmission() {
-    // HTML5 method that triggers form validation and fires the native submit event
     this._formEl.requestSubmit();
   }
 
@@ -220,13 +336,11 @@ export class StForm extends LitElement {
   private _extractFormData(): FormData {
     const formData = new FormData();
 
-    // Query all light-DOM children that have a 'name' attribute
     const fields = this.querySelectorAll<
       HTMLElement & {name: string; value: any; checked?: boolean; files?: FileList}
     >("[name]");
 
     fields.forEach((field) => {
-      // Standard HTML form behavior: ignore disabled fields
       if (field.hasAttribute("disabled")) return;
 
       const isToggle =
@@ -236,7 +350,6 @@ export class StForm extends LitElement {
         field.getAttribute("type") === "checkbox";
 
       if (isToggle) {
-        // Only append radios/checkboxes if they are actively checked
         if (field.checked) {
           formData.append(field.name, field.value || "on");
         }
@@ -245,10 +358,8 @@ export class StForm extends LitElement {
         field.getAttribute("type") === "file" &&
         field.files
       ) {
-        // Handle native file uploads
         Array.from(field.files).forEach((file) => formData.append(field.name, file));
       } else if (field.value !== undefined) {
-        // Handle st-input, st-select, st-textarea, and standard inputs
         formData.append(field.name, field.value);
       }
     });
@@ -256,14 +367,16 @@ export class StForm extends LitElement {
     return formData;
   }
 
-  private _handleSubmit(e: Event) {
+  private async _handleSubmit(e: Event) {
     e.preventDefault();
     e.stopPropagation();
 
-    if (!this._validateCurrentContext()) return;
+    const isValid = await this._validateCurrentContext();
+    if (!isValid) return;
 
-    const hasSteps = this._stepElements.length > 0;
-    const isLastStep = hasSteps ? this._currentStepIndex === this._stepElements.length - 1 : true;
+    const steps = this._steps;
+    const hasSteps = steps.length > 0;
+    const isLastStep = hasSteps ? this._currentStepIndex === steps.length - 1 : true;
 
     if (hasSteps && !isLastStep) {
       this._currentStepIndex++;
@@ -271,7 +384,6 @@ export class StForm extends LitElement {
       return;
     }
 
-    // Use our custom extractor to pierce the Shadow DOM boundary
     const formData = this._extractFormData();
 
     this.dispatchEvent(
@@ -285,8 +397,9 @@ export class StForm extends LitElement {
   }
 
   render() {
-    const hasSteps = this._stepElements?.length > 0;
-    const isLastStep = hasSteps ? this._currentStepIndex === this._stepElements.length - 1 : true;
+    const steps = this._steps;
+    const hasSteps = steps.length > 0;
+    const isInModal = !!this.closest("st-modal");
 
     return html`
       <div class="${hasSteps ? "has-steps" : ""}" aria-busy=${this.loading}>
@@ -300,7 +413,7 @@ export class StForm extends LitElement {
         ${hasSteps
           ? html`
               <div class="progress-nav" role="tablist">
-                ${this._stepElements.map((step, index) => {
+                ${steps.map((step, index) => {
                   const isActive = index === this._currentStepIndex;
                   const isCompleted = index < this._currentStepIndex;
 
@@ -319,7 +432,7 @@ export class StForm extends LitElement {
                       </div>
                       <span>${step.title || `Step ${index + 1}`}</span>
                     </div>
-                    ${index < this._stepElements.length - 1
+                    ${index < steps.length - 1
                       ? html` <div class="step-connector ${isCompleted ? "completed" : ""}"></div> `
                       : ""}
                   `;
@@ -339,30 +452,33 @@ export class StForm extends LitElement {
         <form @submit=${this._handleSubmit}>
           <slot @slotchange=${this._handleSlotChange}></slot>
 
-          <div class="form-actions">
-            ${hasSteps && this._currentStepIndex > 0
-              ? html`
+          ${!isInModal
+            ? html`
+                <div class="form-actions">
+                  ${hasSteps && this._currentStepIndex > 0
+                    ? html`
+                        <st-button
+                          variant="secondary"
+                          type="button"
+                          @click=${this._prevStep}
+                          ?disabled=${this.loading}
+                        >
+                          Back
+                        </st-button>
+                      `
+                    : ""}
+
                   <st-button
-                    variant="secondary"
+                    color="primary"
                     type="button"
-                    @click=${this._prevStep}
+                    @click=${this._triggerSubmission}
                     ?disabled=${this.loading}
                   >
-                    Back
+                    ${this.loading ? "Submitting..." : hasSteps && this._currentStepIndex === steps.length - 1 ? "Submit" : hasSteps ? "Continue" : "Submit"}
                   </st-button>
-                `
-              : ""}
-
-            <!-- Type is "button", but click triggers requestSubmit() on the internal form -->
-            <st-button
-              color="primary"
-              type="button"
-              @click=${this._triggerSubmission}
-              ?disabled=${this.loading}
-            >
-              ${this.loading ? "Submitting..." : isLastStep ? "Submit" : "Continue"}
-            </st-button>
-          </div>
+                </div>
+              `
+            : ""}
         </form>
       </div>
     `;
@@ -373,4 +489,4 @@ declare global {
   interface HTMLElementTagNameMap {
     "st-form": StForm;
   }
-}
+}

package/src/components/icon/icon.ts

@@ -3,10 +3,12 @@ import { customElement, property } from 'lit/decorators.js';
 import { iconPaths, IconName } from './icon-paths.js';
 import { resetStyles } from '../../styles/reset.js';
 
+export type IconVariant = 'outline' | 'solid';
+
 @customElement('st-icon')
 export class StIcon extends LitElement {
   @property({ type: String }) name: IconName | string = '';
-  @property({ type: String }) variant: 'outline' | 'solid' = 'outline';
+  @property({ type: String }) variant: IconVariant = 'outline';
   
   // Semantic Colors
   @property({ type: String, reflect: true }) color: 'default' | 'primary' | 'secondary' | 'success' | 'warning' | 'error' | 'info' = 'default';

package/src/components/input/input.ts

@@ -246,3 +246,9 @@ export class StInput extends LitElement {
     `;
   }
 }
+
+declare global {
+  interface HTMLElementTagNameMap {
+    "st-input": StInput;
+  }
+}

package/src/components/page-layout/page-layout.ts

@@ -153,7 +153,7 @@ export class StPageLayout extends LitElement {
         --layout-surface-color: var(--st-color-surface, #f8fafc);
         --layout-border-color: var(--st-color-border, #e2e8f0);
         --layout-sidebar-width: 16rem;
-        --layout-right-sidebar-width: 14rem;
+        --layout-right-sidebar-width: 16rem;
         --layout-main-max-width: none;
         --layout-nav-height: 4rem;
       }