playbook_ui 14.25.0.pre.alpha.testingcss9700 → 14.25.0.pre.alpha.testingcss9751

data/app/pb_kits/playbook/pb_layout/_layout.tsx

@@ -22,7 +22,7 @@ type LayoutPropTypes = {
   size?: "xs" | "sm" | "md" | "base" | "lg" | "xl",
   variant?: "light" | "dark" | "gradient",
   transparent?: boolean,
-  layout?: "sidebar" | "collection" | "kanban" | "content" | "masonry" | "bracket",
+  layout?: "sidebar" | "collection" | "collection_detail" | "kanban" | "content" | "masonry" | "bracket",
 } & GlobalProps
 
 type LayoutSideProps = {
@@ -81,11 +81,10 @@ const Body = (props: LayoutBodyProps) => {
 // Item component
 const Item = (props: LayoutItemProps) => {
   const { children, className, size = 'sm' } = props
-  const sizeClass = `size_${size}`
   const dynamicInlineProps = globalInlineProps(props)
   return (
     <div
-        className={classnames('layout_item', sizeClass, globalProps(props), className)}
+        className={classnames('layout_item', `layout_item_size_${size}`, globalProps(props), className)}
         style={dynamicInlineProps}
     >
       {children}
@@ -144,20 +143,24 @@ const Layout = (props: LayoutPropTypes) => {
   const htmlProps = buildHtmlProps(htmlOptions)
 
   const layoutCss =
-    (layout == 'collection' || layout == 'bracket')
+    (layout == 'collection' || layout == 'bracket' || layout == 'collection_detail' || layout == 'content' || layout == 'masonry')
       ? `pb_layout_kit_${layout}`
       : layout == 'kanban'
         ? `pb_layout_kit_${layout}${responsiveClass}`
-        : buildCss(`pb_layout_kit_${layout}`, `size_${size}`, position, variant, {
-          dark: dark,
-          transparent: transparent,
-          full: full,
-        })
+        : classnames(
+            'pb_layout_kit_sidebar',
+            `pb_layout_kit_sidebar_size_${size}_${position}_${variant}`,
+            {
+              'pb_layout_kit_sidebar_dark': dark,
+              'pb_layout_kit_sidebar_transparent': transparent,
+              'pb_layout_kit_sidebar_full': full,
+            }
+          )
 
   const layoutCollapseCss =
-    (layout == 'collection' || layout == 'kanban' || layout == 'bracket')
+    (layout == 'collection' || layout == 'collection_detail' || layout == 'content' || layout == 'masonry' || layout == 'kanban' || layout == 'bracket')
       ? ''
-      : buildCss('layout', position, 'collapse', collapse)
+      : `layout_${position}_collapse_${collapse}`
 
   const layoutChildren = React.Children.toArray(children)
 

data/app/pb_kits/playbook/pb_layout/body.rb

@@ -8,7 +8,7 @@ module Playbook
                  default: "div"
 
       def classname
-        generate_classname("layout_body")
+        "layout_body"
       end
     end
   end

data/app/pb_kits/playbook/pb_layout/footer.rb

@@ -8,7 +8,7 @@ module Playbook
                  default: "div"
 
       def classname
-        generate_classname("layout_footer")
+        "layout_footer"
       end
     end
   end

data/app/pb_kits/playbook/pb_layout/header.rb

@@ -8,7 +8,7 @@ module Playbook
                  default: "div"
 
       def classname
-        generate_classname("layout_header")
+        "layout_header"
       end
     end
   end

data/app/pb_kits/playbook/pb_layout/item.rb

@@ -8,13 +8,7 @@ module Playbook
                   default: "sm"
 
       def classname
-        generate_classname("layout_item") + size_class
-      end
-
-    private
-
-      def size_class
-        " size_#{size}"
+        "layout_item layout_item_size_#{size}"
       end
     end
   end

data/app/pb_kits/playbook/pb_layout/layout.rb

@@ -25,38 +25,29 @@ module Playbook
       def classname
         case layout
         when "collection"
-          generate_classname("pb_layout_kit", layout)
+          "pb_layout_kit_collection"
         when "kanban"
-          generate_classname("pb_layout_kit", layout, responsive_class)
+          classes = ["pb_layout_kit_kanban"]
+          classes << "pb_layout_kit_kanban_responsive" if responsive
+          classes.join(" ")
         when "collection_detail"
-          generate_classname("pb_layout_kit", layout)
+          "pb_layout_kit_collection_detail"
         when "content"
-          generate_classname("pb_layout_kit", layout)
+          "pb_layout_kit_content"
         when "masonry"
-          generate_classname("pb_layout_kit", layout)
+          "pb_layout_kit_masonry"
         else
-          first_class = generate_classname("pb_layout_kit_sidebar", "size_#{size}", position, variant, transparent_class)
-          [first_class, full_class, collapse_class].reject(&:empty?).join(" ")
+          # Sidebar layout
+          classes = [
+            "pb_layout_kit_sidebar",
+            "pb_layout_kit_sidebar_size_#{size}_#{position}_#{variant}",
+          ]
+          classes << "pb_layout_kit_sidebar_transparent" if transparent
+          classes << "pb_layout_kit_sidebar_full" if full
+          classes << "layout_#{position}_collapse_#{collapse}"
+          classes.join(" ")
         end
       end
-
-    private
-
-      def responsive_class
-        responsive ? "responsive" : ""
-      end
-
-      def full_class
-        full ? "full" : ""
-      end
-
-      def transparent_class
-        transparent ? "transparent" : nil
-      end
-
-      def collapse_class
-        "layout_#{position}_collapse_#{collapse}"
-      end
     end
   end
 end

data/app/pb_kits/playbook/pb_layout/layout.test.js

@@ -68,11 +68,11 @@ test("render all layout variants", () => {
     },
     {
       layout: "collection_detail",
-      expected: "pb_layout_kit_collection_detail_size_md_left_light",
+      expected: "pb_layout_kit_collection_detail",
     },
     {
       layout: "content",
-      expected: "pb_layout_kit_content_size_md_left_light",
+      expected: "pb_layout_kit_content",
     },
     {
       layout: "kanban",
@@ -80,7 +80,7 @@ test("render all layout variants", () => {
     },
     {
       layout: "masonry",
-      expected: "pb_layout_kit_masonry_size_md_left_light",
+      expected: "pb_layout_kit_masonry",
     },
     {
       layout: "bracket",

data/app/pb_kits/playbook/pb_layout/sidebar.rb

@@ -4,7 +4,7 @@ module Playbook
   module PbLayout
     class Sidebar < Playbook::KitBase
       def classname
-        generate_classname("layout_sidebar")
+        "layout_sidebar"
       end
     end
   end

data/app/pb_kits/playbook/pb_legend/_legend.scss

@@ -9,26 +9,76 @@
   background: $color;
 }
 
-@mixin indicator-colors($colors-list) {
-  @each $name, $color in $colors-list {
-    &[class*=#{$name}] {
-      .pb_legend_indicator_circle {
-        @include pb_button($color);
-      }
+// Base class for custom colors (no color suffix)
+.pb_legend_kit {
+  .pb_legend_indicator_circle_custom {
+    display: inline-block;
+    width: 10px;
+    height: 10px;
+    border-radius: 5px;
+  }
+}
+
+// Data colors
+@each $name, $color in $data_colors {
+  .pb_legend_kit_#{$name} {
+    .pb_legend_indicator_circle {
+      @include pb_button($color);
+    }
+    
+    .pb_legend_indicator_circle_custom {
+      display: inline-block;
+      width: 10px;
+      height: 10px;
+      border-radius: 5px;
     }
   }
 }
 
-[class^=pb_legend_kit] {
-  @include indicator-colors($data_colors);
-  @include indicator-colors($status_colors);
-  @include indicator-colors($product_colors);
-  @include indicator-colors($category_colors);
+// Status colors
+@each $name, $color in $status_colors {
+  .pb_legend_kit_#{$name} {
+    .pb_legend_indicator_circle {
+      @include pb_button($color);
+    }
+    
+    .pb_legend_indicator_circle_custom {
+      display: inline-block;
+      width: 10px;
+      height: 10px;
+      border-radius: 5px;
+    }
+  }
+}
 
-  .pb_legend_indicator_circle_custom {
-  display: inline-block;
-  width: 10px;
-  height: 10px;
-  border-radius: 5px;
+// Product colors
+@each $name, $color in $product_colors {
+  .pb_legend_kit_#{$name} {
+    .pb_legend_indicator_circle {
+      @include pb_button($color);
+    }
+    
+    .pb_legend_indicator_circle_custom {
+      display: inline-block;
+      width: 10px;
+      height: 10px;
+      border-radius: 5px;
+    }
   }
 }
+
+// Category colors
+@each $name, $color in $category_colors {
+  .pb_legend_kit_#{$name} {
+    .pb_legend_indicator_circle {
+      @include pb_button($color);
+    }
+    
+    .pb_legend_indicator_circle_custom {
+      display: inline-block;
+      width: 10px;
+      height: 10px;
+      border-radius: 5px;
+    }
+  }
+}

data/app/pb_kits/playbook/pb_legend/legend.test.js

@@ -39,7 +39,7 @@ test('prefixText prop renders', () => {
   )
 
   const kit = screen.getByTestId('primary-test')
-  const prefix = kit.querySelector(".pb_title_kit_size_4")
+  const prefix = kit.querySelector(".pb_title_kit.pb_title_4")
   expect(prefix).toBeInTheDocument()
 })
 

data/app/pb_kits/playbook/pb_lightbox/lightbox.scss

@@ -66,18 +66,18 @@ $lightbox-z-index-floor: $z_10 !default;
   width: 100%;
   z-index: $lightbox-z-index-floor + 1;
 
-  [class^="react-transform-wrapper"] {
+  .react-transform-wrapper {
     flex-shrink: 0;
     width: 100%;
     height: 100%;
   }
 
-  [class^="react-transform-content"] {
+  .react-transform-content {
     width: 100%;
     height: 100%;
   }
 
-  [class^="react-transform-component"] {
+  .react-transform-component {
     width: 100%;
     height: 100%;
     img {
@@ -164,4 +164,4 @@ $lightbox-z-index-floor: $z_10 !default;
   align-items: center;
   background-color: white;
   box-shadow: 0 0 6px white;
-}
+}

data/app/pb_kits/playbook/pb_lightbox/lightbox.test.jsx

@@ -113,7 +113,7 @@ test('Closes on close button', async () => {
   )
 
   const kit = screen.getByTestId(testId)
-  const closeIcon = kit.getElementsByClassName('pb_button_kit_link_inline_enabled')[0]
+  const closeIcon = kit.querySelector('.pb_button_kit.pb_button_link.pb_button_inline.pb_button_enabled')
 
   fireEvent(
     closeIcon,

data/app/pb_kits/playbook/pb_link/_link.scss

@@ -3,7 +3,8 @@
 @import "../tokens/typography";
 @import "../tokens/border_radius";
 
-[class^=pb_link_kit]{
+// Base styles
+.pb_link_kit {
   @include pb_link($primary);
   &:hover {
     color: $text_lt_default;
@@ -25,42 +26,135 @@
       color: $text_dk_default;
     }
   }
-  @each $color_name, $color_value in $pb_link_colors {
-    &[class*=_#{"" + $color_name}] {
-      @include pb_link($color_value);
+}
 
+// Color variants
+@each $color_name, $color_value in $pb_link_colors {
+  @if $color_name != "default" {
+    .pb_link_kit_#{$color_name} {
+      @include pb_link($color_value);
       &:hover {
         color: map-get($pb_link_hover_colors, $color_name);
       }
-
       &:visited {
         color: darken($primary_action, 10%);
       }
+      &:focus {
+        outline: none;
+      }
+      &:focus-visible {
+        border-radius: $border_rad_light;
+        outline: 1px solid $primary;
+        outline-offset: 2px;
+      }
     }
   }
+}
 
-  @each $dark_color_name, $dark_color_value in $pb_dark_link_colors{
-    &[class*=_#{$dark_color_name}][class*=dark]{
-      @include pb_link($dark_color_value);
-      
+// Dark mode color variants
+@each $dark_color_name, $dark_color_value in $pb_dark_link_colors {
+  .pb_link_kit_#{$dark_color_name}.dark {
+    @include pb_link($dark_color_value);
+    &:hover {
+      color: map-get($pb_dark_link_hover_colors, $dark_color_name);
+    }
+    &:visited {
+      color: darken($primary_action, 10%);
+    }
+  }
+}
+
+// Underline modifier
+.pb_link_kit_underline,
+.pb_link_kit_body_underline,
+.pb_link_kit_muted_underline,
+.pb_link_kit_destructive_underline,
+.pb_link_kit_underline_disabled,
+.pb_link_kit_body_underline_disabled,
+.pb_link_kit_muted_underline_disabled,
+.pb_link_kit_destructive_underline_disabled {
+  text-decoration: underline;
+}
+
+// Disabled modifier
+.pb_link_kit_disabled,
+.pb_link_kit_body_disabled,
+.pb_link_kit_muted_disabled,
+.pb_link_kit_destructive_disabled,
+.pb_link_kit_underline_disabled,
+.pb_link_kit_body_underline_disabled,
+.pb_link_kit_muted_underline_disabled,
+.pb_link_kit_destructive_underline_disabled {
+  pointer-events: none;
+  cursor: default;
+  color: $text_lt_lighter;
+}
+
+// Color + underline combinations
+@each $color_name, $color_value in $pb_link_colors {
+  @if $color_name != "default" {
+    .pb_link_kit_#{$color_name}_underline {
+      @include pb_link($color_value);
       &:hover {
-        color: map-get($pb_dark_link_hover_colors, $dark_color_name);
+        color: map-get($pb_link_hover_colors, $color_name);
       }
-
       &:visited {
         color: darken($primary_action, 10%);
       }
+      &:focus {
+        outline: none;
+      }
+      &:focus-visible {
+        border-radius: $border_rad_light;
+        outline: 1px solid $primary;
+        outline-offset: 2px;
+      }
     }
   }
+}
 
-  &[class*=_underline] {
-    text-decoration: underline;
+// Color + disabled combinations
+@each $color_name, $color_value in $pb_link_colors {
+  @if $color_name != "default" {
+    .pb_link_kit_#{$color_name}_disabled {
+      @include pb_link($color_value);
+      &:hover {
+        color: map-get($pb_link_hover_colors, $color_name);
+      }
+      &:visited {
+        color: darken($primary_action, 10%);
+      }
+      &:focus {
+        outline: none;
+      }
+      &:focus-visible {
+        border-radius: $border_rad_light;
+        outline: 1px solid $primary;
+        outline-offset: 2px;
+      }
+    }
   }
+}
 
-  &[class*=_disabled] {
-    pointer-events: none;
-    cursor: default;
-    color: $text_lt_lighter;
+// Color + underline + disabled combinations
+@each $color_name, $color_value in $pb_link_colors {
+  @if $color_name != "default" {
+    .pb_link_kit_#{$color_name}_underline_disabled {
+      @include pb_link($color_value);
+      &:hover {
+        color: map-get($pb_link_hover_colors, $color_name);
+      }
+      &:visited {
+        color: darken($primary_action, 10%);
+      }
+      &:focus {
+        outline: none;
+      }
+      &:focus-visible {
+        border-radius: $border_rad_light;
+        outline: 1px solid $primary;
+        outline-offset: 2px;
+      }
+    }
   }
-
-}
+}

data/app/pb_kits/playbook/pb_list/_list.scss

@@ -28,4 +28,4 @@
   &[class*=_layout_right] li {
     @include pb_list_layout_right;
   }
-}
+}

data/app/pb_kits/playbook/pb_loading_inline/_loading_inline.scss

@@ -1,19 +1,21 @@
 @import "../pb_body/body";
 @import "../pb_icon/icon";
 
-[class^=pb_loading_inline_kit] {
+.pb_loading_inline_kit_left,
+.pb_loading_inline_kit_center,
+.pb_loading_inline_kit_right {
   display: flex;
   align-items: baseline;
+}
 
-  &[class*=_center] {
-    justify-content: center;
-  }
-
-  &[class*=_left] {
-    justify-content: flex-start;
-  }
+.pb_loading_inline_kit_center {
+  justify-content: center;
+}
 
-  &[class*=_right] {
-    justify-content: flex-end;
-  }
+.pb_loading_inline_kit_left {
+  justify-content: flex-start;
 }
+
+.pb_loading_inline_kit_right {
+  justify-content: flex-end;
+}

data/app/pb_kits/playbook/pb_map/_map.scss

@@ -4,7 +4,7 @@
 @import "../tokens/border_radius";
 @import "./_pb_map_button_mixin.scss";
 
-[class*="pb_map"] {
+.pb_map {
   .pb_map-custom-button {
     @include pb_map_button;
     position: relative;

data/app/pb_kits/playbook/pb_map/docs/_map_default.jsx

@@ -26,7 +26,7 @@ const MapDefault = (props) => {
         new maplibregl.Marker({
           color: mapTheme.marker,
         }).setLngLat(defaultPosition)
-        .setPopup(new maplibregl.Popup({closeButton: false}).setHTML(`<h4 class="pb_title_kit_size_4">Hello World!</h4>`)) // add popup
+        .setPopup(new maplibregl.Popup({closeButton: false}).setHTML(`<h4 class="pb_title_kit pb_title_4">Hello World!</h4>`)) // add popup
         .addTo(map);
 
         // disable map zoom when using scroll

data/app/pb_kits/playbook/pb_map/docs/_map_with_custom_button.jsx

@@ -29,7 +29,7 @@ const MapWithCustomButton = (props) => {
         new maplibregl.Marker({
           color: mapTheme.marker,
         }).setLngLat(defaultPosition)
-        .setPopup(new maplibregl.Popup({closeButton: false}).setHTML(`<h4 class="pb_title_kit_size_4">Hello World!</h4>`)) // add popup
+        .setPopup(new maplibregl.Popup({closeButton: false}).setHTML(`<h4 class="pb_title_kit pb_title_4">Hello World!</h4>`)) // add popup
         .addTo(map);
 
         // disable map zoom when using scroll

data/app/pb_kits/playbook/pb_map/docs/_map_with_plugin.jsx

@@ -26,7 +26,7 @@ const MapWithPlugin = (props) => {
         new maplibregl.Marker({
           color: mapTheme.marker,
         }).setLngLat(defaultPosition)
-        .setPopup(new maplibregl.Popup({className: 'map_popup', closeButton: false}).setHTML(`<h4 class="pb_title_kit_size_4">Hello World!</h4>`)) // add popup
+        .setPopup(new maplibregl.Popup({closeButton: false}).setHTML(`<h4 class="pb_title_kit pb_title_4">Hello World!</h4>`)) // add popup
         .addTo(map);
 
         //add maplibre default zoom controls

data/app/pb_kits/playbook/pb_message/_message.scss

@@ -7,21 +7,26 @@
 $mention_bg_color_yellow: rgba(249, 187, 0, 0.2);
 $mention_bg_color_blue: #e5eefa;
 
-[class^=pb_message_kit] {
-  @include pb_message;
+.pb_message_kit {
+  @include pb_message(false);
+}
+
+.pb_message_kit_avatar {
+  @include pb_message(true);
+}
 
-  [class^=pb_message_mention] {
-    border-radius: $space_xxs;
-    color: $primary;
-    display: inline;
-    padding: $space_xxs;
-    
-    &[class*=_user] {
-      background-color: $mention_bg_color_blue;
-    }
-    
-    &[class*=_self] {
-      background-color: $mention_bg_color_yellow;
-    }
-  }
+.pb_message_mention_user,
+.pb_message_mention_self {
+  border-radius: $space_xxs;
+  color: $primary;
+  display: inline;
+  padding: $space_xxs;
 }
+
+.pb_message_mention_user {
+  background-color: $mention_bg_color_blue;
+}
+
+.pb_message_mention_self {
+  background-color: $mention_bg_color_yellow;
+}

data/app/pb_kits/playbook/pb_message/_message.tsx

@@ -84,6 +84,7 @@ const Message = (props: MessageProps) => {
         <Flex
             justify={alignTimestamp === 'left' ? 'none' : 'between'}
             orientation="row"
+            width="100%"
         >
           {label &&
             <Title

data/app/pb_kits/playbook/pb_message/_message_mixins.scss

@@ -1,19 +1,18 @@
 @import "../tokens/spacing";
 
-@mixin pb_message {
+@mixin pb_message($with_avatar: false) {
   display: flex;
   justify-content: flex-start;
 
   .content_wrapper {
     width: 100%;
     font-size: $font_base - 1;
-
-    .pull-left {
+    @if $with_avatar {
       margin-left: $space-xs;
     }
 
-    [class^=pb_flex_] {
-      width: 100%;
+    .pull-left {
+      margin-left: $space-xs;
     }
 
     .message_text {
@@ -25,12 +24,6 @@
     }
   }
 
-  &[class*=_avatar] {
-    .content_wrapper {
-      margin-left: $space-xs;
-    }
-  }
-
   .pb_message_body {
     font-size: 15px;
   }
@@ -48,4 +41,4 @@
       display: none;
     }
   }
-}
+}