@brickclay-org/ui 0.1.89 → 0.1.91

package/index.d.ts

@@ -1579,7 +1579,7 @@ declare class BkSelect implements ControlValueAccessor, AfterViewInit, OnDestroy
      * existing click/Enter-to-open behaviour is unchanged.
      */
     openOnFocus: _angular_core.InputSignal<boolean>;
-    dropdownPosition: _angular_core.InputSignal<"top" | "bottom">;
+    dropdownPosition: _angular_core.InputSignal<"bottom" | "top">;
     hasError: boolean;
     errorMessage: string;
     /**
@@ -1618,7 +1618,7 @@ declare class BkSelect implements ControlValueAccessor, AfterViewInit, OnDestroy
     private gridDraftOptions;
     searchTerm: _angular_core.WritableSignal<string>;
     markedIndex: _angular_core.WritableSignal<number>;
-    placement: _angular_core.WritableSignal<"top" | "bottom">;
+    placement: _angular_core.WritableSignal<"bottom" | "top">;
     visibleCount: _angular_core.WritableSignal<number>;
     private resizeObserver?;
     /**
@@ -1868,6 +1868,19 @@ declare class BkInput implements OnInit, OnDestroy, AfterViewInit, ControlValueA
     phoneOverlay?: CdkConnectedOverlay;
     isFocused: boolean;
     inputValue: string;
+    /**
+     * What the template's `[value]` binding actually renders. Deliberately a separate property
+     * from `inputValue` (which still mirrors the live text for `isFilled`/bookkeeping on every
+     * keystroke): re-applying `[value]` to an element that already displays that exact text is a
+     * no-op on most inputs, but on `type="number"` it isn't - Chromium doesn't skip the write just
+     * because the string is unchanged, and since `type="number"` also refuses `setSelectionRange`
+     * (it throws), we have no way to put the caret back afterwards. So `displayValue` is only ever
+     * written by genuinely external changes (writeValue(), the phone country-prefix swap) - never
+     * from handleInput() or the "parent mutated the native value" resync, both of which are only
+     * echoing back text the DOM already has. That keeps every self-originated edit from re-touching
+     * `.value` at all, which is the only reliable fix here given number inputs can't be caret-restored.
+     */
+    displayValue: string;
     isDropdownOpen: boolean;
     private pendingMaskedValue;
     private isPropagatingViewValue;
@@ -2921,7 +2934,7 @@ declare class BkHierarchicalSelect implements ControlValueAccessor, AfterViewIni
      */
     appendToBody: _angular_core.InputSignal<boolean>;
     /** Open above or below the trigger (also used when appendToBody is true for fixed coordinates). */
-    dropdownPosition: _angular_core.InputSignal<"top" | "bottom">;
+    dropdownPosition: _angular_core.InputSignal<"bottom" | "top">;
     /** Key for option color (e.g. "color"). When set, option label and selected value use this color. */
     colorKey: _angular_core.InputSignal<string>;
     /**
@@ -2955,7 +2968,7 @@ declare class BkHierarchicalSelect implements ControlValueAccessor, AfterViewIni
     /** Index of the keyboard-highlighted option within the current level (`filteredItems`). */
     markedIndex: _angular_core.WritableSignal<number>;
     private el;
-    placement: _angular_core.WritableSignal<"top" | "bottom">;
+    placement: _angular_core.WritableSignal<"bottom" | "top">;
     /**
      * Panel width, kept equal to the control's — matches the old `dropdownStyle().width`
      * behaviour. Set on open and re-measured by a ResizeObserver on the control while open, so a
@@ -3538,7 +3551,6 @@ interface MenuItem {
  *   flip — they stay under their trigger and scroll instead.
  */
 declare class BkMenu implements OnChanges, OnDestroy {
-    private readonly host;
     items: MenuItem[];
     orientation: MenuOrientation;
     submenuMode: MenuSubmenuMode;
@@ -3609,7 +3621,6 @@ declare class BkMenu implements OnChanges, OnDestroy {
      */
     private static popupHeightCeiling;
     private closeTimer;
-    constructor(host: ElementRef<HTMLElement>);
     /**
      * Inline mode: whenever the active leaf is (re)set — including the very
      * first binding, so a deep link or a page refresh lands with the section
@@ -3658,7 +3669,7 @@ declare class BkMenu implements OnChanges, OnDestroy {
     private toggle;
     private closeBranch;
     private closeAll;
-    onDocumentClick(event: MouseEvent): void;
+    onDocumentClick(): void;
     onViewportChange(): void;
     static ɵfac: _angular_core.ɵɵFactoryDeclaration<BkMenu, never>;
     static ɵcmp: _angular_core.ɵɵComponentDeclaration<BkMenu, "bk-menu", never, { "items": { "alias": "items"; "required": false; }; "orientation": { "alias": "orientation"; "required": false; }; "submenuMode": { "alias": "submenuMode"; "required": false; }; "trigger": { "alias": "trigger"; "required": false; }; "size": { "alias": "size"; "required": false; }; "hoverCloseDelay": { "alias": "hoverCloseDelay"; "required": false; }; "singleOpen": { "alias": "singleOpen"; "required": false; }; "submenuBackground": { "alias": "submenuBackground"; "required": false; }; "emitParentClick": { "alias": "emitParentClick"; "required": false; }; "activeItemId": { "alias": "activeItemId"; "required": false; }; "tintIcons": { "alias": "tintIcons"; "required": false; }; "maxHeight": { "alias": "maxHeight"; "required": false; }; }, { "activeItemIdChange": "activeItemIdChange"; "itemClick": "itemClick"; }, never, never, true, never>;
@@ -4568,7 +4579,7 @@ declare class BkTd extends BkCellBase implements AfterViewInit, OnDestroy {
      */
     protected readonly truncatedTooltip: _angular_core.WritableSignal<string>;
     /** Forwarded to the content wrapper's `bkTooltip`. Override on a column whose default 'right' would run into its neighbour. */
-    tooltipPosition: _angular_core.InputSignal<"left" | "right" | "top" | "bottom">;
+    tooltipPosition: _angular_core.InputSignal<"left" | "right" | "bottom" | "top">;
     /** Only present when `ellipsis()` is true — see the `@else` branch in the template. */
     private ellipsisTextRef?;
     checkbox: _angular_core.InputSignal<boolean>;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@brickclay-org/ui",
-  "version": "0.1.89",
+  "version": "0.1.91",
   "schematics": "./schematics/collection.json",
   "ng-add": {
     "save": "dependencies"

package/src/lib/calender/components/custom-calendar/custom-calendar.component.css

@@ -598,129 +598,6 @@
   justify-content: center;
 } */
 
-/* --------------------------
-   CUSTOM TIME PICKER
----------------------------*/
-.custom-time-picker {
-  display: flex;
-  flex-direction: column;
-  gap: 8px;
-  align-items: start;
-}
-
-.time-input-group {
-  display: flex;
-  align-items: center;
-  justify-content: center;
-  /*width: 100%;*/
-  gap: 8px;
-  background: #f8f9fa;
-  padding: 12px;
-  border-radius: 8px;
-  border: 1px solid #e0e0e0;
-}
-
-.time-control {
-  display: flex;
-  flex-direction: column;
-  align-items: center;
-  /* gap: 4px; */
-}
-
-.time-btn {
-  background: #fff;
-  border: 1px solid #ddd;
-  width: 28px;
-  height: 20px;
-  cursor: pointer;
-  font-size: 10px;
-  color: #666;
-  border-radius: 4px;
-  transition: all 0.2s;
-  display: flex;
-  align-items: center;
-  justify-content: center;
-  padding: 0;
-  line-height: 1;
-}
-
-.time-btn:hover {
-  background: #e4e4e4;
-  color: white;
-  border-color: #e4e4e4;
-}
-
-.time-btn.up {
-  border-bottom-left-radius: 0;
-  border-bottom-right-radius: 0;
-  border-bottom: none;
-}
-
-.time-btn.down {
-  border-top-left-radius: 0;
-  border-top-right-radius: 0;
-  border-top: none;
-}
-
-.time-input {
-  width: 40px;
-  height: 32px;
-  text-align: center;
-  border: 1px solid #ddd;
-  border-radius: 4px;
-  font-size: 16px;
-  font-weight: 600;
-  background: #fff;
-  color: #333;
-}
-
-.time-separator {
-  font-size: 18px;
-  font-weight: 600;
-  color: #666;
-  margin: 0 2px;
-}
-
-.ampm-control {
-  display: flex;
-  flex-direction: column;
-  gap: 4px;
-  margin-left: 8px;
-}
-
-.ampm-btn {
-  padding: 6px 12px;
-  border: 1px solid #ddd;
-  background: #fff;
-  border-radius: 4px;
-  cursor: pointer;
-  font-size: 12px;
-  font-weight: 600;
-  color: #666;
-  transition: all 0.2s;
-  min-width: 45px;
-}
-
-.ampm-btn:hover {
-  background: #f0f0f0;
-}
-
-.ampm-btn.active {
-  background: #000;
-  color: white;
-  border-color: #000;
-}
-
-.html5-time-input {
-  margin-top: 8px;
-  padding: 8px;
-  border: 1px solid #ddd;
-  border-radius: 6px;
-  font-size: 14px;
-  width: 100%;
-  max-width: 120px;
-}
-
 /* --------------------------
    FOOTER (Apply/Cancel Buttons)
 ---------------------------*/
@@ -830,10 +707,6 @@
     margin-bottom: 16px;
   }
 
-  .time-input-group {
-    flex-wrap: wrap;
-    justify-content: center;
-  }
 }
 
 /* --------------------------

package/src/lib/menu/menu.css

@@ -131,10 +131,28 @@
   @apply rounded-r-[var(--menu-radius)];
 }
 
+/* Inline only: once the last vertical item's submenu opens, it sits flush
+   below the button — the rounded-bottom corner above no longer matches the
+   container's true edge, so it'd leave an odd notch against the flat panel
+   underneath. Square it off while open. Pop-up fly-outs go sideways, not
+   below, so they never hit this — hence the :not(.bk-menu--popup) guard. */
+.bk-menu--vertical:not(.bk-menu--popup) > .bk-menu__list > .bk-menu__item--open:last-child > .bk-menu__button {
+  border-bottom-left-radius: 0;
+  border-bottom-right-radius: 0;
+}
+
 .bk-menu__button:hover:not(:disabled):not(.bk-menu__button--active) {
   @apply bg-[var(--menu-hover-bg)] text-[color:var(--menu-hover-text)];
 }
 
+/* Open state (submenu expanded — inline or flown-out): same tint as hover,
+   so a parent stays visibly "open" once the pointer moves off it instead of
+   reading as a plain unhovered row. Skipped when it's also the active
+   button — the dark active background already marks that more strongly. */
+.bk-menu__item--open > .bk-menu__button:not(.bk-menu__button--active) {
+  @apply bg-[var(--menu-hover-bg)] text-[color:var(--menu-hover-text)];
+}
+
 .bk-menu__button--active {
   @apply bg-[var(--menu-active-bg)] text-[color:var(--menu-active-text)];
 }

package/src/lib/toastr/toastr.css

@@ -14,8 +14,10 @@
   pointer-events: none;
 }
 
-/* Guarantee toasts stack above dialogs/popovers/etc. that also use CDK overlays — CDK's own
-   z-index otherwise just reflects creation order. Targets the pane CDK wraps this panel in. */
+/* Belt-and-braces: pins the pane itself in case some future overlay ever nests a pane inside
+   this one's wrapper. The stacking fight against *other* overlays (dialogs, selects, popovers)
+   actually happens one level up, at the wrapper CDK puts around the pane — see the
+   `hostElement.style.zIndex` line in toastr.service.ts, which is the real fix for that. */
 .cdk-overlay-pane.bk-toastr-panel {
   z-index: 2147483647 !important;
 }