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

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a5780d8a980b27fef50be2bd9f824684732373c72b707d5af5f8071c627b878a
-  data.tar.gz: b304d2a4a3bd2a77b73ba7e95b70f5aff9bd6b1f4b78ed99a494712666ce6a1a
+  metadata.gz: ec3832a5a9d172af005bea73161b414b33cce3a4b9351680a75be21567f70f1f
+  data.tar.gz: 9b88540268a3c696e5e810642e48ceeededa245b3ce5a3e46e23562b85e0a339
 SHA512:
-  metadata.gz: c8133ed39e3328aa2e76508e31c51e7ff0c81103c99f8c2324367333e2168ea08610f59fdff5d1bef97d3372637ee29150ee3d6541d83c3cea7417414bed5481
-  data.tar.gz: f5488d8e0b060a56590d3c390d832a1760d9bd7e1b97a02afc0958c9252ae67f9667f2856d2ff70d3cdfc8fe71658aa2135bccef62f154e68a1db7f2a20fa807
+  metadata.gz: ad0694a212c9b5924b651c30038152649a9f779f11ea81ad5970315aefcb3a14534654c366049d8ba1d725180c907a9611c3b7c4840ec4cd7f971a11599d135f
+  data.tar.gz: 7c69619d23491fde3af8952c6e899afffff9c65df6a42d81c938fc6fd37c08e90706bb7eb36f9d9dc6267c83826851424964c5adbc4ac4bd67a04e4294947b02

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/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:

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_card/docs/_card_light.html.erb

@@ -1,36 +1,4 @@
-<%= pb_rails("flex", props: {
-  gap: "md",
-  orientation: "column"
-}) do %>
-
-  <%= pb_rails("caption", props: { text: "flex direction responsive: row on md + xl, column as default" }) %>
-  <%= pb_rails("card", props: {
-    flex_direction: { default: "column", md: "row", xl:"row" }
-  })  do %>
-      <%= pb_rails("body", props: { text: "Item 1", color: "default" }) %>
-    <%= pb_rails("body", props: { text: "Item 2", color: "default" }) %>
-    <%= pb_rails("body", props: { text: "Item 3", color: "default" }) %>
-  <% end %>
-
-  <%= pb_rails("caption", props: { text: "align items responsive: start on default, center on md, end on lg" }) %>
-  <%= pb_rails("card", props: {
-    display: "flex",
-    flex_direction: "row",
-    align_items: { default: "start", md: "center", lg: "end" },
-  })  do %>
-    <%= pb_rails("card", props: {height:"xs" }) do %> Card 1 <% end %>
-    <%= pb_rails("card", props: { height:"sm" }) do %> Card 2 <% end %>
-    <%= pb_rails("card", props: { height:"xs" }) do %> Card 3 <% end %>
-  <% end %>
-
-<%= pb_rails("caption", props: { text: "Text Align responsive: left on default, center on md, right on lg" }) %>
-  <%= pb_rails("card", props: {
-    text_align: { default: "left", md: "center", lg: "right" },
-    width: "100%"
-  })  do %>
-    <%= pb_rails("body", props: { }) do %> text 1 <% end %>
-    <%= pb_rails("body", props: { }) do %> text 2 <% end %>
-    <%= pb_rails("body", props: { }) do %> text 3 <% end %>
-  <% end %>
+<%= pb_rails("card", props: {
+})  do %>
+  Card content
 <% end %>
-

data/app/pb_kits/playbook/pb_form/docs/_form_with_required_indicator.html.erb

@@ -1,8 +1,6 @@
 <%= pb_form_with(scope: :example, url: "", method: :get, validate: true) do |form| %>
   <%= form.text_field :example_text_field, props: { label: true, required: true, required_indicator: true } %>
   <%= form.text_field :example_text_field_2, props: { label: "Text Field Custom Label", required: true, required_indicator: true } %>
-  <%= form.text_area :example_text_area, props: { label: true, required: true, required_indicator: true } %>
-  <%= form.text_area :example_text_area_2, props: { label: "Textarea Custom Label", required: true, required_indicator: true } %>
   <%= form.email_field :example_email_field, props: { label: true, required: true, required_indicator: true } %>
   <%= form.number_field :example_number_field, props: { label: true, required: true, required_indicator: true } %>
   <%= form.search_field :example_search_field, props: { label: true, required: true, required_indicator: true } %>
@@ -13,4 +11,4 @@
     <%= action.submit %>
     <%= action.button props: { type: "reset", text: "Cancel", variant: "secondary" } %>
   <% end %>
-<% end %>
+<% end %>

data/app/pb_kits/playbook/pb_icon/icon.rb

@@ -109,18 +109,13 @@ module Playbook
 
         svg["class"] = %w[pb_custom_icon svg-inline--fa].concat([object.custom_icon_classname]).join(" ")
         svg["id"] = object.id
+        svg["data"] = object.data
         svg["height"] = "auto"
         svg["width"] = "auto"
         svg["tabindex"] = object.tabindex
         fill_color = object.color || "currentColor"
         doc.at_css("path")["fill"] = fill_color
 
-        if object.data.present?
-          object.data.each do |key, value|
-            svg["data-#{key}"] = value
-          end
-        end
-
         if object.aria.present?
           object.aria.each do |key, value|
             k = key.to_s

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

@@ -1,39 +1,37 @@
 examples:
   rails:
-    - multi_level_select_default: Default
-    - multi_level_select_single: Single Select
-    - multi_level_select_single_children_only: Single Select w/ Hidden Radios
-    - multi_level_select_return_all_selected: Return All Selected
-    - multi_level_select_input_display: With Input Display None
-    - multi_level_select_selected_ids: Selected Ids
-    - multi_level_select_with_form: With Form
-    - multi_level_select_color: With Pills (Custom Color)
-    - multi_level_select_reset: Reset Selection
-    - multi_level_select_label: With Label
-    - multi_level_select_error: Error
-    - multi_level_select_disabled: Disabled Input
-    - multi_level_select_disabled_options_default: Disabled Options (Default)
-    - multi_level_select_disabled_options: Disabled Options (Return All Selected)
-    - multi_level_select_disabled_options_parent_default: Disabled Parent Option (Default)
-    - multi_level_select_disabled_options_parent: Disabled Parent Option (Return All Selected)
-    - multi_level_select_single_disabled: Disabled Options (Single Select)
+  - multi_level_select_default: Default
+  - multi_level_select_single: Single Select
+  - multi_level_select_single_children_only: Single Select w/ Hidden Radios
+  - multi_level_select_return_all_selected: Return All Selected
+  - multi_level_select_selected_ids: Selected Ids
+  - multi_level_select_with_form: With Form
+  - multi_level_select_color: With Pills (Custom Color)
+  - multi_level_select_reset: Reset Selection
+  - multi_level_select_label: With Label
+  - multi_level_select_error: Error
+  - multi_level_select_disabled: Disabled Input
+  - multi_level_select_disabled_options_default: Disabled Options (Default)
+  - multi_level_select_disabled_options: Disabled Options (Return All Selected)
+  - multi_level_select_disabled_options_parent_default: Disabled Parent Option (Default)
+  - multi_level_select_disabled_options_parent: Disabled Parent Option (Return All Selected)
+  - multi_level_select_single_disabled: Disabled Options (Single Select)
 
   react:
-    - multi_level_select_default: Default
-    - multi_level_select_react_hook: React Hook
-    - multi_level_select_single: Single Select
-    - multi_level_select_single_children_only: Single Select w/ Hidden Radios
-    - multi_level_select_return_all_selected: Return All Selected
-    - multi_level_select_input_display: With Input Display None
-    - multi_level_select_selected_ids_react: Selected Ids
-    - multi_level_select_color: With Pills (Custom Color)
-    - multi_level_select_with_children: Checkboxes With Children
-    - multi_level_select_with_children_with_radios: Single Select With Children
-    - multi_level_select_label: With Label
-    - multi_level_select_error: Error
-    - multi_level_select_disabled: Disabled Input
-    - multi_level_select_disabled_options_default: Disabled Options (Default)
-    - multi_level_select_disabled_options: Disabled Options (Return All Selected)
-    - multi_level_select_disabled_options_parent_default: Disabled Parent Option (Default)
-    - multi_level_select_disabled_options_parent: Disabled Parent Option (Return All Selected)
-    - multi_level_select_single_disabled: Disabled Options (Single Select)
+  - multi_level_select_default: Default
+  - multi_level_select_react_hook: React Hook
+  - multi_level_select_single: Single Select
+  - multi_level_select_single_children_only: Single Select w/ Hidden Radios
+  - multi_level_select_return_all_selected: Return All Selected
+  - multi_level_select_selected_ids_react: Selected Ids
+  - multi_level_select_color: With Pills (Custom Color)
+  - multi_level_select_with_children: Checkboxes With Children
+  - multi_level_select_with_children_with_radios: Single Select With Children
+  - multi_level_select_label: With Label
+  - multi_level_select_error: Error
+  - multi_level_select_disabled: Disabled Input
+  - multi_level_select_disabled_options_default: Disabled Options (Default)
+  - multi_level_select_disabled_options: Disabled Options (Return All Selected)
+  - multi_level_select_disabled_options_parent_default: Disabled Parent Option (Default)
+  - multi_level_select_disabled_options_parent: Disabled Parent Option (Return All Selected)
+  - multi_level_select_single_disabled: Disabled Options (Single Select)

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

@@ -4,7 +4,6 @@ export { default as MultiLevelSelectSingleChildrenOnly } from './_multi_level_se
 export { default as MultiLevelSelectReturnAllSelected } from './_multi_level_select_return_all_selected.jsx'
 export { default as MultiLevelSelectSelectedIdsReact } from "./_multi_level_select_selected_ids_react.jsx"
 export { default as MultiLevelSelectColor } from './_multi_level_select_color.jsx'
-export { default as MultiLevelSelectInputDisplay } from './_multi_level_select_input_display.jsx'
 export { default as MultiLevelSelectWithChildren } from './_multi_level_select_with_children.jsx'
 export { default as MultiLevelSelectWithChildrenWithRadios } from './_multi_level_select_with_children_with_radios.jsx'
 export { default as MultiLevelSelectDisabled } from './_multi_level_select_disabled.jsx'

data/app/pb_kits/playbook/pb_multiple_users/docs/_multiple_users_with_tooltip.jsx

@@ -38,5 +38,5 @@ const MultipleUsersWithTooltip = (props) => {
         </div>
     )
 }
-
+``
 export default MultipleUsersWithTooltip

data/app/pb_kits/playbook/pb_multiple_users/docs/_multiple_users_with_tooltip.md

@@ -1 +1 @@
-Use the `withTooltip` / `with_tooltip` boolean prop to enable setting user-specific tooltip content via the `tooltip` property in the users array.
+Use the `withTooltip` boolean prop to enable setting user-specific tooltip content via the `tooltip` property in the users array.

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

@@ -4,7 +4,6 @@ examples:
   - multiple_users_default: Default
   - multiple_users_reverse: Reverse
   - multiple_users_size: Size
-  - multiple_users_with_tooltip: With Tooltip
 
 
   react:

data/app/pb_kits/playbook/pb_multiple_users/multiple_users.html.erb

@@ -1,55 +1,11 @@
 <%= pb_content_tag do %>
-    <% if object.with_tooltip %>
-      <% object.users.take(object.display_count).each_with_index do |user, index| %>
-        <% user_id = "multiple-users-#{object.object_id}-user-#{index}" %>
-        <div id="<%= user_id %>" class="pb_multiple_users_item">
-          <%= pb_rails("avatar", props: user.except(:tooltip).merge({size: object.size, dark: object.dark})) %>
-        </div>
-      <% end %>
-
-      <% if object.more_than_four %>
-        <% badge_id = "multiple-users-#{object.object_id}-badge" %>
-        <div id="<%= badge_id %>" class="pb_multiple_users_item multiple_users_badge_<%= object.size %><%= object.dark ? " dark" : "" %>">
-          <%= "+#{object.users.count - object.display_count}" %>
-        </div>
-      <% end %>
-
-      <% object.users.take(object.display_count).each_with_index do |user, index| %>
-        <% if user[:tooltip].present? %>
-          <% user_id = "multiple-users-#{object.object_id}-user-#{index}" %>
-          <%= pb_rails("tooltip", props: {
-            trigger_element_selector: "##{user_id}",
-            tooltip_id: "#{user_id}-tooltip",
-            position: "top",
-            z_index: "10"
-          }) do %>
-            <%= user[:tooltip] %>
-          <% end %>
-        <% end %>
-      <% end %>
-
-      <% if object.more_than_four %>
-        <% badge_id = "multiple-users-#{object.object_id}-badge" %>
-        <%= pb_rails("tooltip", props: {
-          trigger_element_selector: "##{badge_id}",
-          tooltip_id: "#{badge_id}-tooltip",
-          position: "top",
-          z_index: "10"
-        }) do %>
-          <% object.users.drop(object.display_count).each do |user| %>
-            <div><%= user[:tooltip] || user[:name] %></div>
-          <% end %>
-        <% end %>
-      <% end %>
-    <% else %>
-      <% object.users.take(object.display_count).each do |user| %>
-        <%= pb_rails("avatar", props: user.merge({size: object.size, classname: "pb_multiple_users_item", dark: object.dark})) %>
-      <% end %>
+    <% object.users.take(object.display_count).each do |user| %>
+      <%= pb_rails("avatar", props: user.merge({size: object.size, classname: "pb_multiple_users_item", dark: object.dark}) ) %>
+    <% end %>
 
-      <% if object.more_than_four %>
-        <div class="pb_multiple_users_item multiple_users_badge_<%= object.size %><%= object.dark ? " dark" : "" %>">
-          <%= "+#{object.users.count - object.display_count}" %>
-        </div>
-      <% end %>
+    <% if object.more_than_four %>
+      <div class="pb_multiple_users_item multiple_users_badge_<%= object.size %><%= object.dark ? " dark" : "" %>">
+        <%= "+#{object.users.count - object.display_count}" %>
+      </div>
     <% end %>
 <% end %>

data/app/pb_kits/playbook/pb_multiple_users/multiple_users.rb

@@ -8,7 +8,6 @@ module Playbook
                   values: %w[xxs xs],
                   default: "xs"
       prop :users, type: Playbook::Props::HashArray, required: true
-      prop :with_tooltip, type: Playbook::Props::Boolean, default: false
 
       def more_than_four
         users.count > 4

data/app/pb_kits/playbook/pb_rich_text_editor/_rich_text_editor.tsx

@@ -4,8 +4,6 @@ import { TrixEditor } from 'react-trix'
 
 import inlineFocus from './inlineFocus'
 import useFocus from './useFocus'
-import Caption from '../pb_caption/_caption'
-import colors from '../tokens/exports/_colors.module.scss'
 import { globalProps, GlobalProps } from '../utilities/globalProps'
 import { buildAriaProps, buildDataProps, noop, buildHtmlProps } from '../utilities/props'
 
@@ -42,14 +40,12 @@ type RichTextEditorProps = {
   inputOptions?: { [key: string]: string | number | boolean | (() => void) },
   id?: string,
   inline?: boolean,
-  label?: string,
   extensions?: { [key: string]: string }[],
   name?: string,
   onChange: (html: string, text: string) => void,
   placeholder?: string,
   inputHeight?: "sm" | "md" | "lg",
   inputMinHeight?: "sm" | "md" | "lg",
-  requiredIndicator?: boolean,
   simple?: boolean,
   sticky?: boolean,
   template: string,
@@ -68,7 +64,6 @@ const RichTextEditor = (props: RichTextEditorProps): React.ReactElement => {
     data = {},
     focus = false,
     htmlOptions = {},
-    id,
     inputOptions = {},
     inline = false,
     extensions,
@@ -81,9 +76,7 @@ const RichTextEditor = (props: RichTextEditorProps): React.ReactElement => {
     sticky = false,
     template = '',
     value = '',
-    maxWidth = "md",
-    requiredIndicator = false,
-    label,
+    maxWidth = "md"
   } = props
 
   const ariaProps = buildAriaProps(aria),
@@ -93,7 +86,7 @@ const RichTextEditor = (props: RichTextEditorProps): React.ReactElement => {
     containerRef = useRef<HTMLDivElement>(null)
 
   const htmlProps = buildHtmlProps(htmlOptions)
-
+  
   const handleOnEditorReady = (editorInstance: Editor) => {
     setEditor(editorInstance)
     setTimeout(() => {
@@ -101,7 +94,7 @@ const RichTextEditor = (props: RichTextEditorProps): React.ReactElement => {
       if (oldId) {
         const hiddenInput = document.getElementById(oldId)
         if (hiddenInput) {
-          const newId = (inputOptions.id as string) || oldId
+          const newId = (inputOptions.id as string) || oldId  
           hiddenInput.id = newId
           editorInstance.element.setAttribute('input', newId)
 
@@ -126,7 +119,7 @@ const RichTextEditor = (props: RichTextEditorProps): React.ReactElement => {
       // set button attributes
       inlineCodeButton.dataset.trixAttribute = 'inlineCode'
       blockCodeButton.insertAdjacentElement('afterend', inlineCodeButton)
-    }
+    } 
 
     if (toolbarBottom) editor.element.after(toolbarElement)
 
@@ -154,7 +147,7 @@ const RichTextEditor = (props: RichTextEditorProps): React.ReactElement => {
     if (!advancedEditor || !focus) return
 
     const handleFocus = () => setShowToolbarOnFocus(true)
-
+    
     const handleClickOutside = (event: Event) => {
       if (isClickInContainer(event) || isClickInPopover(event)) return
       setShowToolbarOnFocus(false)
@@ -222,29 +215,9 @@ const RichTextEditor = (props: RichTextEditorProps): React.ReactElement => {
         className={css}
         ref={focus ? containerRef : undefined}
     >
-    {label && (
-      <label  htmlFor={id}>
-        {
-          requiredIndicator ? (
-            <Caption className="pb_text_input_kit_label"
-                marginBottom="xs"
-            >
-              {label} <span style={{ color: `${colors.error}` }}>*</span>
-            </Caption>
-          ) : (
-            <Caption
-                className="pb_text_input_kit_label"
-                marginBottom="xs"
-                text={label}
-            />
-          )
-        }
-
-      </label>
-    )}
       {
         advancedEditor ? (
-          <div
+          <div 
               className={classnames(
                 "pb_rich_text_editor_advanced_container",
                 { [`input_height_${inputHeight}`]: !!inputHeight,[`input_min_height_${inputMinHeight}`]: !!inputMinHeight ,["toolbar-active"]: shouldShowToolbar }

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

@@ -9,7 +9,6 @@ examples:
   - rich_text_editor_templates: Templates
   # - rich_text_editor_toolbar_bottom: Toolbar Bottom
   - rich_text_editor_inline: Inline
-  - rich_text_editor_required_indicator: Required Indicator
   - rich_text_editor_preview: Preview
 
   react:
@@ -32,7 +31,5 @@ examples:
   - rich_text_editor_advanced_inline: Advanced (Inline)
   - rich_text_editor_advanced_height: Advanced Height
   - rich_text_editor_advanced_min_height: Advanced Min Height
-  - rich_text_editor_required_indicator: Required Indicator
-  - rich_text_editor_advanced_required_indicator: Advanced Required Indicator
   - rich_text_editor_preview: Preview
   - rich_text_editor_advanced_preview: Advanced Preview

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

@@ -19,5 +19,3 @@ export { default as RichTextEditorAdvancedSticky } from './_rich_text_editor_adv
 export { default as RichTextEditorAdvancedInline } from './_rich_text_editor_advanced_inline.jsx'
 export { default as RichTextEditorAdvancedHeight } from './_rich_text_editor_advanced_height.jsx'
 export { default as RichTextEditorAdvancedMinHeight } from './_rich_text_editor_advanced_min_height.jsx'
-export { default as RichTextEditorRequiredIndicator } from './_rich_text_editor_required_indicator.jsx'
-export { default as RichTextEditorAdvancedRequiredIndicator } from './_rich_text_editor_advanced_required_indicator.jsx'