playbook_ui 14.9.0.pre.alpha.play1742globalheightfixes4766 → 14.9.0.pre.rc.0

data/lib/playbook/classnames.rb

@@ -13,7 +13,6 @@ module Playbook
         prop(:classname),
         spacing_props,
         dark_props,
-        width_props,
         min_width_props,
         max_width_props,
         z_index_props,
@@ -44,9 +43,6 @@ module Playbook
         right_props,
         bottom_props,
         vertical_align_props,
-        height_props,
-        min_height_props,
-        max_height_props,
       ].compact.join(" ")
     end
 

data/lib/playbook/forms/builder/typeahead_field.rb

@@ -5,19 +5,6 @@ module Playbook
     class Builder
       def typeahead(name, _options = {}, _html_options = {}, props: {})
         props[:name] = name
-
-        input_id = "#{name}_input"
-        props[:input_options] ||= {}
-        props[:input_options][:id] ||= input_id
-
-        props[:label] = @template.label(@object_name, name, for: props[:input_options][:id]) if props[:label] == true
-
-        if props[:validation]
-          validation_message = props[:validation][:message]
-          props[:input_options][:data] ||= {}
-          props[:input_options][:data][:validation_message] = validation_message
-        end
-
         @template.pb_rails("typeahead", props: props)
       end
     end

data/lib/playbook/kit_base.rb

@@ -30,9 +30,6 @@ require "playbook/top"
 require "playbook/right"
 require "playbook/bottom"
 require "playbook/vertical_align"
-require "playbook/height"
-require "playbook/min_height"
-require "playbook/max_height"
 
 module Playbook
   include ActionView::Helpers
@@ -70,9 +67,6 @@ module Playbook
     include Playbook::Right
     include Playbook::Bottom
     include Playbook::VerticalAlign
-    include Playbook::Height
-    include Playbook::MinHeight
-    include Playbook::MaxHeight
 
     prop :id
     prop :data, type: Playbook::Props::HashProp, default: {}
@@ -172,17 +166,8 @@ module Playbook
     end
 
     def dynamic_inline_props
-      styles = global_inline_props.map { |key, value| "#{key.to_s.gsub('_', '-')}: #{value}" if inline_validator(key, value) }.compact
+      styles = global_inline_props.map { |key, value| "#{key.to_s.gsub('_', '-')}: #{value}" if value.present? }.compact
       styles.join("; ").presence
     end
-
-    def inline_validator(key, value)
-      return false if value.nil?
-      return false if height_values.include?(value) && key == :height
-      return false if min_height_values.include?(value) && key == :min_height
-      return false if max_height_values.include?(value) && key == :max_height
-
-      true
-    end
   end
 end

data/lib/playbook/spacing.rb

@@ -12,7 +12,6 @@ module Playbook
       base.prop :margin_y
       base.prop :max_width
       base.prop :min_width
-      base.prop :width
       base.prop :padding
       base.prop :padding_bottom
       base.prop :padding_left
@@ -34,12 +33,6 @@ module Playbook
       }
     end
 
-    def width_options
-      {
-        width: "w",
-      }
-    end
-
     def max_width_values
       %w[0% xs sm md lg xl xxl 0 none 100%]
     end
@@ -48,10 +41,6 @@ module Playbook
       %w[0% xs sm md lg xl xxl 0 none 100%]
     end
 
-    def width_values
-      %w[0% xs sm md lg xl xxl 0 none 100%]
-    end
-
     def spacing_options
       {
         margin: "m",
@@ -136,15 +125,5 @@ module Playbook
         "max_width_#{filter_classname(width_value)}" if max_width_values.include? width_value
       end.compact.join(" ")
     end
-
-    def width_props
-      selected_w_props = width_options.keys.select { |sk| try(sk) }
-      return nil unless selected_w_props.present?
-
-      selected_w_props.map do |k|
-        width_value = send(k)
-        "width_#{filter_classname(width_value)}" if width_values.include? width_value
-      end.compact.join(" ")
-    end
   end
 end

data/lib/playbook/version.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
 module Playbook
-  PREVIOUS_VERSION = "14.9.0"
-  VERSION = "14.9.0.pre.alpha.play1742globalheightfixes4766"
+  PREVIOUS_VERSION = "14.8.0"
+  VERSION = "14.9.0.pre.rc.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: playbook_ui
 version: !ruby/object:Gem::Version
-  version: 14.9.0.pre.alpha.play1742globalheightfixes4766
+  version: 14.9.0.pre.rc.0
 platform: ruby
 authors:
 - Power UX
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-12-04 00:00:00.000000000 Z
+date: 2024-11-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack
@@ -270,8 +270,6 @@ files:
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_collapsible_trail.md
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_custom_cell.jsx
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_custom_cell.md
-- app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_custom_cell_rails.html.erb
-- app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_custom_cell_rails.md
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_default.jsx
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_default.md
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_expanded_control.jsx
@@ -280,10 +278,6 @@ files:
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_inline_row_loading.md
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_loading.jsx
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_loading.md
-- app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_pagination.jsx
-- app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_pagination.md
-- app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_pagination_with_props.jsx
-- app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_pagination_with_props.md
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_responsive.jsx
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_responsive.md
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_sort.jsx
@@ -298,7 +292,6 @@ files:
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_table_props.md
 - app/pb_kits/playbook/pb_advanced_table/docs/_mock_data_inline_loading.js
 - app/pb_kits/playbook/pb_advanced_table/docs/advanced_table_mock_data.json
-- app/pb_kits/playbook/pb_advanced_table/docs/advanced_table_pagination_mock_data.json
 - app/pb_kits/playbook/pb_advanced_table/docs/example.yml
 - app/pb_kits/playbook/pb_advanced_table/docs/index.js
 - app/pb_kits/playbook/pb_advanced_table/index.js
@@ -774,8 +767,6 @@ files:
 - app/pb_kits/playbook/pb_currency/docs/_currency_comma_separator.md
 - app/pb_kits/playbook/pb_currency/docs/_currency_matching_decimals.html.erb
 - app/pb_kits/playbook/pb_currency/docs/_currency_matching_decimals.jsx
-- app/pb_kits/playbook/pb_currency/docs/_currency_negative.html.erb
-- app/pb_kits/playbook/pb_currency/docs/_currency_negative.jsx
 - app/pb_kits/playbook/pb_currency/docs/_currency_no_symbol.html.erb
 - app/pb_kits/playbook/pb_currency/docs/_currency_no_symbol.jsx
 - app/pb_kits/playbook/pb_currency/docs/_currency_props_swift.md
@@ -1143,8 +1134,6 @@ files:
 - app/pb_kits/playbook/pb_drawer/docs/_drawer_breakpoints.jsx
 - app/pb_kits/playbook/pb_drawer/docs/_drawer_default.html.erb
 - app/pb_kits/playbook/pb_drawer/docs/_drawer_default.jsx
-- app/pb_kits/playbook/pb_drawer/docs/_drawer_menu.jsx
-- app/pb_kits/playbook/pb_drawer/docs/_drawer_menu.md
 - app/pb_kits/playbook/pb_drawer/docs/_drawer_overlay.jsx
 - app/pb_kits/playbook/pb_drawer/docs/_drawer_sizes.jsx
 - app/pb_kits/playbook/pb_drawer/docs/example.yml
@@ -2617,8 +2606,6 @@ files:
 - app/pb_kits/playbook/pb_table/docs/_table_sticky.html.erb
 - app/pb_kits/playbook/pb_table/docs/_table_sticky.jsx
 - app/pb_kits/playbook/pb_table/docs/_table_sticky.md
-- app/pb_kits/playbook/pb_table/docs/_table_sticky_left_columns.jsx
-- app/pb_kits/playbook/pb_table/docs/_table_sticky_left_columns_react.md
 - app/pb_kits/playbook/pb_table/docs/_table_striped.html.erb
 - app/pb_kits/playbook/pb_table/docs/_table_striped.jsx
 - app/pb_kits/playbook/pb_table/docs/_table_striped.md
@@ -2652,10 +2639,8 @@ files:
 - app/pb_kits/playbook/pb_table/styles/_mobile_collapse.scss
 - app/pb_kits/playbook/pb_table/styles/_outer_padding.scss
 - app/pb_kits/playbook/pb_table/styles/_reset.scss
-- app/pb_kits/playbook/pb_table/styles/_scroll.scss
 - app/pb_kits/playbook/pb_table/styles/_side_highlight.scss
 - app/pb_kits/playbook/pb_table/styles/_single-line.scss
-- app/pb_kits/playbook/pb_table/styles/_sticky_columns.scss
 - app/pb_kits/playbook/pb_table/styles/_sticky_header.scss
 - app/pb_kits/playbook/pb_table/styles/_striped.scss
 - app/pb_kits/playbook/pb_table/styles/_structure.scss
@@ -3102,7 +3087,6 @@ files:
 - app/pb_kits/playbook/tokens/_colors.scss
 - app/pb_kits/playbook/tokens/_container.scss
 - app/pb_kits/playbook/tokens/_display.scss
-- app/pb_kits/playbook/tokens/_height.scss
 - app/pb_kits/playbook/tokens/_line_height.scss
 - app/pb_kits/playbook/tokens/_opacity.scss
 - app/pb_kits/playbook/tokens/_overflow.scss
@@ -3118,7 +3102,6 @@ files:
 - app/pb_kits/playbook/tokens/_vertical_align.scss
 - app/pb_kits/playbook/tokens/exports/_border_radius.module.scss
 - app/pb_kits/playbook/tokens/exports/_colors.module.scss
-- app/pb_kits/playbook/tokens/exports/_height.module.scss
 - app/pb_kits/playbook/tokens/exports/_line_height.module.scss
 - app/pb_kits/playbook/tokens/exports/_opacity.module.scss
 - app/pb_kits/playbook/tokens/exports/_positioning.module.scss
@@ -3136,7 +3119,6 @@ files:
 - app/pb_kits/playbook/utilities/_display.scss
 - app/pb_kits/playbook/utilities/_flexbox.scss
 - app/pb_kits/playbook/utilities/_focus.scss
-- app/pb_kits/playbook/utilities/_height.scss
 - app/pb_kits/playbook/utilities/_hover.scss
 - app/pb_kits/playbook/utilities/_line_height.scss
 - app/pb_kits/playbook/utilities/_max_width.scss
@@ -3150,7 +3132,6 @@ files:
 - app/pb_kits/playbook/utilities/_text_align.scss
 - app/pb_kits/playbook/utilities/_truncate.scss
 - app/pb_kits/playbook/utilities/_vertical_align.scss
-- app/pb_kits/playbook/utilities/_width.scss
 - app/pb_kits/playbook/utilities/flexbox_global_props/_align_content.scss
 - app/pb_kits/playbook/utilities/flexbox_global_props/_align_items.scss
 - app/pb_kits/playbook/utilities/flexbox_global_props/_align_self.scss
@@ -3189,11 +3170,11 @@ files:
 - app/pb_kits/playbook/utilities/test/globalProps/truncate.test.js
 - app/pb_kits/playbook/utilities/text.ts
 - app/pb_kits/playbook/utilities/validEmojiChecker.ts
-- dist/chunks/_typeahead-B8fkIeXA.js
-- dist/chunks/_weekday_stacked-DjRTXEi-.js
+- dist/chunks/_typeahead-D0PihN_3.js
+- dist/chunks/_weekday_stacked-uMIX8f-A.js
 - dist/chunks/lazysizes-B7xYodB-.js
-- dist/chunks/lib-SyD3buPZ.js
-- dist/chunks/pb_form_validation-Dt8UJgrJ.js
+- dist/chunks/lib-BC6ESsxG.js
+- dist/chunks/pb_form_validation-B_Z9rEbg.js
 - dist/chunks/vendor.js
 - dist/menu.yml
 - dist/playbook-doc.js
@@ -3230,7 +3211,6 @@ files:
 - lib/playbook/forms/builder/select_field.rb
 - lib/playbook/forms/builder/star_rating_field.rb
 - lib/playbook/forms/builder/typeahead_field.rb
-- lib/playbook/height.rb
 - lib/playbook/hover.rb
 - lib/playbook/justify_content.rb
 - lib/playbook/justify_self.rb
@@ -3238,8 +3218,6 @@ files:
 - lib/playbook/kit_resolver.rb
 - lib/playbook/left.rb
 - lib/playbook/line_height.rb
-- lib/playbook/max_height.rb
-- lib/playbook/min_height.rb
 - lib/playbook/number_spacing.rb
 - lib/playbook/order.rb
 - lib/playbook/overflow.rb

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

@@ -1,53 +0,0 @@
-<%
-  column_definitions = [
-    {
-      accessor: "year",
-      label: "Year",
-      cellAccessors: ["quarter", "month", "day"],
-      custom_renderer: ->(row, value) {
-        capture do
-          pb_rails("flex") do
-            pb_rails("title", props: { text: value, size: 4 }) +
-            pb_rails("badge", props: { dark: true, margin_left: "xxs", text: row[:newEnrollments].to_i > 20 ? "High" : "Low", variant: "neutral" })
-          end
-        end
-      }
-    },
-    {
-      accessor: "newEnrollments",
-      label: "New Enrollments",
-      custom_renderer: ->(row, value) { pb_rails("pill", props: { text: value, variant: "success" }) }
-    },
-    {
-      accessor: "scheduledMeetings",
-      label: "Scheduled Meetings",
-      custom_renderer: ->(row, value) { content_tag(:a, value, href: "#") }
-    },
-    {
-      accessor: "attendanceRate",
-      label: "Attendance Rate",
-      custom_renderer: ->(row, value) {
-        capture do
-          pb_rails("flex", props: { align_items: "end", orientation: "column" }) do
-            pb_rails("detail", props: { bold: true, color: "default", text: value }) +
-            pb_rails("caption", props: { size: "xs", text: row[:graduatedStudents] })
-          end
-        end
-      }
-    },
-    {
-      accessor: "completedClasses",
-      label: "Completed Classes",
-    },
-    {
-      accessor: "classCompletionRate",
-      label: "Class Completion Rate",
-    },
-    {
-      accessor: "graduatedStudents",
-      label: "Graduated Students",
-    }
-  ]
-%>
-
-<%= pb_rails("advanced_table", props: { id: "custom_cell", table_data: @table_data, column_definitions: column_definitions }) %>

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

@@ -1,5 +0,0 @@
-The Advanced Table also allows for rendering custom components within individual Cells. To achieve this, you can make use of the optional `custom_renderer` item within each column_definitions. This function gives you access to the current Cell's value if you just want to use that with a custom Kit, but it also gives you access to the entire `row` object. The row object provides all data for the current row.
-
-See [here](https://playbook.powerapp.cloud/kits/advanced_table/rails#column_definitions) for more indepth information on column_definitions are how to use them.
-
-See [here](https://github.com/powerhome/playbook/tree/master/playbook/app/pb_kits/playbook/pb_advanced_table#readme) for the structure of the table_data used.

data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_pagination.jsx

@@ -1,50 +0,0 @@
-import React from "react"
-import { AdvancedTable } from "playbook-ui"
-import PAGINATION_MOCK_DATA from "./advanced_table_pagination_mock_data.json"
-
-const AdvancedTablePagination = (props) => {
-  const columnDefinitions = [
-    {
-      accessor: "year",
-      label: "Year",
-      cellAccessors: ["quarter", "month", "day"],
-    },
-    {
-      accessor: "newEnrollments",
-      label: "New Enrollments",
-    },
-    {
-      accessor: "scheduledMeetings",
-      label: "Scheduled Meetings",
-    },
-    {
-      accessor: "attendanceRate",
-      label: "Attendance Rate",
-    },
-    {
-      accessor: "completedClasses",
-      label: "Completed Classes",
-    },
-    {
-      accessor: "classCompletionRate",
-      label: "Class Completion Rate",
-    },
-    {
-      accessor: "graduatedStudents",
-      label: "Graduated Students",
-    },
-  ]
-
-  return (
-    <>
-      <AdvancedTable
-          columnDefinitions={columnDefinitions}
-          pagination
-          tableData={PAGINATION_MOCK_DATA}
-          {...props}
-      />
-    </>
-  )
-}
-
-export default AdvancedTablePagination

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

@@ -1 +0,0 @@
-`pagination` is an optional prop that can be used to add pagination to the AdvancedTable. If present, it will add pagination with default values for pageSize, etc. To customize pagination, you can also use `paginationProps` shown in the next example.

data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_pagination_with_props.jsx

@@ -1,57 +0,0 @@
-import React from "react"
-import { AdvancedTable } from "playbook-ui"
-import PAGINATION_MOCK_DATA from "./advanced_table_pagination_mock_data.json"
-
-const AdvancedTablePaginationWithProps = (props) => {
-  const columnDefinitions = [
-    {
-      accessor: "year",
-      label: "Year",
-      cellAccessors: ["quarter", "month", "day"],
-    },
-    {
-      accessor: "newEnrollments",
-      label: "New Enrollments",
-    },
-    {
-      accessor: "scheduledMeetings",
-      label: "Scheduled Meetings",
-    },
-    {
-      accessor: "attendanceRate",
-      label: "Attendance Rate",
-    },
-    {
-      accessor: "completedClasses",
-      label: "Completed Classes",
-    },
-    {
-      accessor: "classCompletionRate",
-      label: "Class Completion Rate",
-    },
-    {
-      accessor: "graduatedStudents",
-      label: "Graduated Students",
-    },
-  ]
-
-  const paginationProps = {
-    pageIndex: 1,
-    pageSize: 10,
-    range: 2
-  }
-  
-  return (
-    <>
-      <AdvancedTable
-          columnDefinitions={columnDefinitions}
-          pagination
-          paginationProps={paginationProps}
-          tableData={PAGINATION_MOCK_DATA}
-          {...props}
-      />
-    </>
-  )
-}
-
-export default AdvancedTablePaginationWithProps

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

@@ -1,5 +0,0 @@
-`paginationProps` is an optional prop that can be used to further customize pagination for the AdvancedTable. This prop is an object with 2 optional items:
-
-- `pageIndex`: An optional prop to set which page is set to open on initial load. Default is '0'
-- `pageSize`: An optional prop to set total number of rows for each page before the Table paginates. Default is '20'
-- `range`: The range prop determines how many pages to display in the Pagination component. Regardless of this value, the first two and last two pages are always visible to facilitate navigation to the beginning and end of the pagination. If these always-visible pages fall within the specified range, they are included in the display. If they fall outside the range, the pagination will show additional pages up to the number defined by the range prop. See [here for more details](https://playbook.powerapp.cloud/kits/pagination/react#default). Default is set to '5'