playbook_ui 16.1.0.pre.alpha.play277814027 → 16.1.0.pre.rc.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1ba1ccdd547ffbb9a3028c4f7ebe9c545e978f3a156cb09f8d23f72be12ca8e5
-  data.tar.gz: c99e2ee648ba1e7a07f2312a927705a23b2c0d231d25e6d9f7ff896c1994cc47
+  metadata.gz: ec3832a5a9d172af005bea73161b414b33cce3a4b9351680a75be21567f70f1f
+  data.tar.gz: 9b88540268a3c696e5e810642e48ceeededa245b3ce5a3e46e23562b85e0a339
 SHA512:
-  metadata.gz: d3cd8f6f1ce697e21eba5d9e2825d2b9e405282cde1f2944dae69b6bf159c643e10ed0726cd88015fbf15c4da6727408afb25b318c8196bcc58457c976cb7bf2
-  data.tar.gz: 2d7b6db05ed92ce9a0d2c0f8a4bfcb374f4bea28e4807b1591d38a1a792902ba45bd086e4b36dc696330187142185f6995f566bec7d4b3e851363571bd2d7c76
+  metadata.gz: ad0694a212c9b5924b651c30038152649a9f779f11ea81ad5970315aefcb3a14534654c366049d8ba1d725180c907a9611c3b7c4840ec4cd7f971a11599d135f
+  data.tar.gz: 7c69619d23491fde3af8952c6e899afffff9c65df6a42d81c938fc6fd37c08e90706bb7eb36f9d9dc6267c83826851424964c5adbc4ac4bd67a04e4294947b02

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

@@ -66,18 +66,8 @@ const TableCellRenderer = ({
         // Find the "owning" colDefinition by accessor. Needed for multi column logic
         const colDef = findColumnDefByAccessor(columnDefinitions ?? [], column.id)
         const cellAlignment = colDef?.columnStyling?.cellAlignment ?? "right"
-        
-        // Support function-based styling for conditional rendering
-        const cellFontColorValue = colDef?.columnStyling?.fontColor
-        const cellFontColor = typeof cellFontColorValue === 'function' 
-          ? cellFontColorValue(row) 
-          : cellFontColorValue
-        
-        const cellBackgroundColorValue = colDef?.columnStyling?.cellBackgroundColor
-        const cellBackgroundColor = typeof cellBackgroundColorValue === 'function' 
-          ? cellBackgroundColorValue(row) 
-          : cellBackgroundColorValue
-        
+        const cellFontColor = colDef?.columnStyling?.fontColor
+        const cellBackgroundColor = colDef?.columnStyling?.cellBackgroundColor
         const paddingValue = colDef?.columnStyling?.cellPadding ?? customRowStyle?.cellPadding
         const paddingClass = paddingValue ? `p_${paddingValue}` : undefined
 

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

@@ -11,8 +11,8 @@
     <% if content.present? %>
       <% 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 }) %>
+      <%= 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 }) %>
+      <%= 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 }) %>
     <% end %>
   <% end %>
 <% end %>

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

@@ -33,10 +33,6 @@ module Playbook
                              default: false
       prop :row_styling, type: Playbook::Props::Array,
                          default: []
-      prop :inline_row_loading, type: Playbook::Props::Boolean,
-                                default: false
-      prop :persist_toggle_expansion_button, type: Playbook::Props::Boolean,
-                                             default: false
 
       def classname
         additional_classes = [

data/app/pb_kits/playbook/pb_advanced_table/advanced_table.test.jsx

@@ -920,39 +920,6 @@ test("columnStyling.backgroundColor works as excpected", () => {
   expect(firstEnrollmentCell).toHaveStyle({ backgroundColor: colors.error_subtle });
 });
 
-test("columnStyling.cellBackgroundColor works as expected with function", () => {
-  const styledColumnDefs = [
-    {
-      accessor: "year",
-      label: "Year",
-      cellAccessors: ["quarter", "month", "day"],
-    },
-    {
-      accessor: "newEnrollments",
-      label: "New Enrollments",
-      columnStyling:{
-        cellBackgroundColor: (row) => row.original.newEnrollments > 15 ? colors.success_subtle : colors.error_subtle,
-        fontColor: (row) => row.original.newEnrollments > 15 ? colors.success : colors.error,
-      },    
-    },
-    {
-      accessor: "scheduledMeetings",
-      label: "Scheduled Meetings",
-    },
-  ];
-
-  render(
-    <AdvancedTable
-        columnDefinitions={styledColumnDefs}
-        data={{ testid: testId }}
-        tableData={MOCK_DATA}
-    />
-  );
-
-  const firstEnrollmentCell = screen.getAllByText("20")[0].closest("td");
-  expect(firstEnrollmentCell).toHaveStyle({ backgroundColor: colors.success_subtle, color: colors.success });
-}); 
-
 test("columnStyling.headerBackgroundColor works as excpected", () => {
   const styledColumnDefs = [
     {

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

@@ -6,9 +6,9 @@ In the first Advanced Table in this code example, 2021 has an empty children arr
 This prop is set to `false` by default. 
 
 
-### persistToggleExpansionButton
+### persistToggleExpansion
 The `persistToggleExpansionButton` is a boolean prop that renders the toggle-all icon in the top left header cell for complex datasets with empty `children` arrays and advanced querying logic explained in the preceeding doc example. Your logic may require an additional query helper file to update data specifically from requerying via toggle all buttons.
 
 In the second and third Advanced Tables in this code example, all 3 rows have empty children arrays. The second Advanced Table demonstrates that the toggle all button does not render (prior to an initial row expansion) without `persistToggleExpansionButton` in place. The third Advanced Table shows the toggle all button due to `persistToggleExpansionButton`. 
 
-This prop is set to `false` by default and should only be used in conjunction with `inlineRowLoading`.
+This prop is set to false by default and should only be used in conjunction with `inlineRowLoading`.

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

@@ -29,7 +29,6 @@ examples:
   - advanced_table_background_control_rails: Column Styling Background Color
   - advanced_table_background_colors_rails: Column Styling Individual Cell Background Color
   - advanced_table_column_border_color_rails: Column Group Border Color
-  - advanced_table_inline_row_loading_rails: Inline Row Loading
 
 
   react:
@@ -78,7 +77,6 @@ examples:
   - advanced_table_column_styling: Column Styling
   - advanced_table_column_styling_column_headers: Column Styling with Multiple Headers
   - advanced_table_column_styling_background: Column Styling Background Color
-  - advanced_table_column_styling_background_custom: Column Styling Background Color (Custom)
   - advanced_table_column_styling_background_multi: Column Styling Background Color with Multiple Headers
   - advanced_table_padding_control: Padding Control using Column Styling
   - advanced_table_column_border_color: Column Group Border Color

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

@@ -47,5 +47,4 @@ export { default as AdvancedTableSortPerColumnForMultiColumn } from './_advanced
 export { default as AdvancedTablePaddingControl } from './_advanced_table_padding_control.jsx'
 export { default as AdvancedTablePaddingControlPerRow } from './_advanced_table_padding_control_per_row.jsx'
 export { default as AdvancedTableColumnStylingBackground } from './_advanced_table_column_styling_background.jsx'
-export { default as AdvancedTableColumnStylingBackgroundMulti } from './_advanced_table_column_styling_background_multi.jsx'
-export { default as AdvancedTableColumnStylingBackgroundCustom } from './_advanced_table_column_styling_background_custom.jsx'
+export { default as AdvancedTableColumnStylingBackgroundMulti } from './_advanced_table_column_styling_background_multi.jsx'

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

@@ -25,8 +25,6 @@ module Playbook
                              default: false
       prop :row_styling, type: Playbook::Props::Array,
                          default: []
-      prop :inline_row_loading, type: Playbook::Props::Boolean,
-                                default: false
 
       def flatten_columns(columns)
         columns.flat_map do |col|
@@ -71,24 +69,7 @@ module Playbook
                                   end
 
         # Additional class and data attributes needed for toggle logic
-        output << pb_rails("advanced_table/table_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 })
-
-        # Render inline loading row when inline_row_loading is enabled and row has empty children
-        if inline_row_loading
-          children = row_children_for(row)
-          if children.is_a?(::Array) && children.empty?
-            max_depth = cell_accessors_length(column_definitions)
-            if current_depth < max_depth
-              loading_row_data_attributes = {
-                advanced_table_content: "#{new_ancestor_ids.join('-')}-loading",
-                row_depth: current_depth + 1,
-                row_parent: "#{table_id}_#{row.object_id}",
-                inline_loading_row: true,
-              }
-              output << render_inline_loading_row(leaf_columns.length, current_depth, loading_row_data_attributes)
-            end
-          end
-        end
+        output << pb_rails("advanced_table/table_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 })
 
         if row[:children].present?
           row[:children].each do |child_row|
@@ -123,37 +104,6 @@ module Playbook
         ""
       end
 
-      # 3 helper methods for inline row loading
-      def render_inline_loading_row(column_count, depth, data_attributes)
-        padding_left = depth.zero? ? "0.5em" : "#{(depth + 1) * 2}em"
-
-        content_tag(:tr, class: "toggle-content inline-loading-row", data: data_attributes) do
-          content_tag(:td, colspan: column_count, style: "padding-left: #{padding_left}") do
-            pb_rails("loading_inline")
-          end
-        end
-      end
-
-      def row_children_for(row)
-        if row.respond_to?(:children)
-          row.children
-        elsif row.respond_to?(:[])
-          row[:children] || row["children"]
-        end
-      end
-
-      def cell_accessors_length(col_defs)
-        first_col = col_defs.first
-        return 0 unless first_col
-
-        accessors = if first_col.respond_to?(:cellAccessors)
-                      first_col.cellAccessors
-                    elsif first_col.respond_to?(:[])
-                      first_col[:cellAccessors] || first_col["cellAccessors"]
-                    end
-        accessors&.length || 0
-      end
-
     private
 
       def has_grouped_headers?

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

@@ -14,7 +14,7 @@
                   <%= object.render_select_all_checkbox %>
                 <% end %>
               <% end %>
-              <% if object.show_toggle_all_button? %>
+              <% if object.enable_toggle_expansion == "header" || object.enable_toggle_expansion == "all" %>
                 <% if loading %>
                   <div class="<%= object.loading ? 'loading-toggle-icon' : '' %>"></div>
                 <% else %>

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

@@ -19,12 +19,6 @@ module Playbook
                              default: false
       prop :show_actions_bar, type: Playbook::Props::Boolean,
                               default: true
-      prop :inline_row_loading, type: Playbook::Props::Boolean,
-                                default: false
-      prop :persist_toggle_expansion_button, type: Playbook::Props::Boolean,
-                                             default: false
-      prop :table_data, type: Playbook::Props::Array,
-                        default: []
 
       def classname
         additional_classes = []
@@ -141,20 +135,6 @@ module Playbook
         original_def.dig(:column_styling, :header_font_color)
       end
 
-      # Check if any row in the table has children
-      def has_any_sub_rows?
-        return false if table_data.blank?
-
-        table_data.any? { |row| row_has_children?(row) }
-      end
-
-      # Determines if the toggle-all button should be shown in the header based on inline row loading and persist toggle expansion button props
-      def show_toggle_all_button?
-        return false unless enable_toggle_expansion == "header" || enable_toggle_expansion == "all"
-
-        has_any_sub_rows? || (inline_row_loading && persist_toggle_expansion_button)
-      end
-
       # Check if header has custom background color
       def has_custom_header_background_color?(cell)
         cell[:header_background_color].present?
@@ -337,20 +317,6 @@ module Playbook
         end
         nil
       end
-
-      # 2 inline row loading helper methods
-      def row_has_children?(row)
-        children = row_children_for(row)
-        children.present? && children.is_a?(::Array) && !children.empty?
-      end
-
-      def row_children_for(row)
-        if row.respond_to?(:children)
-          row.children
-        elsif row.respond_to?(:[])
-          row[:children] || row["children"]
-        end
-      end
     end
   end
 end

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

@@ -32,7 +32,7 @@
                                     <%= object.render_row_checkbox %>
                                 <% end %>
                             <% end %>
-                            <% if object.show_expand_button? %>
+                            <% if object.row[:children].present? %>
                                 <button
                                     id="<%= "#{object.table_id}_#{object.row.object_id}" %>"
                                     class="gray-icon expand-toggle-icon"

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

@@ -33,8 +33,6 @@ module Playbook
                       default: false
       prop :immediate_parent_row_id, type: Playbook::Props::String,
                                      default: ""
-      prop :inline_row_loading, type: Playbook::Props::Boolean,
-                                default: false
 
       def data
         Hash(prop(:data)).merge(table_data_attributes)
@@ -188,23 +186,6 @@ module Playbook
         end
       end
 
-      # Determines if the row should show an expand button (true if row has children or inline_row_loading true and row has empty children array)
-      def show_expand_button?
-        children = row_children
-        return true if children.present?
-        return true if inline_row_loading && children.is_a?(::Array) && children.empty?
-
-        false
-      end
-
-      def row_children
-        if row.respond_to?(:children)
-          row.children
-        elsif row.respond_to?(:[])
-          row[:children] || row["children"]
-        end
-      end
-
     private
 
       def custom_renderer_value(column, index)

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

@@ -20,4 +20,3 @@ examples:
   - background_category: Category
   - background_size: Size
   - background_overlay: Overlay
-  - background_responsive: Responsive

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

@@ -7,4 +7,3 @@ export { default as BackgroundStatusSubtle } from './_background_status_subtle.j
 export { default as BackgroundCategory } from './_background_category.jsx'
 export { default as BackgroundSize } from './_background_size.jsx'
 export { default as BackgroundOverlay } from './_background_overlay.jsx'
-export { default as BackgroundResponsive } from './_background_responsive.jsx'

data/app/pb_kits/playbook/pb_date_picker/_date_picker.tsx

@@ -40,7 +40,7 @@ type DatePickerProps = {
   maxDate: string,
   minDate: string,
   name: string,
-  pickerId: string,
+  pickerId?: string,
   placeholder?: string,
   positionElement?: HTMLElement | null,
   scrollContainer?: string,
@@ -196,8 +196,6 @@ const DatePicker = (props: DatePickerProps): React.ReactElement => {
 
   const angleDown = getAllIcons()["angleDown"].icon as unknown as { [key: string]: SVGElement }
 
-  const errorId = error ? `${pickerId}-error` : undefined
-
   return (
     <div
         {...ariaProps}
@@ -213,18 +211,14 @@ const DatePicker = (props: DatePickerProps): React.ReactElement => {
       >
 
         {!hideLabel && (
-           <label htmlFor={pickerId}>
-            <Caption
-                className="pb_date_picker_kit_label"
-                text={label}
-            />
-          </label>
+          <Caption
+              className="pb_date_picker_kit_label"
+              text={label}
+          />
         )}
           <>
             <div className="date_picker_input_wrapper">
               <input
-                  aria-describedby={errorId}
-                  aria-invalid={!!error}
                   autoComplete="off"
                   className="date_picker_input"
                   disabled={disableInput}
@@ -238,9 +232,6 @@ const DatePicker = (props: DatePickerProps): React.ReactElement => {
 
               {error &&
                   <Body
-                      aria={{ atomic: "true", live: "polite" }}
-                      htmlOptions={{ role: "alert" }}
-                      id={errorId}
                       status="negative"
                       text={error}
                       variant={null}

data/app/pb_kits/playbook/pb_dialog/_dialog.scss

@@ -135,8 +135,6 @@
     position: sticky;
     top: 0;
     background-color: $white;
-    border-top-left-radius: $border_radius_md;
-    border-top-right-radius: $border_radius_md;
     z-index: $z_8;
   }
 
@@ -258,7 +256,7 @@
   }
   &.full_height_left {
     justify-content: flex-start;
-
+    
     .pb_dialog {
       border-radius: 0;
       height: 100%;
@@ -304,7 +302,7 @@
 
   &.full_height_center {
     justify-content: center;
-
+    
     .pb_dialog {
       border-radius: 0;
       height: 100%;
@@ -348,7 +346,7 @@
 
   &.full_height_right {
     justify-content: flex-end;
-
+    
     .pb_dialog {
       border-radius: 0;
       height: 100%;
@@ -419,7 +417,7 @@
       margin: unset !important;
       margin-right: auto !important;
     }
-
+    
     .pb_dialog {
       border-radius: 0;
       height: 100% !important;
@@ -465,7 +463,7 @@
 
   &.full_height_center {
     justify-content: center;
-
+    
     .pb_dialog {
       border-radius: 0;
       height: 100% !important;
@@ -512,7 +510,7 @@
       margin: unset !important;
       margin-left: auto !important;
     }
-
+    
     .pb_dialog {
       border-radius: 0;
       height: 100% !important;

data/app/pb_kits/playbook/pb_dropdown/_dropdown.scss

@@ -76,12 +76,6 @@
       z-index: $z_1;
       width: 100%;
 
-      &.constrain_height {
-        overflow-y: auto;
-        overflow-x: hidden;
-        max-height: 18em;
-      }
-
       .pb_dropdown_option,
       .pb_dropdown_option_list,
       .pb_dropdown_option_selected,

data/app/pb_kits/playbook/pb_dropdown/_dropdown.tsx

@@ -36,8 +36,6 @@ type DropdownProps = {
     blankSelection?: string;
     children?: React.ReactChild[] | React.ReactChild | React.ReactElement[];
     className?: string;
-    clearable?: boolean;
-    constrainHeight?: boolean;
     customQuickPickDates?: CustomQuickPickDates;
     formPillProps?: GenericObject;
     dark?: boolean;
@@ -51,7 +49,6 @@ type DropdownProps = {
     multiSelect?: boolean;
     onSelect?: (arg: GenericObject) => null;
     options?: GenericObject;
-    placeholder?: string;
     separators?: boolean;
     variant?: "default" | "subtle" | "quickpick";
     rangeEndsToday?: boolean;
@@ -77,8 +74,6 @@ let Dropdown = (props: DropdownProps, ref: any): React.ReactElement | null => {
         blankSelection = '',
         children,
         className,
-        clearable = true,
-        constrainHeight = false,
         customQuickPickDates,
         dark = false,
         data = {},
@@ -92,7 +87,6 @@ let Dropdown = (props: DropdownProps, ref: any): React.ReactElement | null => {
         formPillProps,
         onSelect,
         options,
-        placeholder,
         rangeEndsToday = false,
         controlsStartId,
         controlsEndId,
@@ -217,34 +211,6 @@ let Dropdown = (props: DropdownProps, ref: any): React.ReactElement | null => {
         }
     }, [isDropDownClosed]);
 
-    // Auto-position dropdown above/below based on available space
-    useEffect(() => {
-        if (!isDropDownClosed && dropdownContainerRef.current) {
-            const container = dropdownContainerRef.current;
-            const wrapper = container.closest('.dropdown_wrapper') as HTMLElement;
-            if (!wrapper) return;
-
-            const wrapperRect = wrapper.getBoundingClientRect();
-            const h = container.getBoundingClientRect().height || container.scrollHeight;
-            const spaceBelow = window.innerHeight - wrapperRect.bottom;
-            const spaceAbove = wrapperRect.top;
-
-            // If not enough space below but enough space above, position above
-            if (spaceBelow < h + 10 && spaceAbove >= h + 10) {
-                container.style.top = "auto";
-                container.style.bottom = "calc(100% + 5px)";
-                container.style.marginTop = "0";
-                container.style.marginBottom = "0";
-            } else {
-                // Default: position below
-                container.style.top = "";
-                container.style.bottom = "";
-                container.style.marginTop = "";
-                container.style.marginBottom = "";
-            }
-        }
-    }, [isDropDownClosed, dropdownContainerRef]);
-
 
     const handleChange = (e: React.ChangeEvent<HTMLInputElement>) => {
         setFilterItem(e.target.value);
@@ -409,7 +375,6 @@ let Dropdown = (props: DropdownProps, ref: any): React.ReactElement | null => {
                 value={{
                     activeStyle,
                     autocomplete,
-                    clearable,
                     dropdownContainerRef,
                     filteredOptions,
                     filterItem,
@@ -461,8 +426,8 @@ let Dropdown = (props: DropdownProps, ref: any): React.ReactElement | null => {
                         </>
                     ) : (
                         <>
-                            <DropdownTrigger placeholder={placeholder} />
-                            <DropdownContainer constrainHeight={constrainHeight}>
+                            <DropdownTrigger />
+                            <DropdownContainer>
                                 {optionsWithBlankSelection &&
                                     optionsWithBlankSelection?.map((option: GenericObject) => (
                                         <DropdownOption key={option.id}

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

@@ -21,10 +21,7 @@ examples:
   - dropdown_default_value: Default Value
   - dropdown_multi_select_with_default: Multi Select Default Value
   - dropdown_blank_selection: Blank Selection
-  - dropdown_with_placeholder: Placeholder
   - dropdown_separators_hidden: Separators Hidden
-  - dropdown_with_clearable: Clearable
-  - dropdown_with_constrain_height_rails: Constrain Height
   - dropdown_quickpick_rails: Quick Pick Variant
   - dropdown_quickpick_range_end_rails: Quick Pick Variant (Range Ends Today)
   - dropdown_quickpick_default_dates: Quick Pick Variant (Default Dates)
@@ -55,10 +52,7 @@ examples:
   - dropdown_default_value: Default Value
   - dropdown_multi_select_with_default: Multi Select Default Value
   - dropdown_blank_selection: Blank Selection
-  - dropdown_with_placeholder: Placeholder
   - dropdown_clear_selection: Clear Selection
-  - dropdown_with_clearable: Clearable
-  - dropdown_with_constrain_height: Constrain Height
   - dropdown_separators_hidden: Separators Hidden
   - dropdown_with_external_control: useDropdown Hook
   - dropdown_quickpick: Quick Pick Variant

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

@@ -11,7 +11,6 @@ export { default as DropdownSubcomponentStructure } from './_dropdown_subcompone
 export { default as DropdownError } from './_dropdown_error.jsx'
 export { default as DropdownDefaultValue } from './_dropdown_default_value.jsx'
 export { default as DropdownBlankSelection } from './_dropdown_blank_selection.jsx'
-export { default as DropdownWithPlaceholder } from './_dropdown_with_placeholder.jsx'
 export { default as DropdownClearSelection } from './_dropdown_clear_selection.jsx'
 export { default as DropdownSubtleVariant } from './_dropdown_subtle_variant.jsx'
 export { default as DropdownSeparatorsHidden } from './_dropdown_separators_hidden.jsx'
@@ -28,6 +27,4 @@ export { default as DropdownQuickpick } from './_dropdown_quickpick.jsx'
 export { default as DropdownQuickpickRangeEnd } from './_dropdown_quickpick_range_end.jsx'
 export { default as DropdownQuickpickDefaultDates } from './_dropdown_quickpick_default_dates.jsx'
 export { default as DropdownQuickpickWithDatePickers } from './_dropdown_quickpick_with_date_pickers.jsx'
-export { default as DropdownQuickpickCustom } from './_dropdown_quickpick_custom.jsx'
-export { default as DropdownWithClearable } from './_dropdown_with_clearable.jsx'
-export { default as DropdownWithConstrainHeight } from './_dropdown_with_constrain_height.jsx'
+export { default as DropdownQuickpickCustom } from './_dropdown_quickpick_custom.jsx'

data/app/pb_kits/playbook/pb_dropdown/dropdown.html.erb

@@ -18,8 +18,8 @@
             <%= content.presence %>
             <%= pb_rails("body", props: { status: "negative", text: object.error }) %>
         <% else %>
-            <%= pb_rails("dropdown/dropdown_trigger", props:{ autocomplete: object.autocomplete, multi_select:object.multi_select, placeholder: object.placeholder }) %>
-            <%= pb_rails("dropdown/dropdown_container", props: { searchbar: object.searchbar, constrain_height: object.constrain_height }) do %>
+            <%= pb_rails("dropdown/dropdown_trigger", props:{ autocomplete: object.autocomplete, multi_select:object.multi_select }) %>
+            <%= pb_rails("dropdown/dropdown_container", props: { searchbar: object.searchbar }) do %>
                 <% if options_with_blank.present? %>
                     <% options_with_blank.each do |option| %>
                         <%= pb_rails("dropdown/dropdown_option", props: {option: option}) %>

data/app/pb_kits/playbook/pb_dropdown/dropdown.rb

@@ -36,8 +36,6 @@ module Playbook
                              default: ""
       prop :controls_start_id, type: Playbook::Props::String,
                                default: ""
-      prop :clearable, type: Playbook::Props::Boolean,
-                       default: true
       prop :start_date_id, type: Playbook::Props::String,
                            default: "start_date_id"
       prop :start_date_name, type: Playbook::Props::String,
@@ -46,16 +44,12 @@ module Playbook
                          default: "end_date_id"
       prop :end_date_name, type: Playbook::Props::String,
                            default: "end_date_name"
-      prop :placeholder, type: Playbook::Props::String
-      prop :constrain_height, type: Playbook::Props::Boolean,
-                              default: false
 
       def data
         Hash(prop(:data)).merge(
           pb_dropdown: true,
           pb_dropdown_multi_select: multi_select,
           pb_dropdown_variant: variant,
-          pb_dropdown_clearable: clearable,
           form_pill_props: form_pill_props.to_json,
           start_date_id: variant == "quickpick" ? start_date_id : nil,
           end_date_id: variant == "quickpick" ? end_date_id : nil,