playbook_ui 16.11.0.pre.alpha.bettermetadata18084 → 16.11.0.pre.alpha.doublealpha17836

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dc8efac2071a693a945c9ad2b6bdd0ce9e7c092250ad1912188e33a24a2ed86e
-  data.tar.gz: 4c02ea4ad383b32bc3a153172ebb4a838c5f540d2d893d635e76ff8b55fa346b
+  metadata.gz: 79b45f2b67aeb4fa377761f1766b42a5a42d933f551775ab9438ff24610d5827
+  data.tar.gz: 578692dffae05005a1f3f30cd86d12060ff73cc667b8c2d5d9fe495fd10044b2
 SHA512:
-  metadata.gz: 37e1d808f292e4f6190b49305d9adfdedb1ee03b5e034c06f98b186164bb90c26f64d433b294e4803bb655598ad4f110d7148fbb004e53d6ac3834c0c0e5644f
-  data.tar.gz: 58f3be19870b5bbc06d6a4852dc171c76040410536636319f74db242e255b3a5014d1f070a273d32e56a4846fd5d37392f253307824a4f386aea1335a1914cef
+  metadata.gz: 96cac4feeec493c42d7b0a45af5372db920bda94369cc4dd92588c4c943a7a182f4c322377c2e83d7d42b49375ce2f18787c5032e4910daa17a4a49447ff5e3a
+  data.tar.gz: 0f0d19afcd066a7596002a83f952debf24ae6494cdac84f9ae9beb991bc5f45cfa05ff16e51561d711a7a4cf32c55b284542489e53518422ed7024d18dfa2a85

data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_column_styling.md

@@ -1,12 +1,15 @@
 The `columnStyling` prop is an optional item that can be used within `columnDefinitions` as shown in the code snippet below. It is an object that has several optional key/value pairs, this doc example highlights the following:
 
-1. `headerAlignment`: This will allow you to control alignment of header content which is set to right aligned by default. you can set this to `left`, `right` or `center`.
-2. `cellAlignment`: This will allow you to control alignment of content within all cells in the given column. This is set to right aligned by default. you can set this to `left`, `right` or `center`.
-3. `fontColor`: This will allow you to control the font color for a given column.
-4. Column width: optional keys on `columnStyling` are `minWidth`, `width`, and `maxWidth` (numbers = pixels; CSS strings allowed). This example sets `width` on Year for a fixed hierarchy column (see the width doc for `minWidth` and bands).
+1) `headerAlignment`: This will allow you to control alignment of header content which is set to right aligned by default. you can set this to `left`, `right` or `center`.
+
+2) `cellAlignment`: This will allow you to control alignment of content within all cells in the given column. This is set to right aligned by default. you can set this to `left`, `right` or `center`.
+
+3) `fontColor`: This will allow you to control the font color for a given column.
+
+4) Column width: optional keys on `columnStyling` are `minWidth`, `width`, and `maxWidth` (numbers = pixels; CSS strings allowed). This example sets `width` on Year for a fixed hierarchy column (see the width doc for `minWidth` and bands).
 
 Fixed width: pass `width` only (or TanStack `size` only) and Playbook sets min and max to the same value automatically — you do not need all three for an exact width.
 
-See [Column Styling: Width](https://playbook.powerapp.cloud/kits/advanced_table/styling/react#column-styling-width) for the full guide (Playbook ↔ TanStack mapping, floor-only vs bands, and when to use one vs three values).
+See [Column Styling: Width](https://playbook.powerapp.cloud/kits/advanced_table/react#column-styling-width) for the full guide (Playbook ↔ TanStack mapping, floor-only vs bands, and when to use one vs three values).
 
 `columnStyling` can be used within the columnDefinition of all the columns or some of them, as shown. Each column has its own individual control in this way. 

data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_column_styling_rails.html.erb

@@ -3,7 +3,6 @@
       accessor: "year",
       label: "Year",
       cellAccessors: ["quarter", "month", "day"],
-      column_styling: { width: 124 },
     },
     {
       accessor: "newEnrollments",

data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_column_styling_rails.md

@@ -1,12 +1,9 @@
 The `column_styling` prop is an optional item that can be used within `column_definitions` as shown in the code snippet below.  It is an object that has several optional key/value pairs, this doc example highlights the following:
 
 1) `header_alignment`: This will allow you to control alignment of header content which is set to right aligned by default. you can set this to `left`, `right` or `center`.
-2) `cell_alignment`: This will allow you to control alignment of content within all cells in the given column. This is set to right aligned by default. you can set this to `left`, `right` or `center`.
-3) `font_color`: This will allow you to control the font color for a given column.
-4) Column width: optional keys on `column_styling` are `min_width`, `width`, and `max_width` (numbers = pixels; CSS strings allowed). This example sets `width` on Year for a fixed hierarchy column (see the width doc for `min_width` and bands).
 
-Fixed width: pass `width` only and Playbook sets min and max to the same value automatically — you do not need all three for an exact width.
+2) `cell_alignment`: This will allow you to control alignment of content within all cells in the given column. This is set to right aligned by default. you can set this to `left`, `right` or `center`.
 
-See [Column Styling: Width](https://playbook.powerapp.cloud/kits/advanced_table/styling/rails#column-styling-width) for the full guide (floor-only vs bands, and when to use one vs three values).
+3) `font_color`: This will allow you to control the font color for a given column.
 
-`column_styling` can be used within the column_definition of all the columns or some of them, as shown. Each column has its own individual control in this way.
+`column_styling` can be used within the column_definition of all the columns or some of them, as shown. Each column has its own individual control in this way. 

data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_column_styling_width.md

@@ -1,6 +1,6 @@
 AdvancedTable column width is controlled in two equivalent places on each leaf `columnDefinitions` entry. Playbook maps them to inline styles on header and body cells (and forwards numeric values into TanStack Table’s column model).
 
-**1)** Playbook `columnStyling` and TanStack `ColumnDef` use the same three ideas:
+1) Playbook `columnStyling` and TanStack `ColumnDef` use the same three ideas:
 
 - Preferred / target width: `columnStyling` `width` maps to TanStack `size` on the same column object.
 - Minimum width (floor): `columnStyling` `minWidth` maps to TanStack `minSize`.
@@ -10,7 +10,7 @@ Numbers are pixels. You can also pass CSS length strings on `columnStyling` (e.g
 
 If both APIs set the same axis, `columnStyling` wins for that axis when Playbook builds cell styles.
 
-**2)** Fixed width: set `width` only
+2) Fixed width: set `width` only
 
 If you pass only `width` (or only `size`) and do not set `minWidth` / `maxWidth` (or `minSize` / `maxSize`), Playbook treats that as a fixed column: it sets all three to the same value under the hood so you do not have to repeat yourself.
 
@@ -26,7 +26,7 @@ size: 200
 
 Use this when the column should stay one width (e.g. a hierarchy column with expand controls).
 
-**3)** Floor only: `minWidth` / `minSize`
+3) Floor only: `minWidth` / `minSize`
 
 Set only a minimum when the column may grow with the table or content but must not shrink below a baseline (common fix for horizontal “jump” when rows expand):
 
@@ -34,7 +34,7 @@ Set only a minimum when the column may grow with the table or content but must n
 columnStyling: { minWidth: 160 }
 ```
 
-**4)** Flexible band: min + preferred + max
+4) Flexible band: min + preferred + max
 
 Set two or three values when you want a range. CSS uses preferred `width` clamped between `minWidth` and `maxWidth`:
 
@@ -46,7 +46,7 @@ Example from the table below (Attendance): `minWidth: 108`, `width: 124`, `maxWi
 
 You only need all three when you want that band. If min and max are omitted, `width` alone is enough for a fixed column.
 
-**5)** TanStack band without `columnStyling`
+5) TanStack band without `columnStyling`
 
 The Meetings column uses only TanStack fields:
 
@@ -56,7 +56,7 @@ The Meetings column uses only TanStack fields:
 
 Playbook applies the same min / preferred / max idea to cell styles. Setting only `size: 200` would lock all three to 200, same as `width: 200` in `columnStyling`.
 
-**6)** What the example table shows
+6) What the example table shows
 
 - Year (fixed): `columnStyling: { width: 128 }` — locked to 128px.
 - Enrollments (floor): `columnStyling: { minWidth: 160 }` — at least 160px; can grow.

data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_pinned_rows_rails.html.erb

@@ -19,8 +19,8 @@
       label: "Year",
       cellAccessors: ["quarter", "month", "day"],
       sort_menu: [
-        { item: "Year", link: "?sort=year_asc#pinned_rows_top_table", active: params["sort"] == "year_asc", direction: "asc" },
-        { item: "Year", link: "?sort=year_desc#pinned_rows_top_table", active: params["sort"] == "year_desc", direction: "desc" }
+        { item: "Year", link: "?sort=year_asc#pinned_rows_table", active: params["sort"] == "year_asc", direction: "asc" },
+        { item: "Year", link: "?sort=year_desc#pinned_rows_table", active: params["sort"] == "year_desc", direction: "desc" }
       ],
     },
     {
@@ -51,7 +51,7 @@
 
 <% pinned_rows = { top: ["8"] } %>
 
-<%= pb_rails("advanced_table", props: { id: "pinned_rows_top_table", table_data: @table_data_with_id, column_definitions: column_definitions, max_height: "xs", pinned_rows: pinned_rows, responsive: "none", table_props: { sticky: true }}) do %>
-  <%= pb_rails("advanced_table/table_header", props: { table_id: "pinned_rows_top_table", column_definitions: column_definitions }) %>
-  <%= pb_rails("advanced_table/table_body", props: { table_id: "pinned_rows_top_table", table_data: @table_data_with_id, column_definitions: column_definitions, pinned_rows: pinned_rows }) %>
+<%= pb_rails("advanced_table", props: { id: "pinned_rows_table", table_data: @table_data_with_id, column_definitions: column_definitions, max_height: "xs", pinned_rows: pinned_rows, responsive: "none", table_props: { sticky: true }}) do %>
+  <%= pb_rails("advanced_table/table_header", props: { table_id: "pinned_rows_table", column_definitions: column_definitions }) %>
+  <%= pb_rails("advanced_table/table_body", props: { table_id: "pinned_rows_table", table_data: @table_data_with_id, column_definitions: column_definitions, pinned_rows: pinned_rows }) %>
 <% end %>

data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_pinned_rows_rails.md

@@ -1,7 +1,7 @@
-Use the `pinned_rows` prop to pin specific rows to the top or bottom of an Advanced Table. Pinned rows will remain at the top or bottom when scrolling through table data and will not change position if sorting is used.
+Use the `pinned_rows` prop to pin specific rows to the top of an Advanced Table. Pinned rows will remain at the top when scrolling through table data and will not change position if sorting is used.
 
 **NOTE:**
 - Sticky header required: Pinned rows must be used with `sticky: true` via `table_props` (works with both responsive and non-responsive tables)
 - Row ids required: Each object within the `table_data` array must contain a unique `id` in order to attach an id to all Rows for this to function.
-- `pinned_rows` takes a hash with a `top` and/or `bottom` key containing an array of row ids, as shown in the code snippet below.
-- For expandable rows, use the parent id in `pinned_rows[:top]` or `pinned_rows[:bottom]`; all its children will automatically be pinned with it. If a child id is passed without the parent being pinned, nothing will be pinned.
+- `pinned_rows` takes a hash with a `top` key containing an array of row ids, as shown in the code snippet below.
+- For expandable rows, use the parent id in `pinned_rows[:top]`; all its children will automatically be pinned with it. If a child id is passed without the parent being pinned, nothing will be pinned.

data/app/pb_kits/playbook/pb_advanced_table/docs/_playground.json

@@ -12,7 +12,6 @@
     "fullWidthCell": false,
     "inlineRowLoading": false,
     "loading": false,
-    "maxHeight": "auto",
     "responsive": "scroll",
     "scrollBarNone": false,
     "selectableRows": false,

data/app/pb_kits/playbook/pb_advanced_table/docs/example.yml

@@ -9,9 +9,7 @@ examples:
   - advanced_table_table_props: Table Props
   - advanced_table_sticky_header_rails: Sticky Header
   - advanced_table_table_props_sticky_header: Sticky Header for Responsive Table
-  - advanced_table_pinned_rows_rails: Pinned Rows (Top)
-  - advanced_table_pinned_rows_bottom_rails: Pinned Rows (Bottom)
-  - advanced_table_pinned_rows_both_rails: Pinned Rows (Both)
+  - advanced_table_pinned_rows_rails: Pinned Rows
   - advanced_table_beta_sort: Enable Sorting
   - advanced_table_responsive: Responsive Tables
   - advanced_table_custom_cell_rails: Custom Components for Cells
@@ -30,7 +28,6 @@ examples:
   - advanced_table_row_styling: Row Styling
   - advanced_table_padding_control_per_row_rails: Padding Control using Row Styling
   - advanced_table_column_styling_rails: Column Styling
-  - advanced_table_column_styling_width_rails: Column Styling Width
   - advanced_table_column_styling_column_headers_rails: Column Styling with Multiple Headers
   - advanced_table_padding_control_rails: Padding Control using Column Styling
   - advanced_table_background_control_rails: Column Styling Background Color

data/app/pb_kits/playbook/pb_advanced_table/index.js

@@ -356,40 +356,23 @@ export default class PbAdvancedTable extends PbEnhancedElement {
   }
 
   /**
-   * Recompute sticky top/bottom for visible pinned rows so collapsed rows don't leave a gap.
+   * Recompute sticky top for visible pinned rows so collapsed rows don't leave a gap.
    * Call after expand/collapse and on load.
    */
   static updatePinnedRowsStickyTops(advancedTableWrapper) {
-    const headerOffset =
-      "calc(var(--advanced-table-header-height, 44px) + var(--advanced-table-action-bar-height, 0px))";
-
-    const pinnedTopTbody = advancedTableWrapper?.querySelector("tbody.pinned-rows-tbody");
-    if (pinnedTopTbody) {
-      const pinnedRows = Array.from(pinnedTopTbody.querySelectorAll("tr.pinned-row"));
-      const visibleRows = pinnedRows.filter(
-        (tr) => tr.style.display !== "none" && tr.offsetParent !== null
-      );
-
-      visibleRows.forEach((tr, index) => {
-        tr.style.top = `calc(${headerOffset} + 2.5em * ${index})`;
-        tr.style.bottom = "";
-      });
-    }
+    const pinnedTbody = advancedTableWrapper?.querySelector("tbody.pinned-rows-tbody");
+    if (!pinnedTbody) return;
 
-    const pinnedBottomTbody = advancedTableWrapper?.querySelector(
-      "tbody.pinned-rows-tbody-bottom"
+    const pinnedRows = Array.from(pinnedTbody.querySelectorAll("tr.pinned-row"));
+    const visibleRows = pinnedRows.filter(
+      (tr) => tr.style.display !== "none" && tr.offsetParent !== null
     );
-    if (pinnedBottomTbody) {
-      const pinnedRows = Array.from(pinnedBottomTbody.querySelectorAll("tr.pinned-row"));
-      const visibleRows = pinnedRows.filter(
-        (tr) => tr.style.display !== "none" && tr.offsetParent !== null
-      );
 
-      visibleRows.forEach((tr, index) => {
-        tr.style.bottom = `calc(2.5em * ${visibleRows.length - 1 - index})`;
-        tr.style.top = "";
-      });
-    }
+    const headerOffset =
+      "calc(var(--advanced-table-header-height, 44px) + var(--advanced-table-action-bar-height, 0px))";
+    visibleRows.forEach((tr, index) => {
+      tr.style.top = `calc(${headerOffset} + 2.5em * ${index})`;
+    });
   }
 
   hideCloseIcon() {

data/app/pb_kits/playbook/pb_advanced_table/kit.schema.json

@@ -125,8 +125,7 @@
     "maxHeight": {
       "type": "enum",
       "platforms": [
-        "react",
-        "rails"
+        "react"
       ],
       "values": [
         "auto",
@@ -137,8 +136,7 @@
         "xl",
         "xxl",
         "xxxl"
-      ],
-      "default": "auto"
+      ]
     },
     "onRowToggleClick": {
       "type": "function",

data/app/pb_kits/playbook/pb_advanced_table/table_body.html.erb

@@ -1,12 +1,10 @@
 <% table_data = object.table_data || [] %>
 <% if object.has_pinned_rows? %>
-  <% if object.has_pinned_top_rows? %>
-    <%= pb_content_tag(:tbody, class: "pinned-rows-tbody") do %>
-      <% next_index = 0 %>
-      <% object.pinned_top_root_rows.each do |root_info| %>
-        <% row_output, next_index = object.render_row_and_children(root_info[:row], object.column_definitions, root_info[:depth], root_info[:depth] > 0, root_info[:ancestor_ids] || [], root_info[:ancestor_ids]&.first, immediate_parent_row_id: object.row_id_for(root_info[:parent_row]), is_pinned_row: true, pinned_index: next_index, pinned_position: "top", initial_table_data_attributes: root_info[:depth].to_i > 0 ? object.pinned_root_initial_data_attributes(root_info) : nil) %>
-        <%= row_output %>
-      <% end %>
+  <%= pb_content_tag(:tbody, class: "pinned-rows-tbody") do %>
+    <% next_index = 0 %>
+    <% object.pinned_root_rows.each do |root_info| %>
+      <% row_output, next_index = object.render_row_and_children(root_info[:row], object.column_definitions, root_info[:depth], root_info[:depth] > 0, root_info[:ancestor_ids] || [], root_info[:ancestor_ids]&.first, immediate_parent_row_id: object.row_id_for(root_info[:parent_row]), is_pinned_row: true, pinned_index: next_index, initial_table_data_attributes: root_info[:depth].to_i > 0 ? object.pinned_root_initial_data_attributes(root_info) : nil) %>
+      <%= row_output %>
     <% end %>
   <% end %>
   <%= pb_content_tag(:tbody) do %>
@@ -15,16 +13,6 @@
       <%= result.is_a?(Array) ? result[0] : result %>
     <% end %>
   <% end %>
-  <% if object.has_pinned_bottom_rows? %>
-    <%= pb_content_tag(:tbody, class: "pinned-rows-tbody-bottom") do %>
-      <% next_index = 0 %>
-      <% bottom_total = object.pinned_bottom_row_count %>
-      <% object.pinned_bottom_root_rows.each do |root_info| %>
-        <% row_output, next_index = object.render_row_and_children(root_info[:row], object.column_definitions, root_info[:depth], root_info[:depth] > 0, root_info[:ancestor_ids] || [], root_info[:ancestor_ids]&.first, immediate_parent_row_id: object.row_id_for(root_info[:parent_row]), is_pinned_row: true, pinned_index: next_index, pinned_position: "bottom", pinned_bottom_total: bottom_total, initial_table_data_attributes: root_info[:depth].to_i > 0 ? object.pinned_root_initial_data_attributes(root_info) : nil) %>
-        <%= row_output %>
-      <% end %>
-    <% end %>
-  <% end %>
 <% else %>
   <%= pb_content_tag(:tbody) do %>
     <% table_data.each do |row| %>

data/app/pb_kits/playbook/pb_advanced_table/table_body.rb

@@ -46,7 +46,7 @@ module Playbook
         end.compact
       end
 
-      def render_row_and_children(row, column_definitions, current_depth, first_parent_child, ancestor_ids = [], top_parent_id = nil, additional_classes: "", table_data_attributes: {}, immediate_parent_row_id: nil, is_pinned_row: false, pinned_index: nil, skip_pinned_ids: nil, initial_table_data_attributes: nil, pinned_position: "top", pinned_bottom_total: nil)
+      def render_row_and_children(row, column_definitions, current_depth, first_parent_child, ancestor_ids = [], top_parent_id = nil, additional_classes: "", table_data_attributes: {}, immediate_parent_row_id: nil, is_pinned_row: false, pinned_index: nil, skip_pinned_ids: nil, initial_table_data_attributes: nil)
         if skip_pinned_ids && row_id_for(row) && skip_pinned_ids.include?(row_id_for(row).to_s)
           return is_pinned_row ? [ActiveSupport::SafeBuffer.new, pinned_index] : ActiveSupport::SafeBuffer.new
         end
@@ -71,11 +71,10 @@ module Playbook
         # Subrow header if applicable
         if is_first_child_of_subrow && enable_toggle_expansion == "all"
           subrow_props = { row: row, column_definitions: leaf_columns, depth: current_depth, subrow_header: subrow_headers_arr[current_depth - 1], collapsible_trail: collapsible_trail, classname: "toggle-content", responsive: responsive, subrow_data_attributes: subrow_data_attributes, last_row: last_row, immediate_parent_row_id: immediate_parent_row_id }
-          if is_pinned_row && !next_pinned_index.nil?
+          if is_pinned_row && next_pinned_index
             subrow_props[:is_pinned_row] = true
-            subrow_props[:pinned_position] = pinned_position.to_s
             subrow_props[:pinned_index] = next_pinned_index
-            subrow_props[:html_options] = { style: build_pinned_row_style(next_pinned_index, background: "var(--pb_table_sticky_bg, #f5f5f5)", position: pinned_position, bottom_total: pinned_bottom_total) }
+            subrow_props[:html_options] = { style: build_pinned_row_style(next_pinned_index, background: "var(--pb_table_sticky_bg, #f5f5f5)") }
             next_pinned_index += 1
           end
           output << pb_rails("advanced_table/table_subrow_header", props: subrow_props)
@@ -93,12 +92,11 @@ module Playbook
 
         # Additional class and data attributes needed for toggle logic
         row_props = { table_id: table_id, row: row, column_definitions: leaf_columns, depth: current_depth, collapsible_trail: collapsible_trail, classname: additional_classes, table_data_attributes: current_data_attributes, responsive: responsive, loading: loading, selectable_rows: selectable_rows, row_id: row[:id], enable_toggle_expansion: enable_toggle_expansion, row_styling: row_styling, last_row: last_row, immediate_parent_row_id: immediate_parent_row_id, inline_row_loading: inline_row_loading, full_width_cell: full_width_cell }
-        if is_pinned_row && !next_pinned_index.nil?
+        if is_pinned_row && next_pinned_index
           row_props[:is_pinned_row] = true
-          row_props[:pinned_position] = pinned_position.to_s
           row_props[:pinned_index] = next_pinned_index
           row_bg = (row_styling || []).find { |s| s[:row_id].to_s == row_id_for(row).to_s }&.[](:background_color) || "white"
-          row_props[:html_options] = { style: build_pinned_row_style(next_pinned_index, background: row_bg, position: pinned_position, bottom_total: pinned_bottom_total) }
+          row_props[:html_options] = { style: build_pinned_row_style(next_pinned_index, background: row_bg) }
           next_pinned_index += 1
         end
         output << pb_rails("advanced_table/table_row", props: row_props)
@@ -136,8 +134,6 @@ module Playbook
             child_opts = { additional_classes: "toggle-content", table_data_attributes: child_data_attributes, immediate_parent_row_id: row[:id] }
             child_opts[:is_pinned_row] = is_pinned_row
             child_opts[:pinned_index] = next_pinned_index if is_pinned_row
-            child_opts[:pinned_position] = pinned_position if is_pinned_row
-            child_opts[:pinned_bottom_total] = pinned_bottom_total if is_pinned_row
             child_opts[:skip_pinned_ids] = skip_pinned_ids if skip_pinned_ids
 
             child_output, next_pinned_index = render_row_and_children(child_row, column_definitions, current_depth + 1, is_first_child, new_ancestor_ids, top_parent_id, **child_opts)
@@ -191,16 +187,17 @@ module Playbook
       end
 
       def pinned_top_ids
-        pinned_ids_for(:top)
-      end
+        return [] if pinned_rows.nil? || !pinned_rows.respond_to?(:[])
 
-      def pinned_bottom_ids
-        pinned_ids_for(:bottom)
+        top = pinned_rows["top"] || pinned_rows[:top]
+        Array(top).map(&:to_s)
       end
 
       def pinned_ids_set
+        return Set.new if pinned_top_ids.blank?
+
         set = Set.new
-        (pinned_top_root_rows + pinned_bottom_root_rows).each do |root|
+        pinned_root_rows.each do |root|
           collect_row_and_descendant_ids(root[:row], set)
         end
         set
@@ -223,55 +220,21 @@ module Playbook
         nil
       end
 
-      def pinned_top_root_rows
+      def pinned_root_rows
         return [] if pinned_top_ids.blank?
 
         pinned_top_ids.filter_map { |id| find_row_by_id(table_data, id) }
       end
 
-      # Alias kept for existing call sites / specs
-      alias pinned_root_rows pinned_top_root_rows
-
-      def pinned_bottom_root_rows
-        return [] if pinned_bottom_ids.blank?
-
-        pinned_bottom_ids.filter_map { |id| find_row_by_id(table_data, id) }
-      end
-
-      def pinned_bottom_row_count
-        pinned_bottom_root_rows.size
-      end
-
-      def has_pinned_top_rows?
-        pinned_top_root_rows.any?
-      end
-
-      def has_pinned_bottom_rows?
-        pinned_bottom_root_rows.any?
-      end
-
       def has_pinned_rows?
-        has_pinned_top_rows? || has_pinned_bottom_rows?
+        pinned_root_rows.any?
       end
 
       # Build inline style for sticky pinned row (matches React). Pass via html_options so the tr gets the attribute.
-      def build_pinned_row_style(pinned_index, background: "white", position: "top", bottom_total: nil)
-        if position.to_s == "bottom"
-          offset_index = bottom_total ? [bottom_total - 1 - pinned_index, 0].max : pinned_index
-          "position: sticky; bottom: calc(2.5em * #{offset_index}); z-index: 3; background: #{background};"
-        else
-          header_offset = "calc(var(--advanced-table-header-height, 44px) + var(--advanced-table-action-bar-height, 0px))"
-          row_offset = "calc(2.5em * #{pinned_index})"
-          "position: sticky; top: calc(#{header_offset} + #{row_offset}); z-index: 3; background: #{background};"
-        end
-      end
-
-      def pinned_ids_for(position)
-        return [] if pinned_rows.nil? || !pinned_rows.respond_to?(:[])
-
-        key = position.to_s
-        ids = pinned_rows[key] || pinned_rows[key.to_sym]
-        Array(ids).map(&:to_s)
+      def build_pinned_row_style(pinned_index, background: "white")
+        header_offset = "calc(var(--advanced-table-header-height, 44px) + var(--advanced-table-action-bar-height, 0px))"
+        row_offset = "calc(2.5em * #{pinned_index})"
+        "position: sticky; top: calc(#{header_offset} + #{row_offset}); z-index: 3; background: #{background};"
       end
 
       def pinned_root_initial_data_attributes(root_info)

data/app/pb_kits/playbook/pb_advanced_table/table_header.rb

@@ -1,11 +1,8 @@
 # frozen_string_literal: true
 
-require_relative "column_layout_helper"
-
 module Playbook
   module PbAdvancedTable
     class TableHeader < Playbook::KitBase
-      include Playbook::PbAdvancedTable::ColumnLayoutHelper
       prop :table_id, type: Playbook::Props::String,
                       default: ""
       prop :column_definitions, type: Playbook::Props::Array,
@@ -167,9 +164,6 @@ module Playbook
       def header_component_info(cell, cell_index, row_index)
         header_id = cell[:accessor].present? ? cell[:accessor] : "header_#{row_index}_#{cell_index}"
         classname = [th_classname(is_first_column: cell_index.zero?), ("last-header-cell" if cell[:is_last_in_group] && cell_index != 0)].compact.join(" ")
-        layout_styles = header_layout_styles(cell)
-        style_hash = layout_styles.dup
-        style_hash[:color] = cell[:header_font_color] if cell[:header_font_color].present?
 
         if has_custom_header_background_color?(cell)
           component_name = "background"
@@ -181,8 +175,9 @@ module Playbook
           component_props[:html_options] = {
             id: header_id,
             colspan: cell[:colspan],
-            style: style_hash,
+            style: { color: cell[:header_font_color] },
           }
+          component_props[:html_options][:style].delete(:color) unless cell[:header_font_color].present?
         else
           component_name = "table/table_header"
           component_props = {
@@ -191,7 +186,7 @@ module Playbook
             classname: classname,
             sort_menu: cell[:accessor] ? cell[:sort_menu] : nil,
           }
-          component_props[:html_options] = { style: style_hash } if style_hash.present?
+          component_props[:html_options] = { style: { color: cell[:header_font_color] } } if cell[:header_font_color].present?
         end
 
         { name: component_name, props: component_props }
@@ -212,13 +207,6 @@ module Playbook
 
     private
 
-      def header_layout_styles(cell)
-        original_def = find_original_column_def_for_cell(cell)
-        return {} unless original_def
-
-        build_column_layout_styles_from_column(original_def)
-      end
-
       # Find the original column definition for a cell
       def find_original_column_def_for_cell(cell)
         # Try accessor first (for leaf columns)

data/app/pb_kits/playbook/pb_advanced_table/table_row.rb

@@ -1,11 +1,8 @@
 # frozen_string_literal: true
 
-require_relative "column_layout_helper"
-
 module Playbook
   module PbAdvancedTable
     class TableRow < Playbook::KitBase
-      include Playbook::PbAdvancedTable::ColumnLayoutHelper
       prop :table_id, type: Playbook::Props::String,
                       default: ""
       prop :column_definitions, type: Playbook::Props::Array,
@@ -42,9 +39,6 @@ module Playbook
                              default: false
       prop :is_pinned_row, type: Playbook::Props::Boolean,
                            default: false
-      prop :pinned_position, type: Playbook::Props::Enum,
-                             values: %w[top bottom],
-                             default: "top"
       prop :pinned_index, type: Playbook::Props::Numeric,
                           default: nil
       prop :html_options, type: Playbook::Props::HashProp,
@@ -59,7 +53,6 @@ module Playbook
       def classname
         classes = ["pb_table_tr", "pb-bg-row-white", subrow_depth_classname]
         classes << "pinned-row" if is_pinned_row
-        classes << "pinned-row-bottom" if is_pinned_row && pinned_position.to_s == "bottom"
         classes.reject!(&:blank?)
         generate_classname(*classes, separator: " ")
       end
@@ -120,7 +113,6 @@ module Playbook
         column_font_color = cell_font_color(column)
         effective_font_color = column_font_color || font_color
         effective_font_weight = font_weight_value(font_weight)
-        layout_styles = cell_layout_styles(column)
 
         if has_custom_background_color?(column)
           custom_bg_color = cell_background_color(column)
@@ -130,13 +122,13 @@ module Playbook
             tag: "td",
             classname: td_classname(column, index),
           }
-          style_hash = layout_styles.dup
+          style_hash = {}
           style_hash[:color] = effective_font_color if effective_font_color.present?
           style_hash[:"font-weight"] = effective_font_weight if effective_font_weight.present?
           component_props[:html_options] = { style: style_hash } if style_hash.present?
         else
           component_name = "table/table_cell"
-          style_hash = layout_styles.merge("background-color": bg_color)
+          style_hash = { "background-color": bg_color }
           style_hash[:color] = effective_font_color if effective_font_color.present?
           style_hash[:"font-weight"] = effective_font_weight if effective_font_weight.present?
           component_props = {
@@ -242,15 +234,6 @@ module Playbook
 
     private
 
-      def cell_layout_styles(column)
-        return {} unless column[:accessor].present?
-
-        orig_def = find_column_def_by_accessor(column_definitions, column[:accessor])
-        return {} unless orig_def
-
-        build_column_layout_styles_from_column(orig_def)
-      end
-
       def custom_renderer_value(column, index)
         return nil unless column[:accessor].present?
 

data/app/pb_kits/playbook/pb_advanced_table/table_subrow_header.rb

@@ -21,9 +21,6 @@ module Playbook
                         default: "scroll"
       prop :is_pinned_row, type: Playbook::Props::Boolean,
                            default: false
-      prop :pinned_position, type: Playbook::Props::Enum,
-                             values: %w[top bottom],
-                             default: "top"
       prop :pinned_index, type: Playbook::Props::Numeric,
                           default: nil
       prop :html_options, type: Playbook::Props::HashProp,
@@ -36,7 +33,6 @@ module Playbook
       def classname
         classes = ["pb_table_tr", "bg-silver", "pb_subrow_header", subrow_depth_classname]
         classes << "pinned-row" if is_pinned_row
-        classes << "pinned-row-bottom" if is_pinned_row && pinned_position.to_s == "bottom"
         generate_classname(*classes, separator: " ")
       end
 

data/app/pb_kits/playbook/pb_avatar/docs/_playground.json

@@ -2,7 +2,6 @@
   "template": "<Avatar{{props}} />",
   "propTargets": {},
   "defaults": {
-    "dark": false,
     "grayscale": false,
     "imageAlt": "",
     "name": "",

data/app/pb_kits/playbook/pb_avatar/kit.schema.json

@@ -17,10 +17,8 @@
     "dark": {
       "type": "boolean",
       "platforms": [
-        "react",
-        "rails"
-      ],
-      "default": false
+        "react"
+      ]
     },
     "grayscale": {
       "type": "boolean",

data/app/pb_kits/playbook/pb_background/kit.schema.json

@@ -14,7 +14,7 @@
       ]
     },
     "backgroundColor": {
-      "type": "enum",
+      "type": "ResponsiveProp<BackgroundColors> | BackgroundColors",
       "platforms": [
         "react",
         "rails"
@@ -85,7 +85,7 @@
       "default": "light"
     },
     "backgroundSize": {
-      "type": "enum",
+      "type": "ResponsiveProp<BackgroundSizes> | BackgroundSizes",
       "platforms": [
         "react",
         "rails"
@@ -106,7 +106,7 @@
       "default": null
     },
     "backgroundRepeat": {
-      "type": "enum",
+      "type": "ResponsiveProp<BackgroundRepeat> | BackgroundRepeat",
       "platforms": [
         "react",
         "rails"

data/app/pb_kits/playbook/pb_card/docs/_playground.json

@@ -7,7 +7,6 @@
   "defaults": {
     "background": "none",
     "borderNone": false,
-    "borderRadius": "md",
     "draggableItem": false,
     "dragHandle": true,
     "selected": false,