@one-grid-core/angular 0.5.3 → 0.5.4

package/lib/components/one-grid-row-action/one-grid-row-action.component.d.ts

@@ -37,6 +37,7 @@ export declare class OneGridRowActionComponent<T> implements OnInit {
     protected detailActiveExcludeLevels: Set<number>;
     protected gotoDetailExcludeLevels: Set<number>;
     protected addNewExcludeLevels: Set<number>;
+    protected gotoListExcludeLevels: Set<number>;
     /**
      * Accessible names for the action buttons.
      *

package/lib/core/models/one-column-def-types.model.d.ts

@@ -98,6 +98,7 @@ export interface OneCellRendererParams<T = any> {
     copyButtonStatus?: OneCellStatus;
     deleteButtonStatus?: OneCellStatus;
     gotoListButtonStatus?: OneCellStatus;
+    gotoListExcludeLevels?: number[];
     optionButtonStatus?: OneCellStatus;
     languageButtonStatus?: OneCellStatus;
     editButtonStatus?: OneCellStatus;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@one-grid-core/angular",
-  "version": "0.5.3",
+  "version": "0.5.4",
   "description": "OneGrid — a feature-rich Angular data grid component with tree data, master/detail, editing, selection, pagination, and virtual scrolling.",
   "keywords": [
     "angular",

package/scss/_one-grid.scss

@@ -57,9 +57,22 @@ $row-drag-indicator-size: 2px !default;
 
 // Master / detail
 //
-// The detail area is a continuation of the row that opened it, so it sits
-// flush by default. Set this to inset it.
-$detail-padding: 0 !default;
+// The inset around a detail row's nested grid — one step of the spacing scale
+// (the same 8px as `$cell-item-space`, declared literally because that
+// variable comes later in this file).
+//
+// A gap this size is what separates the nested grid from the master rows above
+// and below it without turning it into a card: it reads as content belonging
+// to its row, and the left inset indents it the way nesting is read everywhere
+// else in the grid. Anything larger starts to float; anything smaller stops
+// registering as deliberate.
+//
+// It costs height. `node.detailRowHeight` is the nested grid's own height, and
+// the block around it is that plus this inset top and bottom, which the body's
+// fixed-`itemSize` virtual scroller cannot see — the same drift an expanded
+// row already causes, 2 × this larger. Set it to `0` for a flush detail area
+// that adds nothing at all.
+$detail-padding: 8px !default;
 
 // Cell
 $cell-height: 30px !default;
@@ -531,17 +544,18 @@ $_og-defaults: (
                 outline-offset: -2px;
             }
 
-            // A detail row is a continuation of the master row above it, not a
-            // card floating inside the grid, so it sits flush: its left edge
-            // lines up with the master grid's and it occupies exactly the
-            // height the API reserved.
+            // A detail row belongs to the master row above it, so it is inset
+            // rather than boxed: `detail-padding` all round separates it from
+            // the rows either side and indents it, and the body background
+            // shows through the gap instead of a border or a card surface.
             //
-            // That reserved height is the reason the default is 0 rather than
-            // a taste call. `node.detailRowHeight` is what the grid computes
-            // for an open row, and the body's virtual-scroll viewport is
-            // configured with a single fixed `itemSize` — any padding here is
-            // height the scroller cannot see, so it becomes scroll drift on
-            // top of the drift an expanded row already causes.
+            // The inset is the only height this box adds. The nested grid
+            // keeps exactly the height the API reserved for it — padding is
+            // never taken out of it, which would clip its last row — so the
+            // block is `node.detailRowHeight` plus the inset top and bottom.
+            // The body's virtual-scroll viewport runs on a single fixed
+            // `itemSize` and cannot see any of it; `--one-grid-detail-padding: 0`
+            // restores a flush detail area that adds nothing beyond the grid.
             //
             // `height: inherit` used to sit here. It resolved to `auto`, since
             // that is the content wrapper's computed height, so it never did