playbook_ui 14.26.0.pre.rc.0 → 15.0.0.pre.alpha.PLAY1981sectionseparatorverticalchildren10469

data/app/pb_kits/playbook/pb_radio/_radio.scss

@@ -4,7 +4,12 @@
 @import "../pb_body/body";
 @import "../tokens/transition";
 
-[class^="pb_radio_kit"] {
+.pb_radio_kit,
+.pb_radio_kit.error,
+.pb_radio_kit.vertical,
+.pb_radio_kit.error.vertical,
+.pb_radio_kit_vertical,
+.pb_radio_kit_vertical.error {
   display: inline-flex;
   cursor: pointer;
 
@@ -59,71 +64,85 @@
       border: 6px solid $neutral;
     }
   }
+}
 
-  &[class*="vertical"] {
-    flex-direction: column;
-    align-items: center;
-    .pb_radio_button {
-      margin-right: 0px;
-      margin-bottom: $space_xs;
-    }
+.pb_radio_kit.vertical,
+.pb_radio_kit.error.vertical,
+.pb_radio_kit_vertical,
+.pb_radio_kit_vertical.error {
+  flex-direction: column;
+  align-items: center;
+  .pb_radio_button {
+    margin-right: 0px;
+    margin-bottom: $space_xs;
   }
+}
 
-  &.dark {
-    .pb_radio_button {
-      border-color: $border_dark;
-    }
-    input {
-      &:focus-visible {
-        & ~ .pb_radio_button {
-          box-shadow: 0px 0px 0px 2px $bg_dark_card, 0px 0px 0px 4px $primary;
-        }
+.pb_radio_kit.dark,
+.pb_radio_kit.dark.error,
+.pb_radio_kit.dark.vertical,
+.pb_radio_kit.dark.error.vertical,
+.pb_radio_kit_vertical.dark,
+.pb_radio_kit_vertical.dark.error {
+  .pb_radio_button {
+    border-color: $border_dark;
+  }
+  input {
+    &:focus-visible {
+      & ~ .pb_radio_button {
+        box-shadow: 0px 0px 0px 2px $bg_dark_card, 0px 0px 0px 4px $primary;
       }
-      &:checked {
-        & ~ .pb_radio_button {
-          border: 6px solid $active_dark;
-        }
+    }
+    &:checked {
+      & ~ .pb_radio_button {
+        border: 6px solid $active_dark;
       }
+    }
 
-      &:disabled + .pb_radio_button {
-        cursor: not-allowed;
-        background-color: mix($charcoal, $white, 80%);
-        border: 2px solid $border_dark;
-
-        & ~ .pb_body_kit {
-          cursor: not-allowed;
-          color: $text_dk_light;
-        }
-      }
+    &:disabled + .pb_radio_button {
+      cursor: not-allowed;
+      background-color: mix($charcoal, $white, 80%);
+      border: 2px solid $border_dark;
 
-      &:disabled:checked {
-        & ~ .pb_radio_button {
-          background-color: mix($charcoal, $white, 80%);
-          border: 6px solid $border_dark;
-        }
+      & ~ .pb_body_kit {
+        cursor: not-allowed;
+        color: $text_dk_light;
       }
     }
 
-    @media (hover: hover) {
-      &:hover {
-        .pb_radio_button {
-          background-color: rgba($white, 0.1);
-          border-color: $border_dark;
-        }
+    &:disabled:checked {
+      & ~ .pb_radio_button {
+        background-color: mix($charcoal, $white, 80%);
+        border: 6px solid $border_dark;
       }
     }
-    &.error {
-      > .pb_radio_button {
-        border-color: $error_dark;
-      }
-      > .pb_body_kit_negative {
-        color: $error_dark;
+  }
+
+  @media (hover: hover) {
+    &:hover {
+      .pb_radio_button {
+        background-color: rgba($white, 0.1);
+        border-color: $border_dark;
       }
     }
   }
-  &.error {
-    > .pb_radio_button {
-      border-color: $error;
-    }
+}
+
+.pb_radio_kit.dark.error,
+.pb_radio_kit.dark.error.vertical,
+.pb_radio_kit_vertical.dark.error {
+  > .pb_radio_button {
+    border-color: $error_dark;
+  }
+  > .pb_body_kit_negative {
+    color: $error_dark;
+  }
+}
+
+.pb_radio_kit.error,
+.pb_radio_kit.error.vertical,
+.pb_radio_kit_vertical.error {
+  > .pb_radio_button {
+    border-color: $error;
   }
 }

data/app/pb_kits/playbook/pb_radio/_radio.tsx

@@ -60,7 +60,7 @@ const Radio = forwardRef<HTMLInputElement, RadioProps>(({
   const htmlProps = buildHtmlProps(htmlOptions)
 
   const classes = classnames(
-    buildCss('pb_radio_kit', alignment),
+    buildCss('pb_radio_kit', `${alignment === 'vertical' ? 'vertical' : ''}`),
     dark ? 'dark' : null,
     error ? 'error' : null,
     globalProps(props),

data/app/pb_kits/playbook/pb_radio/docs/_radio_react_hook.jsx

@@ -19,7 +19,6 @@ const RadioReactHook = () => {
           paddingRight="lg"
         >
         <Radio
-            alignment="left"
             label="Small"
             marginBottom='sm'
             name="size"
@@ -28,7 +27,6 @@ const RadioReactHook = () => {
         />
         <br />
         <Radio
-            alignment="left"
             label="Medium"
             marginBottom='sm'
             name="size"
@@ -37,7 +35,6 @@ const RadioReactHook = () => {
         />
         <br />
         <Radio
-            alignment="left"
             label="Large"
             marginBottom='sm'
             name="size"

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

@@ -10,7 +10,7 @@
 @import "./trix_styles";
 @import "./tiptap_styles";
 
-[class^=pb_rich_text_editor_kit] {
+.pb_rich_text_editor_kit {
   // Disables File Attatchment Button in Toolbar
   trix-toolbar [data-trix-button-group="file-tools"] {
     display: none;
@@ -126,7 +126,7 @@
   }
   // Trix doesn't provide customization to their icons.
   // https://github.com/basecamp/trix/issues/464#issuecomment-564683997
-  &[class*=dark] {
+  &.dark {
     // Dark SVG Icon Styles
     .trix-button--icon-bold::before {
       background-image: url("data:image/svg+xml,%3Csvg width='11px' height='14px' viewBox='0 0 11 14' version='1.1' xmlns='http://www.w3.org/2000/svg' xmlns:xlink='http://www.w3.org/1999/xlink'%3E%3Cg id='Symbols' stroke='none' stroke-width='1' fill='none' fill-rule='evenodd' fill-opacity='0.6'%3E%3Cg id='Rich-Text/Dark/Field-Only/Empty' transform='translate(-19.000000, -17.000000)' fill='%23FFFFFF' fill-rule='nonzero'%3E%3Cg id='Styling1' transform='translate(7.000000, 7.000000)'%3E%3Cpath d='M20.6,16.79 C21.565,16.115 22.25,15.025 22.25,14 C22.25,11.745 20.505,10 18.25,10 L12,10 L12,24 L19.04,24 C21.135,24 22.75,22.3 22.75,20.21 C22.75,18.69 21.885,17.395 20.6,16.79 Z M15,12.5 L18,12.5 C18.83,12.5 19.5,13.17 19.5,14 C19.5,14.83 18.83,15.5 18,15.5 L15,15.5 L15,12.5 L15,12.5 Z M18.5,21.5 L15,21.5 L15,18.5 L18.5,18.5 C19.33,18.5 20,19.17 20,20 C20,20.83 19.33,21.5 18.5,21.5 Z' id='Shape'%3E%3C/path%3E%3C/g%3E%3C/g%3E%3C/g%3E%3C/svg%3E");
@@ -291,14 +291,14 @@
         background-color: $focus_input_light;
       }
     }
-    &[class*=dark]:hover {
+    &.dark:hover {
       .trix-button-row,
       trix-editor {
         background-color: rgba($white, 0.1);
         border-color: rgba($white, 0.15);
       }
     }
-    &[class*=dark]:focus-within {
+    &.dark:focus-within {
       .trix-button-row,
       trix-editor {
         background-color: rgba($white, 0.03);

data/app/pb_kits/playbook/pb_section_separator/_section_separator.scss

@@ -18,7 +18,7 @@ $section_colors_dark: (
   card:         tint($bg_dark,10),
 );
 
-[class^=pb_section_separator_kit] {
+.pb_section_separator_kit {
   display: flex;
   justify-content: center;
   align-items: center;
@@ -26,7 +26,7 @@ $section_colors_dark: (
   span {
     padding: 0;
     display: flex;
-    [class*="pb_caption_kit"] {
+    .pb_caption_kit_md {
       padding: 0 $space_xs;
     }
   }
@@ -42,96 +42,115 @@ $section_colors_dark: (
     flex: 1;
     @include section_separator_horizontal;
   }
-  @each $color_name, $color_value in $section_selector_colors {
-    &[class*="color_#{$color_name}"] {
-      &::before, &::after {
+}
+
+// Horizontal orientation (default)
+.pb_section_separator_horizontal {
+  justify-content: center;
+}
+
+// Vertical orientation
+.pb_section_separator_vertical {
+  margin-left: $space_xs;
+  margin-right: $space_xs;
+  flex-direction: column;
+  &::before {
+    flex: 1;
+    @include section_separator_vertical(false);
+  }
+  &::after {
+    flex: 1;
+    @include section_separator_vertical(false);
+  }
+}
+
+// Dashed line style
+.pb_section_separator_dashed {
+  &::before, &::after {
+    @include section_separator_dashed(false);
+  }
+}
+
+// Color variations
+@each $color_name, $color_value in $section_selector_colors {
+  .pb_section_separator_color_#{$color_name} {
+    &::before, &::after {
+      background: $color_value;
+    }
+
+    &.pb_section_separator_vertical {
+      &::after {
         background: $color_value;
       }
-
-      &[class*=_vertical] {
-        &::after {
-          background: $color_value;
-        }
+    }
+    
+    &.pb_section_separator_dashed {
+      &::before, &::after {
+        border: 1px dashed $color_value;
+        background: none;
       }
-      &[class*=_dashed] {
+      &.pb_section_separator_vertical {
         &::before, &::after {
           border: 1px dashed $color_value;
-        }
-        &[class*=_vertical] {
-          &::after {
-            border: 1px dashed $color_value;
-            background: none;
-          }
+          background: none;
         }
       }
     }
   }
+}
 
-  &[class*=_horizontal] {
-    justify-content: center;
+// Dark mode styles
+.pb_section_separator_kit.dark {
+  &::before {
+    @include section_separator_horizontal(true);
   }
-  &[class*=_vertical] {
-    margin-left: $space_xs;
-    margin-right: $space_xs;
-    &::before {
-      display: none;
-    }
-    &::after {
-      @include section_separator_vertical(false);
-    }
+  &::after {
+    @include section_separator_horizontal(true);
   }
-
-  &[class*=_dashed] {
+  
+  &.pb_section_separator_vertical {
     &::before, &::after {
-      @include section_separator_dashed(false);
+      @include section_separator_vertical(true);
     }
   }
-
-  // Dark =========================
-
-  &.dark {
-    @each $color_name, $color_value in $section_selector_colors {
-      &[class*="color_#{$color_name}"] {
-        &::before, &::after {
-          background: $color_value;
-        }
-
-        &[class*=_vertical] {
-          &::after {
-            @include section_separator_vertical(false);
-            background: $color_value;
-          }
-        }
-
-        &[class*=_dashed] {
-          &::before, &::after {
-            border: 1px dashed $color_value;
-            background: none;
-          }
-          &[class*=_vertical] {
-            &::after {
-              border: 1px dashed $color_value;
-              background: none;
-            }
-          }
-        }
-      }
+  
+  &.pb_section_separator_dashed {
+    &::before, &::after {
+      @include section_separator_dashed(true);
     }
-    &::before {
-      @include section_separator_horizontal(true);
+    &.pb_section_separator_vertical {
+      &::before, &::after {
+        @include section_separator_dashed(true);
+      }
     }
-    &::after {
-      @include section_separator_horizontal(true);
+  }
+}
+
+// Dark mode color variations
+@each $color_name, $color_value in $section_selector_colors {
+  .pb_section_separator_kit.dark.pb_section_separator_color_#{$color_name} {
+    &::before, &::after {
+      background: $color_value;
     }
-    &[class*=_vertical] {
-      &::after {
-        @include section_separator_vertical(true);
+
+    &.pb_section_separator_vertical {
+      &::before, &::after {
+        @include section_separator_vertical(false);
+        background: $color_value;
       }
     }
-    &[class*=_dashed] {
+
+    &.pb_section_separator_dashed {
       &::before, &::after {
-        @include section_separator_dashed(true);
+        border: 1px dashed $color_value;
+        background: none;
+      }
+      &.pb_section_separator_vertical {
+        &::before, &::after {
+          border: 1px dashed $color_value;
+          background: none;
+        }
       }
     }
   }
-}
+}

data/app/pb_kits/playbook/pb_section_separator/_section_separator.tsx

@@ -39,7 +39,15 @@ const SectionSeparator = (props: SectionSeparatorProps): React.ReactElement => {
   const ariaProps = buildAriaProps(aria)
   const dataProps = buildDataProps(data)
   const htmlProps = buildHtmlProps(htmlOptions)
-  const classes = classnames(buildCss('pb_section_separator_kit', variant, orientation, lineStyle === "dashed" ? lineStyle : "", color !== "default" ? `color_${color}` : ''), globalProps(props), className)
+  const classes = classnames(
+    'pb_section_separator_kit',
+    `pb_section_separator_${variant}`,
+    `pb_section_separator_${orientation}`,
+    lineStyle === "dashed" ? 'pb_section_separator_dashed' : 'pb_section_separator_solid',
+    color !== "default" ? `pb_section_separator_color_${color}` : 'pb_section_separator_color_default',
+    globalProps(props),
+    className
+  )
   const dynamicInlineProps = globalInlineProps(props)
 
   return (

data/app/pb_kits/playbook/pb_section_separator/_section_separator_mixin.scss

@@ -30,4 +30,4 @@
         border: 1px dashed $border_dark;
         background: none;
     }
-}
+}

data/app/pb_kits/playbook/pb_section_separator/docs/_section_separator_children.html.erb

@@ -11,4 +11,27 @@
             padding_right: "xs" 
         }) %>
     <% end %>
+<% end %>
+
+<%= pb_rails("flex", props: { classname: "flex-container", margin_top: "lg", vertical: "stretch" }) do %>
+    <%= pb_rails("body", props: { classname: "flex-item" }) do %>
+        Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua
+    <% end %>
+    <%= pb_rails("section_separator", props: { orientation: "vertical" }) do %>
+        <%= pb_rails("card", props: {
+            border_radius: "rounded",
+            justify_content: "center",
+            padding: "none"
+        })  do %>
+            <%= pb_rails("caption", props: {
+                text: "TODAY",
+                size: "xs",
+                padding_left: "xs",
+                padding_right: "xs" 
+            }) %>
+        <% end %>
+    <% end %>
+    <%= pb_rails("body", props: { classname: "flex-item" }) do %>
+        Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua
+    <% end %>
 <% end %>

data/app/pb_kits/playbook/pb_section_separator/docs/_section_separator_children.jsx

@@ -2,8 +2,25 @@ import React from 'react'
 import Card from '../../pb_card/_card'
 import Caption from '../../pb_caption/_caption'
 import SectionSeparator from '../../pb_section_separator/_section_separator'
+import Flex from '../../pb_flex/_flex'
+import FlexItem from '../../pb_flex/_flex_item'
 
-const children = (
+const childrenHorizontal = (
+    <Card
+        borderRadius="rounded"
+        justifyContent="center"
+        padding="none"
+    >
+        <Caption
+            paddingLeft="xs"
+            paddingRight="xs"
+            size="xs"
+            text="TODAY"
+        />
+    </Card>
+)
+
+const childrenVertical = (
     <Card
         borderRadius="rounded"
         justifyContent="center"
@@ -20,12 +37,33 @@ const children = (
 
 const SectionSeparatorChildren = (props) => {
     return (
-        <SectionSeparator
-            {...props}
-            lineStyle='dashed'
-        >
-            {children}
-        </SectionSeparator>
+        <>
+            <SectionSeparator
+                {...props}
+                lineStyle='dashed'
+            >
+                {childrenHorizontal}
+            </SectionSeparator>
+            <Flex
+                inline="flex-container"
+                marginTop="lg"
+                vertical="stretch"
+            >
+                <FlexItem>
+                    {'Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua'}
+                </FlexItem>
+                    <SectionSeparator
+                        orientation="vertical"
+                        {...props}
+                    >
+                        {childrenVertical}
+                    </SectionSeparator>
+                <FlexItem>
+                    {'Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua'}
+                </FlexItem>
+            </Flex>
+        </>
+        
     )
 }
 

data/app/pb_kits/playbook/pb_section_separator/docs/_section_separator_color.jsx

@@ -5,7 +5,9 @@ import Detail from '../../pb_detail/_detail'
 import Icon from '../../pb_icon/_icon'
 
 const children = (
-  <Flex padding="xs">
+  <Flex alignItems="center"
+      padding="xs"
+  >
     <Icon color="primary"
         icon="arrow-down"
     />

data/app/pb_kits/playbook/pb_section_separator/section_separator.html.erb

@@ -7,7 +7,10 @@
         <% end %>
     <% end %>
     <% if object.orientation === 'vertical' %>
-        <%= pb_rails("flex", props: { orientation: "column"}) do %>
+        <% if content %>
+            <%= content %>
+        <% elsif object.text %>
+            <span><%= pb_rails("caption", props: { text: object.text }) %></span>
         <% end %>
     <%end%>
 <% end %>

data/app/pb_kits/playbook/pb_section_separator/section_separator.rb

@@ -20,7 +20,13 @@ module Playbook
                         default: "solid"
 
       def classname
-        generate_classname("pb_section_separator_kit", variant, orientation, line_style == "dashed" ? "dashed" : nil, color != "default" ? "color_#{color}" : nil)
+        class_names = ["pb_section_separator_kit"]
+        class_names << "pb_section_separator_#{variant}"
+        class_names << "pb_section_separator_#{orientation}"
+        class_names << "pb_section_separator_#{line_style}"
+        class_names << "pb_section_separator_color_#{color}"
+
+        generate_classname(class_names.join(" "), separator: " ")
       end
 
     private