playbook_ui 16.11.0.pre.alpha.PLAY3102dropdownautocomppletedefaultcombo18078 → 16.11.0.pre.alpha.PLAY308717843

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2274169a7343e1aeae6a9f331f25d131f4a4a5ff429fa2fc327922fc970cc0e1
-  data.tar.gz: 2e78ec075b610edf5ae20ae9aa99c9821e98e6062b8e42b3acdcca9919d7cca6
+  metadata.gz: 00db3d23c89ba37db73a4991f6df1fe6642c7ad40790dea3523d91d27a6eac09
+  data.tar.gz: e10a651ffc72c33c9ad69f93a61ab4a7b617552a86adbaf810cc0b2521c496b4
 SHA512:
-  metadata.gz: 9840e815b2ff2a5120649a0428b357528ceaa2883066300c86f907eb979258c4ec3d27c0ede23577736431e76ffdf9acacf62aeea458b4a9c3968ddf728655e2
-  data.tar.gz: a42bd6d0a80b9f57d75a34e40426a5583de70c26d1d4558fb9df83f2f2e34c2a14d1d2a5875acaec17d6985d1040e8c014b24eb52cb05950e321c4001c24e8eb
+  metadata.gz: 1cc120b2aade21a5e4835307a8fcc5efefe19e69b5f245b8f964b62a6bb04392dd13d9d11670762ae95196239ab64a14bf8c2e2f6f0a63af9137f534e21a9b1d
+  data.tar.gz: c01f8df2a5af7711620e408c50848a5f2ca28139d2c8d4f00e2a67c288d799b33183c5877978750300ac7e9d93f14e6f4dc8b5e7e769291002c0ae14715ad7de

data/app/pb_kits/playbook/pb_advanced_table/Components/CustomCell.tsx

@@ -21,7 +21,6 @@ interface CustomCellProps {
   customRenderer?: (row: Row<GenericObject>, value: string | undefined) => React.ReactNode
   selectableRows?: boolean
   customStyle?: GenericObject
-  fullWidthCell?: boolean
 } 
 
 export const CustomCell = ({
@@ -32,7 +31,6 @@ export const CustomCell = ({
   customRenderer,
   selectableRows,
   customStyle = {},
-  fullWidthCell,
 }: CustomCellProps & GlobalProps) => {
   const { setExpanded, expanded, expandedControl, inlineRowLoading, hasAnySubRows, cascadeCollapse } = useContext(AdvancedTableContext);
 
@@ -60,7 +58,7 @@ export const CustomCell = ({
   const renderButton = inlineRowLoading ? RowHasChildren : row.getCanExpand()
 
   return (
-    <div style={{ paddingLeft: `${row.depth * 1.25}em`, ...(fullWidthCell ? { width: "100%" } : {}) }}>
+    <div style={{ paddingLeft: `${row.depth * 1.25}em`}}>
       <Flex 
           alignItems="center" 
           columnGap="xs"
@@ -96,10 +94,7 @@ export const CustomCell = ({
             )}
           </button>
         ) : null}
-        <FlexItem
-            paddingLeft={renderButton? "none" : "xs"}
-            width={fullWidthCell ? "100%" : undefined}
-        >
+        <FlexItem paddingLeft={renderButton? "none" : "xs"}>
           {row.depth === 0 ? (
             customRenderer ? customRenderer(row, getValue()) : getValue()
            ) :(

data/app/pb_kits/playbook/pb_advanced_table/Components/RegularTableView.tsx

@@ -213,8 +213,6 @@ export const RegularTableView = ({
       ))}
       {totalRows.map((row: Row<GenericObject>, rowIndex: number) => {
         const isFirstChildofSubrow = row.depth > 0 && row.index === 0;
-        const subRowHeaderText = subRowHeaders?.[row.depth - 1]?.trim();
-        const shouldRenderSubRowHeader = isFirstChildofSubrow && !!subRowHeaderText;
         const numberOfColumns = table.getAllFlatColumns().length;
         const isFirstRegularRow = rowIndex === 0 && !row.getIsPinned();
         const customRowStyle = getRowStyle(rowStyling, row);
@@ -225,7 +223,7 @@ export const RegularTableView = ({
 
         return (
           <React.Fragment key={`${row.index}-${row.id}-${row.depth}-row`}>
-            {shouldRenderSubRowHeader && (
+            {isFirstChildofSubrow && subRowHeaders && (
               <SubRowHeaderRow
                   collapsibleTrail={collapsibleTrail}
                   enableToggleExpansion={enableToggleExpansion}

data/app/pb_kits/playbook/pb_advanced_table/Hooks/useTableState.ts

@@ -42,7 +42,6 @@ interface UseTableStateProps {
   rowStyling?: GenericObject[];
   inlineRowLoading?: boolean;
   sortParentOnly?: boolean;
-  fullWidthCell?: boolean;
 }
 
 export function useTableState({
@@ -63,8 +62,7 @@ export function useTableState({
   pinnedRows,
   rowStyling,
   inlineRowLoading = false,
-  sortParentOnly = false,
-  fullWidthCell = false,
+  sortParentOnly = false
 }: UseTableStateProps) {
 
   // Create a local state for expanded and setExpanded if expandedControl not used
@@ -147,14 +145,13 @@ export function useTableState({
           isFirstColumn,
           onRowToggleClick,
           selectableRows,
-          rowStyling,
-          fullWidthCell
+          rowStyling
         );
       }
 
       return columnStructure;
     }) || [];
-  }, [columnHelper, onRowToggleClick, rowStyling, selectableRows, fullWidthCell]);
+  }, [columnHelper, onRowToggleClick, rowStyling, selectableRows]);
 
   const columns = useMemo(() => buildColumns(columnDefinitions), [buildColumns, columnDefinitions]);
 

data/app/pb_kits/playbook/pb_advanced_table/Utilities/CellRendererUtils.tsx

@@ -12,8 +12,6 @@ import { getRowStyle } from "./RowUtils";
  * @param isFirstColumn Whether this is the first column (special handling)
  * @param onRowToggleClick Optional callback for row toggle
  * @param selectableRows Whether rows are selectable
- * @param rowStyling Optional row styling configuration
- * @param fullWidthCell Whether first-column cell content should take full width
  */
 export const createCellFunction = (
   cellAccessors: string[],
@@ -21,8 +19,7 @@ export const createCellFunction = (
   isFirstColumn?: boolean,
   onRowToggleClick?: (row: Row<GenericObject>) => void,
   selectableRows?: boolean,
-  rowStyling?: GenericObject[],
-  fullWidthCell?: boolean
+  rowStyling?: GenericObject[]
 ) => {
   // Add display name to the returned function
   const cellRenderer = ({
@@ -42,7 +39,6 @@ export const createCellFunction = (
             <CustomCell
                 customRenderer={customRenderer}
                 customStyle={customStyle}
-                fullWidthCell={fullWidthCell}
                 getValue={getValue}
                 onRowToggleClick={onRowToggleClick}
                 row={row}
@@ -58,7 +54,6 @@ export const createCellFunction = (
             <CustomCell
                 customRenderer={customRenderer}
                 customStyle={customStyle}
-                fullWidthCell={fullWidthCell}
                 onRowToggleClick={onRowToggleClick}
                 row={row}
                 selectableRows={selectableRows}

data/app/pb_kits/playbook/pb_advanced_table/_advanced_table.tsx

@@ -36,7 +36,6 @@ type AdvancedTableProps = {
   expandedControl?: GenericObject
   expandByDepth?: GenericObject[]
   onExpandByDepthClick?: (arg: number, arg1: any) => void
-  fullWidthCell?: boolean
   htmlOptions?: {[key: string]: string | number | boolean | (() => void)},
   id?: string
   initialLoadingRowsCount?: number
@@ -99,7 +98,6 @@ const AdvancedTable = (props: AdvancedTableProps) => {
     expandedControl,
     expandByDepth,
     onExpandByDepthClick,
-    fullWidthCell = false,
     htmlOptions = {},
     id,
     initialLoadingRowsCount = 10,
@@ -166,8 +164,7 @@ const AdvancedTable = (props: AdvancedTableProps) => {
     pinnedRows,
     rowStyling,
     inlineRowLoading,
-    sortParentOnly,
-    fullWidthCell,
+    sortParentOnly
   });
 
   // Initialize table actions

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

@@ -12,7 +12,7 @@
       <% content.presence %>
     <% else %>
       <%= pb_rails("advanced_table/table_header", props: { table_id: object.id, column_definitions: object.column_definitions, enable_toggle_expansion: object.enable_toggle_expansion, responsive: object.responsive, loading: object.loading, selectable_rows: object.selectable_rows, show_actions_bar: object.show_actions_bar, inline_row_loading: object.inline_row_loading, persist_toggle_expansion_button: object.persist_toggle_expansion_button, table_data: object.table_data }) %>
-      <%= pb_rails("advanced_table/table_body", props: { table_id: object.id, table_data: object.table_data, column_definitions: object.column_definitions, responsive: object.responsive, loading: object.loading, selectable_rows: object.selectable_rows, enable_toggle_expansion: object.enable_toggle_expansion, row_styling: object.row_styling, inline_row_loading: object.inline_row_loading, full_width_cell: object.full_width_cell, pinned_rows: object.pinned_rows }) %>
+      <%= pb_rails("advanced_table/table_body", props: { table_id: object.id, table_data: object.table_data, column_definitions: object.column_definitions, responsive: object.responsive, loading: object.loading, selectable_rows: object.selectable_rows, enable_toggle_expansion: object.enable_toggle_expansion, row_styling: object.row_styling, inline_row_loading: object.inline_row_loading, pinned_rows: object.pinned_rows }) %>
     <% end %>
   <% end %>
 <% end %>

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

@@ -35,8 +35,6 @@ module Playbook
                          default: []
       prop :inline_row_loading, type: Playbook::Props::Boolean,
                                 default: false
-      prop :full_width_cell, type: Playbook::Props::Boolean,
-                             default: false
       prop :persist_toggle_expansion_button, type: Playbook::Props::Boolean,
                                              default: false
       prop :pinned_rows, type: Playbook::Props::HashProp,

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

@@ -9,10 +9,8 @@
   "defaults": {
     "columnGroupBorderColor": "none",
     "enableToggleExpansion": "header",
-    "fullWidthCell": false,
     "inlineRowLoading": false,
     "loading": false,
-    "maxHeight": "auto",
     "responsive": "scroll",
     "scrollBarNone": false,
     "selectableRows": false,
@@ -94,8 +92,7 @@
       "name": "Column Configuration",
       "props": [
         "columnVisibilityControl",
-        "columnGroupBorderColor",
-        "fullWidthCell"
+        "columnGroupBorderColor"
       ]
     },
     {

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

@@ -160,7 +160,7 @@
     },
     {
       "name": "Column Configuration",
-      "props": ["columnVisibilityControl", "columnGroupBorderColor", "fullWidthCell"]
+      "props": ["columnVisibilityControl", "columnGroupBorderColor"]
     },
     {
       "name": "Table engine",

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

@@ -3,19 +3,15 @@ examples:
   - advanced_table_beta: Default (Required Props)
   - advanced_table_loading: Loading State
   - advanced_table_beta_subrow_headers: SubRow Headers
-  - advanced_table_skipping_subrow_headers: Skipping Subrow Headers
   - advanced_table_enable_toggle_expansion_rails: Enable Toggle Expansion
   - advanced_table_collapsible_trail_rails: Collapsible Trail
   - 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
-  - advanced_table_full_width_cell_rails: Full Width Cell
   - advanced_table_with_custom_header_rails: Custom Header Cell
   - advanced_table_with_custom_header_multi_header_rails: Custom Header with Multiple Headers
   - advanced_table_column_headers: Multi-Header Columns
@@ -50,7 +46,6 @@ examples:
   - advanced_table_expanded_control: Expanded Control
   - advanced_table_expand_by_depth: Expand by Depth
   - advanced_table_subrow_headers: SubRow Headers
-  - advanced_table_skipping_subrow_headers: Skipping Subrow Headers
   - advanced_table_cascade_collapse: Cascade Collapse
   - advanced_table_collapsible_trail: Collapsible Trail
   - advanced_table_table_options: Table Options
@@ -62,7 +57,6 @@ examples:
   - advanced_table_sticky_scroll_limitation: Sticky Header and Column Scroll Limitation
   - advanced_table_responsive: Responsive Tables
   - advanced_table_custom_cell: Custom Components for Cells
-  - advanced_table_full_width_cell: Full Width Cell
   - advanced_table_with_custom_header: Custom Header Cell
   - advanced_table_with_custom_header_multi_header: Custom Header with Multiple Headers
   - advanced_table_pagination: Pagination

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

@@ -10,7 +10,6 @@ export { default as AdvancedTableTableProps } from './_advanced_table_table_prop
 export { default as AdvancedTableInlineRowLoading } from './_advanced_table_inline_row_loading.jsx'
 export { default as AdvancedTableResponsive } from './_advanced_table_responsive.jsx'
 export { default as AdvancedTableCustomCell } from './_advanced_table_custom_cell.jsx'
-export { default as AdvancedTableFullWidthCell } from './_advanced_table_full_width_cell.jsx'
 export { default as AdvancedTablePagination } from './_advanced_table_pagination.jsx'
 export { default as AdvancedTablePaginationWithProps } from './_advanced_table_pagination_with_props.jsx'
 export { default as AdvancedTableColumnHeaders } from './_advanced_table_column_headers.jsx'
@@ -54,6 +53,5 @@ export { default as AdvancedTableColumnStylingBackgroundMulti } from './_advance
 export { default as AdvancedTableColumnStylingBackgroundCustom } from './_advanced_table_column_styling_background_custom.jsx'
 export { default as AdvancedTableCascadeCollapse } from './_advanced_table_cascade_collapse.jsx'
 export { default as AdvancedTableSortParentOnly } from './_advanced_table_sort_parent_only.jsx'
-export { default as AdvancedTableSkippingSubrowHeaders } from './_advanced_table_skipping_subrow_headers.jsx'
 export { default as AdvancedTablePinnedRowsBottom } from './_advanced_table_pinned_rows_bottom.jsx'
 export { default as AdvancedTablePinnedRowsBoth } from './_advanced_table_pinned_rows_both.jsx'

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

@@ -92,14 +92,6 @@
         "react"
       ]
     },
-    "fullWidthCell": {
-      "type": "boolean",
-      "platforms": [
-        "react",
-        "rails"
-      ],
-      "default": false
-    },
     "initialLoadingRowsCount": {
       "type": "number",
       "platforms": [
@@ -125,8 +117,7 @@
     "maxHeight": {
       "type": "enum",
       "platforms": [
-        "react",
-        "rails"
+        "react"
       ],
       "values": [
         "auto",
@@ -137,8 +128,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

@@ -27,8 +27,6 @@ module Playbook
                          default: []
       prop :inline_row_loading, type: Playbook::Props::Boolean,
                                 default: false
-      prop :full_width_cell, type: Playbook::Props::Boolean,
-                             default: false
       prop :pinned_rows, type: Playbook::Props::HashProp,
                          default: {}
 
@@ -46,7 +44,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 +69,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)
@@ -92,13 +89,12 @@ module Playbook
                                   end
 
         # 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?
+        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 }
+        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 +132,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 +185,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 +218,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_row.html.erb

@@ -26,7 +26,7 @@
                     <% end %>
                 <% end %>
 
-                <div style="padding-left: <%= depth * 1.25 %>em;<%= ' width: 100%;' if index.zero? && object.full_width_cell %>">
+                <div style="padding-left: <%= depth * 1.25 %>em">
                     <%= pb_rails("flex", props:{align: "center", column_gap: "xs"}) do %>
                         <% if index.zero? %>
                             <% has_integrated_checkbox = object.selectable_rows && object.enable_toggle_expansion != "none" %>
@@ -47,7 +47,7 @@
                                 </button>
                             <% end %>
                         <% end %>
-                        <%= pb_rails("flex/flex_item", props: (index.zero? && object.full_width_cell ? { width: "100%" } : {})) do %>
+                        <%= pb_rails("flex/flex_item") do %>       
                             <% if column[:custom_renderer].present? %>
                                 <%= raw(column[:custom_renderer].call(object.row, custom_renderer_value(column, index))) %>
                             <% elsif index.zero? %>

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

@@ -38,13 +38,8 @@ module Playbook
                                      default: ""
       prop :inline_row_loading, type: Playbook::Props::Boolean,
                                 default: false
-      prop :full_width_cell, type: Playbook::Props::Boolean,
-                             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 +54,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

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": "",