@inglorious/ui 7.2.1 → 7.2.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@inglorious/ui",
-  "version": "7.2.1",
+  "version": "7.2.3",
   "description": "A design system for Inglorious Web with render-function components and customizable themes",
   "author": "Matteo Antony Mistretta <antony.mistretta@gmail.com> (https://ingloriouscoderz.it)",
   "license": "MIT",
@@ -305,10 +305,10 @@
     "**/*.css"
   ],
   "peerDependencies": {
-    "@inglorious/charts": "10.0.3",
+    "@inglorious/charts": "10.0.4",
     "@inglorious/store": "11.0.2",
     "@inglorious/utils": "4.0.0",
-    "@inglorious/web": "6.5.0"
+    "@inglorious/web": "6.5.1"
   },
   "devDependencies": {
     "@fontsource/material-symbols-outlined": "^5.3.5",
@@ -317,7 +317,7 @@
     "@inglorious/store": "11.0.2",
     "@inglorious/vite-plugin-jsx": "2.0.2",
     "@inglorious/vite-plugin-vue": "1.0.3",
-    "@inglorious/web": "6.5.0",
+    "@inglorious/web": "6.5.1",
     "@storybook/addon-docs": "^10.6.0",
     "@storybook/web-components-vite": "^10.6.0",
     "happy-dom": "^20.14.0",

package/src/controls/combobox/combobox.stories.js

@@ -56,8 +56,8 @@ export default {
   },
 }
 
-export const SingleClosed = {}
-SingleClosed.args = {
+export const Default = {}
+Default.args = {
   id: "combobox",
   type: "Combobox",
   label: "Country",
@@ -79,14 +79,76 @@ SingleClosed.args = {
 
 export const SingleOpen = {}
 SingleOpen.args = {
-  ...SingleClosed.args,
+  ...Default.args,
   isOpen: true,
 }
 
 export const MultiOpen = {}
 MultiOpen.args = {
-  ...SingleClosed.args,
+  ...Default.args,
   isMulti: true,
   selectedValue: ["it", "fr"],
   isOpen: true,
 }
+
+export const WithHint = {}
+WithHint.args = {
+  ...Default.args,
+  hint: "Please select a country",
+}
+
+export const WithError = {}
+WithError.args = {
+  ...Default.args,
+  error: "You cannot select this one!",
+}
+
+export const WithHintAndError = {}
+WithHintAndError.args = {
+  ...WithHint.args,
+  error: "You cannot select this one!",
+}
+
+export const Required = {}
+Required.args = {
+  ...Default.args,
+  isRequired: true,
+}
+
+export const Disabled = {}
+Disabled.args = {
+  ...Default.args,
+  isDisabled: true,
+}
+
+export const DisabledOption = {}
+DisabledOption.args = {
+  ...Default.args,
+  options: Default.args.options.map((option, index) =>
+    index % 2 ? { ...option, isDisabled: true } : option,
+  ),
+}
+
+export const Small = {}
+Small.args = {
+  ...Default.args,
+  size: "sm",
+}
+
+export const Medium = {}
+Medium.args = {
+  ...Default.args,
+  size: "md",
+}
+
+export const Large = {}
+Large.args = {
+  ...Default.args,
+  size: "lg",
+}
+
+export const FullWidth = {}
+FullWidth.args = {
+  ...Default.args,
+  isFullWidth: true,
+}

package/src/controls/combobox/style.css

@@ -1,11 +1,11 @@
 @import "../../data-display/chip/style.css";
 
 .iw-combobox {
-  position: relative;
   display: inline-flex;
   flex-direction: column;
   gap: var(--iw-space-1);
   font-family: var(--iw-font-family-sans);
+  position: relative; /* required for dropdown positioning */
 
   &.iw-combobox-full-width {
     width: 100%;
@@ -13,20 +13,40 @@
 
   .iw-combobox-label {
     font-size: var(--iw-font-size-sm);
+    font-weight: var(--iw-font-weight-normal);
     color: var(--iw-color-text);
     line-height: var(--iw-line-height-normal);
   }
 
-  .iw-combobox-control {
-    height: 2.25rem;
+  .iw-combobox-required {
+    color: var(--iw-color-error);
+    margin-left: var(--iw-space-1);
+  }
+
+  .iw-combobox-hint,
+  .iw-combobox-error-message {
+    font-size: var(--iw-font-size-sm);
+    line-height: var(--iw-line-height-normal);
+  }
+
+  .iw-combobox-hint {
+    color: var(--iw-color-text-muted);
+  }
+
+  .iw-combobox-error-message {
+    color: var(--iw-color-error);
+  }
+
+  .iw-combobox-frame {
     display: flex;
     align-items: center;
+    /* The gap spaces the text from the arrow, as it does for Input's icons. */
     gap: var(--iw-space-2);
-    padding: 0 var(--iw-space-10) 0 var(--iw-space-3);
+    padding: 0 var(--iw-space-3);
     border: 1px solid var(--iw-input-border);
     border-radius: var(--iw-input-radius);
-    background: var(--iw-input-bg);
     color: var(--iw-input-text);
+    background-color: var(--iw-input-bg);
     cursor: pointer;
     user-select: none;
     transition:
@@ -34,11 +54,32 @@
       box-shadow var(--iw-transition-fast),
       background-color var(--iw-transition-fast);
 
-    &:hover {
+    &.iw-combobox-sm {
+      height: 2rem;
+      font-size: var(--iw-font-size-sm);
+    }
+
+    &.iw-combobox-md {
+      height: 2.25rem;
+      font-size: var(--iw-font-size-base);
+    }
+
+    &.iw-combobox-lg {
+      height: 2.75rem;
+      font-size: var(--iw-font-size-lg);
+    }
+
+    &:hover:not(.iw-combobox-frame-disabled) {
       border-color: var(--iw-input-border-hover);
       background: var(--iw-input-bg-hover);
     }
 
+    &:focus-within:not(.iw-combobox-frame-disabled) {
+      background-color: var(--iw-input-bg-focus);
+      border-color: var(--iw-input-border-focus);
+      box-shadow: var(--iw-input-shadow-focus);
+    }
+
     /* The control is focusable so it can be opened from the keyboard. */
     &:focus-visible {
       outline: none;
@@ -46,7 +87,7 @@
       box-shadow: var(--iw-input-shadow-focus);
     }
 
-    &.iw-combobox-control-open {
+    &.iw-combobox-frame-open {
       border-color: var(--iw-input-border-focus);
       box-shadow: var(--iw-input-shadow-focus);
       background: var(--iw-input-bg-focus);
@@ -54,14 +95,23 @@
       border-bottom-right-radius: 0;
     }
 
-    &.iw-combobox-control-disabled {
+    &.iw-combobox-frame-disabled {
       opacity: 0.6;
       cursor: not-allowed;
     }
 
-    .iw-combobox-value,
-    .iw-combobox-placeholder,
-    .iw-combobox-multi-value {
+    &.iw-combobox-frame-error {
+      border-color: var(--iw-color-error);
+
+      &:focus-visible {
+        border-color: var(--iw-color-error);
+      }
+    }
+
+    /* The text element, whatever it is showing. Same role as
+       .iw-input-control in Input, so the shared rules live here and the
+       variant classes only add what is specific to them. */
+    .iw-combobox-control {
       flex: 1;
       min-width: 0;
     }
@@ -114,19 +164,20 @@
       display: inline-flex;
       align-items: center;
       justify-content: center;
+      font-size: var(--iw-font-size-sm);
+      line-height: 1;
+      /* A flex child, like Input's icons and the Select arrow. Absolutely
+         positioning it anchored to .iw-combobox rather than the frame, so
+         the glyph drifted whenever a label sat above the control, and it
+         needed a reserved padding strip to stop text running under it. The
+         row's gap does both. */
+      flex: none;
       pointer-events: none;
-      position: absolute;
-      right: var(--iw-space-3);
       color: var(--iw-color-text-muted);
       transition: transform var(--iw-transition-fast);
-
-      span {
-        font-size: 0.65rem;
-        line-height: 1;
-      }
     }
 
-    &.iw-combobox-control-open .iw-combobox-arrow {
+    &.iw-combobox-frame-open .iw-combobox-arrow {
       transform: rotate(180deg);
     }
   }
@@ -198,13 +249,13 @@
     }
   }
 
-  &.iw-combobox-sm .iw-combobox-control {
+  &.iw-combobox-sm .iw-combobox-frame {
     height: 2rem;
-    padding: 0 var(--iw-space-8) 0 var(--iw-space-2);
+    padding: 0 var(--iw-space-2);
   }
 
-  &.iw-combobox-lg .iw-combobox-control {
+  &.iw-combobox-lg .iw-combobox-frame {
     height: 2.75rem;
-    padding: 0 var(--iw-space-12) 0 var(--iw-space-4);
+    padding: 0 var(--iw-space-4);
   }
 }