playbook_ui 14.24.0 → 14.25.0.pre.alpha.PLAY2361datepickerarrownav10273

data/app/pb_kits/playbook/pb_time_range_inline/time_range_inline.rb

@@ -16,7 +16,7 @@ module Playbook
       prop :timezone, default: false
 
       def classname
-        generate_classname("pb_time_range_inline_kit", dark_class, alignment)
+        generate_classname("pb_time_range_inline_kit", alignment)
       end
 
       def format_start_time_string

data/app/pb_kits/playbook/pb_time_stacked/_time_stacked.scss

@@ -2,15 +2,12 @@
 @import "../pb_caption/caption";
 @import "../tokens/colors";
 
-[class^=pb_time_stacked_kit] {
- &[class*=_center]  {
-   text-align: center;
- }
-
- &[class*=_right]  {
-   text-align: right;
- }
+.pb_time_stacked_kit_center {
+  text-align: center;
+}
 
+.pb_time_stacked_kit_right {
+  text-align: right;
 }
 
 .time-spacing {

data/app/pb_kits/playbook/pb_timestamp/_timestamp.scss

@@ -1,16 +1,22 @@
 @import "timestamp-mixins";
 @import "../tokens/colors";
 
-[class^=pb_timestamp_kit]{
+.pb_timestamp_kit_left,
+.pb_timestamp_kit_center,
+.pb_timestamp_kit_right {
   @include pb_timestamp;
+}
+
+.pb_timestamp_kit_center {
+  text-align: center;
+}
+
+.pb_timestamp_kit_right {
+  text-align: right;
+}
 
-  &[class*=_center]  {
-    text-align: center;
-  }
-  &[class*=_right]  {
-    text-align: right;
-  }
-  &[class*=_dark] {
-    color: $text_dk_default;
-  }
+.pb_timestamp_kit_left.dark,
+.pb_timestamp_kit_center.dark,
+.pb_timestamp_kit_right.dark {
+  color: $text_dk_default;
 }

data/app/pb_kits/playbook/pb_timestamp/_timestamp.tsx

@@ -49,10 +49,7 @@ const Timestamp = (props: TimestampProps): React.ReactElement => {
   const dataProps = buildDataProps(data)
   const htmlProps = buildHtmlProps(htmlOptions)
   const classes = classnames(
-    buildCss('pb_timestamp_kit', align, {
-      dark: dark,
-      variant: variant,
-    }),
+    buildCss('pb_timestamp_kit', align),
     globalProps(props),
     className
   )

data/app/pb_kits/playbook/pb_timestamp/timestamp.rb

@@ -40,7 +40,7 @@ module Playbook
       SECS_PER_CENT = 100 * SECS_PER_YEAR # 3,153,600,000 seconds
 
       def classname
-        generate_classname("pb_timestamp_kit", variant_class, align)
+        generate_classname("pb_timestamp_kit", align)
       end
 
       def timestamp_text

data/app/pb_kits/playbook/pb_timestamp/timestamp.test.js

@@ -36,7 +36,7 @@ describe("Timestamp Kit", () => {
     );
 
     const kit = screen.getByTestId(testId);
-    expect(kit).toHaveClass("pb_timestamp_kit_left_variant");
+    expect(kit).toHaveClass("pb_timestamp_kit_left");
   });
 
   test("renders Timestamp time", () => {
@@ -79,7 +79,7 @@ describe("Timestamp Kit", () => {
     );
 
     const kit = screen.getByTestId(testId);
-    expect(kit).toHaveClass("pb_timestamp_kit_center_variant");
+    expect(kit).toHaveClass("pb_timestamp_kit_center");
   });
 
   test("renders Timestamp timezone", () => {

data/app/pb_kits/playbook/pb_title/_title.scss

@@ -1,73 +1,78 @@
 @import "../tokens/titles";
 @import "../tokens/colors";
+@import "../tokens/typography";
 @import "../tokens/screen_sizes";
 @import './title_mixin';
 
-[class^=pb_title_kit] {
-  &[class*=_1] {
+.pb_title_kit {
+  // Base title styles - default to title 3 styling
+  @include pb_title_3;
+  @include pb_title_bold;
+  
+  // Size-based styles (these will override the base)
+  &.pb_title_1 {
     @include pb_title_1;
-    @include title_colors;
     @include pb_title_bold;
   }
 
-  &[class*=_2] {
+  &.pb_title_2 {
     @include pb_title_2;
-    @include title_colors;
     @include pb_title_bold;
   }
 
-  &[class*=_3] {
+  &.pb_title_3 {
     @include pb_title_3;
-    @include title_colors;
     @include pb_title_bold;
   }
 
-  &[class*=_4] {
+  &.pb_title_4 {
     @include pb_title_4;
-    @include title_colors;
   }
 
-  &[class*=_thin] {
+  &.pb_title_thin {
     @include pb_title_thin;
   }
 
-  &[class*=_display] {
+  // Color classes
+  @include title_colors;
+
+  // Display sizes
+  &.pb_title_display {
     font-size: clamp(24px, calc(1.25vw + 5.3vw), 128px);
+    font-weight: $bold;
   }
 
-  &[class*=_dynamic] {
-    &[class*=_xs] {
-      font-size: min(2vw, 80vw / 16); 
-    }
-  
-    &[class*=_sm] {
-      font-size: min(2.5vw, 96vw / 16); 
-    }
-  
-    &[class*=_md] {
-      font-size: min(4vw, 160vw / 16);
-    }
-  
-    &[class*=_lg] {
-      font-size: min(5vw, 192vw / 16);
-    }
-  
-    &[class*=_xl] {
-      font-size: min(6vw, 224vw / 16);
-    }
-  
-    &[class*=_xxl] {
-      font-size: min(8vw, 256vw / 16);
-    }
+  // Dynamic sizes
+  &.pb_title_dynamic_xs {
+    font-size: min(2vw, 80vw / 16); 
+  }
+
+  &.pb_title_dynamic_sm {
+    font-size: min(2.5vw, 96vw / 16); 
+  }
+
+  &.pb_title_dynamic_md {
+    font-size: min(4vw, 160vw / 16);
   }
-  
 
+  &.pb_title_dynamic_lg {
+    font-size: min(5vw, 192vw / 16);
+  }
+
+  &.pb_title_dynamic_xl {
+    font-size: min(6vw, 224vw / 16);
+  }
+
+  &.pb_title_dynamic_xxl {
+    font-size: min(8vw, 256vw / 16);
+  }
 
+  // Responsive breakpoint classes
   @each $size, $size_value in $breakpoints_grid {
     @for $title_size_value from 1 through 4 {
       $min_size: map-get($size_value, "min");
       $max_size: map-get($size_value, "max");
-      &[class*=_#{$size}_#{$title_size_value}] {
+      &.pb_title_#{$size}_#{$title_size_value} {
         @include break_on($min_size, $max_size) {
           @if $title_size_value == 1 { @include pb_title_1; }
           @else if $title_size_value == 2 { @include pb_title_2; }
@@ -83,9 +88,9 @@
   &.dark {
     @include title_dark;
     @each $name, $color in $pb_dark_title_colors {
-      &[class*="_#{$name}"] {
+      &.pb_title_#{$name} {
         color: $color;
       }
     }
   }
-}
+}

data/app/pb_kits/playbook/pb_title/_title.tsx

@@ -43,32 +43,41 @@ const Title = (props: TitleProps): React.ReactElement => {
   const ariaProps: {[key: string]: string | number} = buildAriaProps(aria)
   const dataProps: {[key: string]: string | number} = buildDataProps(data)
   const htmlProps = buildHtmlProps(htmlOptions)
-  const getBold = bold ? '' : 'thin'
   const isSizeNumberOrString = typeof size === "number" || typeof size === "string"
 
   const buildResponsiveSizeCss = () => {
-    let css = ''
+    const classes: string[] = []
 
     if (!isSizeNumberOrString) {
       Object.entries(size).forEach((sizeObj) => {
-        css += `pb_title_kit_${sizeObj[0]}_${sizeObj[1]} `
+        classes.push(`pb_title_${sizeObj[0]}_${sizeObj[1]}`)
       })
     }
 
-    return css.trim()
+    return classes
   }
 
   const buildDisplaySize = () => {
     if (displaySize) {
-      return `pb_title_kit_dynamic_${displaySize}`
+      return [`pb_title_dynamic_${displaySize}`]
     }
+    return []
   }
 
+  const titleClasses = ['pb_title_kit']
+  
+  if (isSizeNumberOrString) {
+    titleClasses.push(`pb_title_${size}`)
+  }
+  if (variant) titleClasses.push(`pb_title_${variant}`)
+  if (color) titleClasses.push(`pb_title_${color}`)
+  if (!bold) titleClasses.push('pb_title_thin')
+  titleClasses.push(...buildDisplaySize())
+  titleClasses.push(...buildResponsiveSizeCss())
+
   const classes = classnames(
-    buildCss('pb_title_kit', isSizeNumberOrString ? `size_${size}` : "", variant, color, getBold),
+    titleClasses.join(' '),
     globalProps(props),
-    buildDisplaySize(),
-    buildResponsiveSizeCss(),
     className
   )
   const Tag: React.ReactElement | any = `${tag}`

data/app/pb_kits/playbook/pb_title/_title_mixin.scss

@@ -20,7 +20,7 @@ $pb_dark_title_colors: (
 
 @mixin title_colors {
   @each $name, $color in $pb_title_colors {
-    &[class*=_#{$name}] {
+    &.pb_title_#{$name} {
       color: $color
     }
   }
@@ -28,4 +28,4 @@ $pb_dark_title_colors: (
 
 @mixin title_dark {
   color: $text_dk_default;
-}
+}

data/app/pb_kits/playbook/pb_title/title.rb

@@ -21,36 +21,46 @@ module Playbook
                           default: nil
 
       def classname
+        title_classes = ["pb_title_kit"]
+
         if is_size_responsive
-          generate_classname("pb_title_kit", variant, color, is_bold) + generate_responsive_size_classname
+          title_classes << "pb_title_#{variant}" if variant.present?
+          title_classes << "pb_title_#{color}" if color.present?
+          title_classes << "pb_title_#{is_bold}" if is_bold.present?
+          title_classes += generate_responsive_size_classes
         else
-          generate_classname("pb_title_kit", size, variant, color, is_bold) + generate_display_size
+          title_classes << "pb_title_#{size}" if size.present?
+          title_classes << "pb_title_#{variant}" if variant.present?
+          title_classes << "pb_title_#{color}" if color.present?
+          title_classes << "pb_title_#{is_bold}" if is_bold.present?
+          title_classes += generate_display_size_classes
         end
+
+        generate_classname(title_classes.compact.join(" "), separator: " ")
       end
 
       def is_bold
         bold ? nil : "thin"
       end
 
-      def generate_display_size
-        return "" if display_size.nil?
+      def generate_display_size_classes
+        return [] if display_size.nil?
 
-        " pb_title_kit_dynamic_#{display_size}"
+        ["pb_title_dynamic_#{display_size}"]
       end
 
       def is_size_responsive
         try(:size).is_a?(::Hash)
       end
 
-      def generate_responsive_size_classname
-        css = ""
+      def generate_responsive_size_classes
+        classes = []
         if is_size_responsive
           size.each do |key, value|
-            css += " pb_title_kit_#{key}_#{value}"
+            classes << "pb_title_#{key}_#{value}"
           end
         end
-
-        css unless css.blank?
+        classes
       end
     end
   end

data/app/pb_kits/playbook/pb_title/title.test.js

@@ -12,7 +12,7 @@ test('returns namespaced class name', () => {
     )
 
     const kit = screen.getByTestId('primary-test')
-    expect(kit).toHaveClass('pb_title_kit_size_3')
+    expect(kit).toHaveClass('pb_title_kit pb_title_3')
 })
 
 test('with thin font weight', () => {
@@ -25,7 +25,7 @@ test('with thin font weight', () => {
     )
 
     const kit = screen.getByTestId('primary-test')
-    expect(kit).toHaveClass('pb_title_kit_size_3_thin')
+    expect(kit).toHaveClass('pb_title_kit pb_title_3 pb_title_thin')
 })
 
 test('with colors', () => {
@@ -38,7 +38,7 @@ test('with colors', () => {
     )
 
     const kit = screen.getByTestId('primary-test')
-    expect(kit).toHaveClass('pb_title_kit_size_3_success')
+    expect(kit).toHaveClass('pb_title_kit pb_title_3 pb_title_success')
 })
 
 test('with responsive title', () => {
@@ -51,5 +51,5 @@ test('with responsive title', () => {
     )
 
     const kit = screen.getByTestId('primary-test')
-    expect(kit).toHaveClass('pb_title_kit pb_title_kit_xs_3 pb_title_kit_sm_2 pb_title_kit_md_1')
+    expect(kit).toHaveClass('pb_title_kit pb_title_xs_3 pb_title_sm_2 pb_title_md_1')
 })

data/app/pb_kits/playbook/pb_title_count/_title_count.scss

@@ -1,20 +1,31 @@
 @import "../pb_title/title";
 @import "../pb_body/body";
 
-[class^=pb_title_count_kit] {
+.pb_title_count_kit_left_lg,
+.pb_title_count_kit_left_sm,
+.pb_title_count_kit_center_lg,
+.pb_title_count_kit_center_sm,
+.pb_title_count_kit_right_lg,
+.pb_title_count_kit_right_sm {
   display: flex;
-  justify-content: flex-start;
   align-items: baseline;
 
   & > .pb_title_count_text {
     padding-right: $space-xs;
   }
+}
 
-  &[class*=_center] {
-    justify-content: center;
-  }
+.pb_title_count_kit_left_lg,
+.pb_title_count_kit_left_sm {
+  justify-content: flex-start;
+}
 
-  &[class*=_right] {
-    justify-content: flex-end;
-  }
+.pb_title_count_kit_center_lg,
+.pb_title_count_kit_center_sm {
+  justify-content: center;
 }
+
+.pb_title_count_kit_right_lg,
+.pb_title_count_kit_right_sm {
+  justify-content: flex-end;
+}

data/app/pb_kits/playbook/pb_title_detail/_title_detail.scss

@@ -1,17 +1,16 @@
 @import "../pb_title/title";
 @import "../pb_body/body";
 
-[class^=pb_title_detail_kit]{
-  &[class*=_center]  {
-    & > [class^=pb_title],
-    & > [class^=pb_body] {
-      text-align: center;
-    }
+.pb_title_detail_kit_center {
+  & > .pb_title_kit,
+  & > .pb_body_kit_light {
+    text-align: center;
   }
-  &[class*=_right]  {
-    & > [class^=pb_title],
-    & > [class^=pb_body] {
-      text-align: right;
-    }
+}
+
+.pb_title_detail_kit_right {
+  & > .pb_title_kit,
+  & > .pb_body_kit_light {
+    text-align: right !important;
   }
 }

data/app/pb_kits/playbook/pb_toggle/_toggle.scss

@@ -5,7 +5,10 @@ $color_checkbox_default: $border_light;
 $color_disabled: darken($border_light, 20%);
 $transition: .2s ease-in-out;
 
-[class^=pb_toggle_kit] {
+.pb_toggle_kit_sm_on,
+.pb_toggle_kit_sm_off,
+.pb_toggle_kit_md_on,
+.pb_toggle_kit_md_off {
   position: relative;
   $width: 44px;
   $height: $width / 2;

data/app/pb_kits/playbook/pb_tooltip/_tooltip.scss

@@ -22,7 +22,7 @@ $tooltip_shadow: rgba(60, 106, 172, 0.18);
   }
 }
 
-[class^="pb_tooltip_kit"] {
+.pb_tooltip_kit {
   ~ .tooltip_tooltip,
   .tooltip_tooltip {
     display: block;
@@ -103,8 +103,7 @@ $tooltip_shadow: rgba(60, 106, 172, 0.18);
     }
   }
 
-  &.dark,
-  &[class*=_dark]{
+  &.dark {
     ~.tooltip_tooltip,
     .tooltip_tooltip {
       //react
@@ -136,7 +135,7 @@ $tooltip_shadow: rgba(60, 106, 172, 0.18);
 }
 
 // Right
-[class^="pb_tooltip_kit"] .tooltip_tooltip {
+.pb_tooltip_kit .tooltip_tooltip {
   $arrow_vertical_offset: calc(50% - #{$space_xs * 1.2});
 
   &[data-popper-placement="right"] {

data/app/pb_kits/playbook/pb_tooltip/tooltip.rb

@@ -26,7 +26,7 @@ module Playbook
       prop :width
 
       def classname
-        generate_classname("pb_tooltip_kit", dark_class)
+        generate_classname("pb_tooltip_kit")
       end
 
       def remove_height_properties(combined_html_options_style)

data/app/pb_kits/playbook/pb_typeahead/_typeahead.test.jsx

@@ -63,7 +63,7 @@ test('typeahead with pills', () => {
   )
 
   const kit = screen.getByTestId('pills-test')
-  const pill = kit.querySelector(".pb_form_pill_kit_primary")
+  const pill = kit.querySelector(".pb_form_pill_kit.pb_form_pill_primary")
   expect(pill).toBeInTheDocument()
 })
 
@@ -89,7 +89,7 @@ test('typeahead multi select with badges and small pills', () => {
   )
 
   const kit = screen.getByTestId('small-pill-test')
-  const badge = kit.querySelector(".pb_form_pill_kit_primary.mr_xs.small")
+  const badge = kit.querySelector(".pb_form_pill_kit.pb_form_pill_primary.mr_xs.pb_form_pill_small")
   expect(badge).toBeInTheDocument()
 })
 
@@ -135,6 +135,6 @@ test('typeahead with colored pills', () => {
   )
 
   const kit = screen.getByTestId('pills-color-test')
-  const pill = kit.querySelector(".pb_form_pill_kit_neutral")
+  const pill = kit.querySelector(".pb_form_pill_kit.pb_form_pill_neutral")
   expect(pill).toBeInTheDocument()
 })

data/app/pb_kits/playbook/pb_typeahead/docs/_typeahead_with_pills.html.erb

@@ -7,7 +7,7 @@
   ]
 %>
 
-<%= pb_rails("typeahead", props: { id: "typeahead-pills-example1", default_options: [options.first], options: options, label: "Colors", name: :foo, pills: true }) %>
+<%= pb_rails("typeahead", props: { id: "typeahead-pills-example1", default_options: [options.first], options: options, label: "Products", name: :foo, pills: true }) %>
 
 <%= pb_rails("button", props: {id: "clear-pills", text: "Clear All Options", variant: "secondary"}) %>
 

data/app/pb_kits/playbook/pb_typeahead/docs/_typeahead_with_pills.jsx

@@ -13,7 +13,7 @@ const TypeaheadWithPills = (props) => {
     <>
       <Typeahead
           isMulti
-          label="Colors"
+          label="Products"
           options={options}
           placeholder=""
           {...props}

data/app/pb_kits/playbook/pb_typeahead/docs/_typeahead_with_pills_color.html.erb

@@ -7,25 +7,4 @@
   ]
 %>
 
-<%= pb_rails("typeahead", props: { id: "typeahead-pills-example1", pill_color: "neutral", default_options: [options.first], options: options, label: "Colors", name: :foo, pills: true }) %>
-
-<%= pb_rails("button", props: {id: "clear-pills", text: "Clear All Options", variant: "secondary"}) %>
-
-<!-- This section is an example of the available JavaScript event hooks -->
-<%= javascript_tag defer: "defer" do %>
-  document.addEventListener("pb-typeahead-kit-typeahead-pills-example1-result-option-select", function(event) {
-    console.log('Option selected')
-    console.dir(event.detail)
-  })
-  document.addEventListener("pb-typeahead-kit-typeahead-pills-example1-result-option-remove", function(event) {
-    console.log('Option removed')
-    console.dir(event.detail)
-  })
-  document.addEventListener("pb-typeahead-kit-typeahead-pills-example1-result-clear", function() {
-    console.log('All options cleared')
-  })
-
-  document.querySelector('#clear-pills').addEventListener('click', function() {
-    document.dispatchEvent(new CustomEvent('pb-typeahead-kit-typeahead-pills-example1:clear'))
-  })
-<% end %>
+<%= pb_rails("typeahead", props: { id: "typeahead-pills-example2", pill_color: "neutral", options: options, label: "Products", name: :foo, pills: true }) %>

data/app/pb_kits/playbook/pb_typeahead/docs/_typeahead_with_pills_color.jsx

@@ -13,7 +13,7 @@ const TypeaheadWithPills = (props) => {
     <>
       <Typeahead
           isMulti
-          label="Colors"
+          label="Products"
           options={options}
           pillColor="neutral"
           placeholder=""

data/app/pb_kits/playbook/pb_typeahead/docs/_typeahead_without_pills.html.erb

@@ -9,9 +9,9 @@
 
 <%= pb_rails("typeahead", props: {
     id: "typeahead-without-pills-example1",
-    placeholder: "All Colors",
+    placeholder: "All Products",
     options: options, 
-    label: "Colors",
+    label: "Products",
     name: :foo,
     is_multi: false
   })