playbook_ui 14.25.0.pre.alpha.testingcss9713 → 14.25.0.pre.alpha.testingcss9751

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

@@ -1,73 +1,76 @@
 @import "../tokens/titles";
 @import "../tokens/colors";
+@import "../tokens/typography";
 @import "../tokens/screen_sizes";
 @import './title_mixin';
 
 .pb_title_kit {
-  &._1 {
+  // Base title styles - default to title 3 styling
+  @include pb_title_3;
+  
+  // Size-based styles (these will override the base)
+  &.pb_title_1 {
     @include pb_title_1;
-    @include title_colors;
     @include pb_title_bold;
   }
 
-  &._2 {
+  &.pb_title_2 {
     @include pb_title_2;
-    @include title_colors;
     @include pb_title_bold;
   }
 
-  &._3 {
+  &.pb_title_3 {
     @include pb_title_3;
-    @include title_colors;
     @include pb_title_bold;
   }
 
-  &._4 {
+  &.pb_title_4 {
     @include pb_title_4;
-    @include title_colors;
   }
 
-  &._thin {
+  &.pb_title_thin {
     @include pb_title_thin;
   }
 
-  &._display {
+  // Color classes
+  @include title_colors;
+
+  // Display sizes
+  &.pb_title_display {
     font-size: clamp(24px, calc(1.25vw + 5.3vw), 128px);
   }
 
-  &._dynamic {
-    &._xs {
-      font-size: min(2vw, 80vw / 16); 
-    }
-  
-    &._sm {
-      font-size: min(2.5vw, 96vw / 16); 
-    }
-  
-    &._md {
-      font-size: min(4vw, 160vw / 16);
-    }
-  
-    &._lg {
-      font-size: min(5vw, 192vw / 16);
-    }
-  
-    &._xl {
-      font-size: min(6vw, 224vw / 16);
-    }
-  
-    &._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 +86,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";
 
-.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;
   }
+}
 
-  &._center {
-    justify-content: center;
-  }
+.pb_title_count_kit_left_lg,
+.pb_title_count_kit_left_sm {
+  justify-content: flex-start;
+}
 
-  &._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";
 
-.pb_title_detail_kit{
-  &._center  {
-    & > .pb_title,
-    & > .pb_body {
-      text-align: center;
-    }
+.pb_title_detail_kit_center {
+  & > .pb_title_kit,
+  & > .pb_body_kit_light {
+    text-align: center;
   }
-  &._right  {
-    & > .pb_title,
-    & > .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,7 @@ $color_checkbox_default: $border_light;
 $color_disabled: darken($border_light, 20%);
 $transition: .2s ease-in-out;
 
-.pb_toggle_kit {
+[class^=pb_toggle_kit] {
   position: relative;
   $width: 44px;
   $height: $width / 2;

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

@@ -104,7 +104,7 @@ $tooltip_shadow: rgba(60, 106, 172, 0.18);
   }
 
   &.dark,
-  &._dark{
+  &[class*=_dark]{
     ~.tooltip_tooltip,
     .tooltip_tooltip {
       //react

data/app/pb_kits/playbook/pb_typeahead/_typeahead.scss

@@ -4,7 +4,7 @@
 @import "../tokens/shadows";
 @import "../tokens/positioning";
 
-.pb_typeahead_kit {
+[class^=pb_typeahead_kit] {
   .typeahead-kit-select__option {
     cursor: pointer;
   }
@@ -28,7 +28,7 @@
       transition: opacity .15s ease-in-out;
     }
   }
-  .pb_text_input_kit {
+  [class^=pb_text_input_kit] {
     .text_input_wrapper {
       .text_input {
         max-height: none;
@@ -47,7 +47,7 @@
       }
     }
   }
-  .pb_list_kit {
+  [class^=pb_list_kit] {
     max-height: 18em;
     overflow-y: auto;
     overscroll-behavior: contain;
@@ -60,15 +60,15 @@
     border-radius: $border_rad_heavier;
     transition: opacity .25s ease-in-out;
   }
-  &:focus-within .pb_list_kit {
+  &:focus-within [class^=pb_list_kit] {
     display: block;
     opacity: 1;
   }
-  &:not(:focus-within) .pb_list_kit {
+  &:not(:focus-within) [class^=pb_list_kit] {
     display: none;
     opacity: 0;
   }
-  .pb_list_kit {
+  [class^=pb_list_kit] {
     li {
       transition: background-color .25s ease-in-out;
     }
@@ -161,7 +161,7 @@
   .typeahead-plus-icon {
     color: $text_lt_lighter;
   }
-  .pb_badge_kit span {
+  [class^=pb_badge_kit] span {
     line-height: 16.5px;
     letter-spacing: normal;
   }
@@ -176,14 +176,14 @@
     }
   }
 
-  &.dark {
+  &[class*=dark] {
     .pb_typeahead_wrapper .pb_typeahead_loading_indicator {
       color: $text_dk_light;
     }
     .pb_text_input_kit_label {
       color: $text_dk_light;
     }
-    .pb_text_input_kit.dark .text_input_wrapper .text_input {
+    [class^=pb_text_input_kit].dark .text_input_wrapper .text_input {
       display: inherit !important;
     }
     .typeahead-kit-select__menu {
@@ -227,7 +227,7 @@
     .typeahead-kit-select__option--is-focused {
       background-color: $active_dark;
     }
-    .pb_list_kit {
+    [class^=pb_list_kit] {
       background-color: $bg_dark;
     }
     .pb_item_kit {

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_user/_user.scss

@@ -3,51 +3,51 @@
 @import "../pb_avatar/avatar";
 @import "../tokens/spacing";
 
-.pb_user_kit{
+[class^=pb_user_kit]{
   display: flex;
   justify-content: flex-start;
   align-items: flex-start;
   flex-direction: row;
 
   // Orientation =============
-  &._vertical  {
+  &[class*=_vertical]  {
     flex-direction: column;
 
-    &._center  {
+    &[class*=_center]  {
       align-items: center;
 
-      .pb_title,
-      .pb_body {
+      [class^=pb_title],
+      [class^=pb_body] {
         text-align: center;
       }
     }
 
-    &._right  {
+    &[class*=_right]  {
       align-items: flex-end;
 
-      .pb_title,
-      .pb_body {
+      [class^=pb_title],
+      [class^=pb_body] {
         text-align: right;
       }
     }
 
-    .pb_avatar {
+    [class^=pb_avatar] {
       margin-bottom: $space-xs;
     }
   }
 
-  &._horizontal  {
+  &[class*=_horizontal]  {
     align-items: center;
 
-    &._center  {
+    &[class*=_center]  {
       justify-content: center;
     }
 
-    &._right  {
+    &[class*=_right]  {
       justify-content: flex-end;
     }
 
-    .pb_avatar {
+    [class^=pb_avatar] {
       margin-right: $space-sm;
     }
   }

data/app/pb_kits/playbook/pb_user/user.test.js

@@ -38,7 +38,7 @@ test('bold prop applies correct styling when false', () => {
   const titleElement = screen.getByText("Anna Black")
   expect(titleElement).toBeInTheDocument()
 
-  expect(titleElement).toHaveClass('pb_title_kit_size_4_thin')
+  expect(titleElement).toHaveClass('pb_title_kit pb_title_4 pb_title_thin')
 })
 
 test('align prop adds desired class', () => {

data/app/pb_kits/playbook/pb_user_badge/_user_badge.scss

@@ -4,7 +4,10 @@ $badge-sizes: (
   "lg":   36px
 );
 
-.pb_user_badge_kit {
+.pb_user_badge_kit,
+.pb_user_badge_kit_sm,
+.pb_user_badge_kit_md,
+.pb_user_badge_kit_lg {
   .pb_user_badge_wrapper {
     svg,
     img {
@@ -12,13 +15,25 @@ $badge-sizes: (
       height: 100%;
     }
   }
+}
 
-  @each $name, $size in $badge-sizes {
-    &[class*=_#{$name}] {
-      .pb_user_badge_wrapper {
-        width: $size;
-        height: $size;
-      }
-    }
+.pb_user_badge_kit_sm {
+  .pb_user_badge_wrapper {
+    width: 16px;
+    height: 16px;
+  }
+}
+
+.pb_user_badge_kit_md {
+  .pb_user_badge_wrapper {
+    width: 26px;
+    height: 26px;
+  }
+}
+
+.pb_user_badge_kit_lg {
+  .pb_user_badge_wrapper {
+    width: 36px;
+    height: 36px;
   }
 }

data/app/pb_kits/playbook/pb_weekday_stacked/_weekday_stacked.scss

@@ -1,11 +1,11 @@
-[class^='pb_weekday_stacked'] {
-  &[class*='_left'] {
-    text-align: left;
-  }
-  &[class*='_center'] {
-    text-align: center;
-  }
-  &[class*='_right'] {
-    text-align: right;
-  }
+.pb_weekday_stacked_kit_left {
+  text-align: left;
+}
+
+.pb_weekday_stacked_kit_center {
+  text-align: center;
+}
+
+.pb_weekday_stacked_kit_right {
+  text-align: right;
 }

data/app/pb_kits/playbook/pb_weekday_stacked/weekday_stacked.test.jsx

@@ -48,7 +48,7 @@ describe("WeekdayStacked Kit", () => {
     );
 
     const kit = screen.getByTestId(testId);
-    const text = kit.querySelector(".pb_title_kit_size_4");
+    const text = kit.querySelector(".pb_title_kit.pb_title_4");
     expect(text.textContent).toEqual("1/1") 
   });
 
@@ -86,7 +86,7 @@ describe("WeekdayStacked Kit", () => {
     );
       
     const kit = screen.getByTestId(testId);
-    const text = kit.querySelector(".pb_title_kit_size_4");
+    const text = kit.querySelector(".pb_title_4");
     expect(text.textContent).toEqual("1") 
   });
 
@@ -99,7 +99,7 @@ describe("WeekdayStacked Kit", () => {
     );
       
     const kit = screen.getByTestId(testId);
-    const text = kit.querySelector(".pb_title_kit_size_4");
+    const text = kit.querySelector(".pb_title_4");
     expect(text.textContent).toEqual("Jan 1") 
   });
 });