@brickclay-org/ui 0.1.89 → 0.1.90

package/index.d.ts

@@ -3538,7 +3538,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 +3608,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 +3656,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>;
@@ -3721,7 +3719,7 @@ declare class BkPopover implements OnDestroy {
      * anything interactive. `hover` opens on the way in and closes on the way
      * out, with a grace period so the pointer can travel into the panel.
      */
-    trigger: _angular_core.InputSignal<"click" | "hover">;
+    trigger: _angular_core.InputSignal<"hover" | "click">;
     /** Optional heading. Empty hides the header row entirely. */
     title: _angular_core.InputSignal<string>;
     /** Show the × in the header. Ignored without a `title` to put it in. */
@@ -4568,7 +4566,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<"right" | "left" | "top" | "bottom">;
     /** 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.90",
   "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)];
 }