playbook_ui 9.10.0.pre.date.time.stacked.1 → 9.13.0

data/app/pb_kits/playbook/pb_button/button.rb

@@ -21,11 +21,12 @@ module Playbook
 
       def options
         {
-          id: id,
-          data: data,
+          aria: aria,
           class: classname,
+          data: data,
           disabled: disabled,
-          aria: aria,
+          id: id,
+          role: "button",
           type: type,
           value: value,
         }.compact
@@ -34,6 +35,7 @@ module Playbook
       def link_options
         options.merge(
           href: link,
+          role: "link",
           target: new_window ? "_blank" : "_self"
         )
       end

data/app/pb_kits/playbook/pb_button/docs/_button_accessibility.html.erb

@@ -1 +1 @@
-<%= pb_rails("button", props: { text: "Button with ARIA", aria: {label: "button"}, tag: "a", link: "http://google.com" }) %>
+<%= pb_rails("button", props: { text: "Button with ARIA", aria: {label: "Go to Google"}, tag: "a", link: "http://google.com" }) %>

data/app/pb_kits/playbook/pb_button/docs/_button_accessibility.jsx

@@ -4,7 +4,7 @@ import { Button } from '../../'
 const ButtonAccessibility = (props) => (
   <div>
     <Button
-        aria={{ label: 'button' }}
+        aria={{ label: 'Go to Google' }}
         link="https://google.com"
         tag="a"
         text="Button with ARIA"

data/app/pb_kits/playbook/pb_button/docs/_button_link.html.erb

@@ -1,3 +1,3 @@
-<%= pb_rails("button", props: { text: "A Tag Button", tag: "a", link: "http://google.com" }) %>
-<%= pb_rails("button", props: { text: "Open in new Window", new_window: true, link: "http://google.com" }) %>
-<%= pb_rails("button", props: { text: "A Tag Button Disabled", disabled: true, link: "http://google.com" }) %>
+<%= pb_rails("button", props: { text: "A Tag Button", aria: { label: "Link to Google" }, tag: "a", link: "http://google.com" }) %>
+<%= pb_rails("button", props: { text: "Open in new Window", aria: { label: "Link to Google in new window" }, new_window: true, link: "http://google.com" }) %>
+<%= pb_rails("button", props: { text: "A Tag Button Disabled", aria: { label: "Disabled link to Google" }, disabled: true, link: "http://google.com" }) %>

data/app/pb_kits/playbook/pb_button/docs/_button_link.jsx

@@ -4,12 +4,14 @@ import { Button } from '../../'
 const ButtonLink = (props) => (
   <div>
     <Button
+        aria={{ label: 'Link to Google' }}
         link="https://google.com"
         text="A Tag Button"
         {...props}
     />
     {' '}
     <Button
+        aria={{ label: 'Link to Google in new window' }}
         link="https://google.com"
         newWindow
         text="Open in New Window"
@@ -17,6 +19,7 @@ const ButtonLink = (props) => (
     />
     {' '}
     <Button
+        aria={{ label: 'Disabled link to Google' }}
         disabled
         link="https://google.com"
         text="A Tag Button Disabled"

data/app/pb_kits/playbook/pb_button/docs/_button_loading.html.erb

@@ -1,3 +1,3 @@
-<%= pb_rails("button", props: { text: "Button Primary", loading: true }) %>
-<%= pb_rails("button", props: { text: "Button Primary", variant: "secondary", loading: true }) %>
-<%= pb_rails("button", props: { text: "Button Primary", variant: "link", loading: true }) %>
+<%= pb_rails("button", props: { aria: { label: "Loading" }, text: "Button Primary", loading: true }) %>
+<%= pb_rails("button", props: { aria: { label: "Loading" }, text: "Button Primary", variant: "secondary", loading: true }) %>
+<%= pb_rails("button", props: { aria: { label: "Loading" }, text: "Button Primary", variant: "link", loading: true }) %>

data/app/pb_kits/playbook/pb_button/docs/_button_loading.jsx

@@ -4,12 +4,14 @@ import { Button } from '../../'
 const ButtonLoading = (props) => (
   <div>
     <Button
+        aria={{ label: 'Loading' }}
         loading
         text="Button Primary"
         {...props}
     />
     {' '}
     <Button
+        aria={{ label: 'Loading' }}
         loading
         text="Button Secondary"
         variant="secondary"
@@ -17,6 +19,7 @@ const ButtonLoading = (props) => (
     />
     {' '}
     <Button
+        aria={{ label: 'Loading' }}
         loading
         text="A Tag Button Disabled"
         variant="link"

data/app/pb_kits/playbook/pb_circle_chart/circle_chart.rb

@@ -6,37 +6,36 @@ module Playbook
       prop :chart_data, type: Playbook::Props::Array,
                         default: []
       prop :style, type: Playbook::Props::Enum,
-                         values: %w[pie],
-                         default: "pie"
+                   values: %w[pie],
+                   default: "pie"
 
       prop :data_labels, type: Playbook::Props::Boolean, default: false
       prop :min_point_size, type: Playbook::Props::Numeric
       prop :max_point_size, type: Playbook::Props::Numeric
       prop :inner_size, type: Playbook::Props::Enum,
-                              values: %w[sm md lg none],
-                              default: "md"
+                        values: %w[sm md lg none],
+                        default: "md"
       prop :z_min, type: Playbook::Props::Numeric
       prop :start_angle, type: Playbook::Props::Numeric
       prop :header_format
-      prop :data_label_html, default: '<div>{point.name}</div>'
+      prop :data_label_html, default: "<div>{point.name}</div>"
       prop :tooltip_html, default: '<span style="font-weight: bold; color:{point.color};">●</span>
                                       {point.name}: ' + '<b>{point.y}
                                     </b>'
       prop :use_html, type: Playbook::Props::Boolean, default: false
       prop :legend, type: Playbook::Props::Boolean, default: false
-      prop :title, default: ''
+      prop :title, default: ""
       prop :rounded, type: Playbook::Props::Boolean, default: false
       prop :colors, type: Playbook::Props::Array,
                     default: []
 
-
       def chart_type
         style == "variablepie" ? "variablepie" : "pie"
       end
 
       def chart_data_formatted
-        chart_data.map{ |hash| hash[:y] = hash.delete :value}
-        return chart_data
+        chart_data.map { |hash| hash[:y] = hash.delete :value }
+        chart_data
       end
 
       def inner_size_format
@@ -57,7 +56,7 @@ module Playbook
       end
 
       def rounded_border_color
-        rounded == true ? 'null' : nil
+        rounded == true ? "null" : nil
       end
 
       def chart_options

data/app/pb_kits/playbook/pb_circle_icon_button/circle_icon_button.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Playbook
   module PbCircleIconButton
     class CircleIconButton < Playbook::KitBase
@@ -8,11 +10,11 @@ module Playbook
                      values: %w[primary secondary link],
                      default: "primary"
       prop :disabled, type: Playbook::Props::Boolean,
-                     default: false
+                      default: false
       prop :icon, required: true
       prop :link
       prop :new_window, type: Playbook::Props::Boolean,
-                     default: false
+                        default: false
       def classname
         generate_classname("pb_circle_icon_button_kit")
       end

data/app/pb_kits/playbook/pb_date_picker/_date_picker.jsx

@@ -25,6 +25,8 @@ type DatePickerProps = {
   id?: String,
   inputAria?: object,
   inputData?: object,
+  inputOnChange?: (String) => void,
+  inputValue?: any,
   label?: String,
   maxDate: String,
   minDate: String,
@@ -55,6 +57,8 @@ const DatePicker = (props: DatePickerProps) => {
     id,
     inputAria,
     inputData,
+    inputOnChange,
+    inputValue,
     label = 'Date Picker',
     maxDate,
     minDate,
@@ -114,7 +118,6 @@ const DatePicker = (props: DatePickerProps) => {
         className={classes}
         id={id}
     >
-      {className}
       <div className="input_wrapper">
         <TextInput
             aria={inputAria}
@@ -126,7 +129,9 @@ const DatePicker = (props: DatePickerProps) => {
             id={pickerId}
             label={hideLabel ? null : label}
             name={name}
+            onChange={inputOnChange}
             placeholder={placeholder}
+            value={inputValue}
         />
         <If condition={!hideIcon}>
           <div

data/app/pb_kits/playbook/pb_date_picker/date_picker_helper.js

@@ -166,6 +166,9 @@ const datePickerHelper = (config) => {
     picker.input.style.caretColor = 'transparent'
     picker.input.style.cursor = 'pointer'
   }
+
+  // Fix event bubbling bug on wrapper
+  document.querySelector(`#${pickerId}`).parentElement.addEventListener('click', (e) => e.stopPropagation())
 }
 
 export default datePickerHelper

data/app/pb_kits/playbook/pb_date_stacked/date_stacked.rb

@@ -5,13 +5,13 @@ module Playbook
     class DateStacked < Playbook::KitBase
       prop :date, type: Playbook::Props::Date, required: true
       prop :align, type: Playbook::Props::Enum,
-                  values: %w[left center right],
-                  default: "left"
+                   values: %w[left center right],
+                   default: "left"
       prop :size, type: Playbook::Props::Enum,
                   values: %w[sm md],
                   default: "sm"
       prop :reverse, type: Playbook::Props::Boolean,
-                  default: false
+                     default: false
       prop :dark, type: Playbook::Props::Boolean,
                   default: false
       prop :bold, type: Playbook::Props::Boolean,
@@ -22,13 +22,13 @@ module Playbook
       end
 
       def title_size
-        size == "md" ? 3: 4
+        size == "md" ? 3 : 4
       end
 
       def day
         day = Playbook::PbKit::PbDateTime.new(date)
         content_tag(:time, datetime: day.to_iso) do
-          "#{day.to_day}"
+          day.to_day.to_s
         end
       end
 
@@ -41,9 +41,8 @@ module Playbook
         year = Playbook::PbKit::PbDateTime.new(date).to_year.to_i
         if current_year != year
           content_tag(:time, datetime: year) do
-            "#{year}"
+            year.to_s
           end
-        else
         end
       end
 
@@ -64,7 +63,6 @@ module Playbook
       def dark_class
         dark ? "dark" : nil
       end
-
     end
   end
 end

data/app/pb_kits/playbook/pb_date_time_stacked/date_time_stacked.rb

@@ -5,15 +5,15 @@ module Playbook
     class DateTimeStacked < Playbook::KitBase
       prop :date, deprecated: true
       prop :date_time, type: Playbook::Props::Date,
-                  default: ::DateTime.current
+                       default: ::DateTime.current
       prop :dark, type: Playbook::Props::Boolean,
                   default: false
       prop :timezone, type: Playbook::Props::String,
-                  default: "America/New_York"
+                      default: "America/New_York"
 
-    def date_time_value
-      date || date_time
-    end
+      def date_time_value
+        date || date_time
+      end
     end
   end
-end
+end

data/app/pb_kits/playbook/pb_distribution_bar/distribution_bar.rb

@@ -7,9 +7,9 @@ module Playbook
                   values: %w[lg sm],
                   default: "lg"
       prop :widths, type: Playbook::Props::NumberArray,
-                  default: [1]
+                    default: [1]
       prop :colors, type: Playbook::Props::Array,
-                  default: []
+                    default: []
 
       def classname
         generate_classname("pb_distribution_bar", size)

data/app/pb_kits/playbook/pb_filter/filter.rb

@@ -3,7 +3,7 @@
 module Playbook
   module PbFilter
     class Filter < Playbook::KitBase
-      prop :filters, type: Playbook::Props::HashArray, default: [{name: ''}]
+      prop :filters, type: Playbook::Props::HashArray, default: [{ name: "" }]
       prop :sort_menu, type: Playbook::Props::HashArray, default: [{}]
       prop :results, type: Playbook::Props::Numeric
       prop :template, type: Playbook::Props::Enum,

data/app/pb_kits/playbook/pb_fixed_confirmation_toast/fixed_confirmation_toast.rb

@@ -8,7 +8,7 @@ module Playbook
                     default: "neutral"
       prop :text, type: Playbook::Props::String
       prop :multi_line, type: Playbook::Props::Boolean,
-                       default: false
+                        default: false
       prop :closeable, type: Playbook::Props::Boolean,
                        default: false
 

data/app/pb_kits/playbook/pb_flex/flex.rb

@@ -74,8 +74,8 @@ module Playbook
       def horizontal_class
         if orientation == "row"
           "justify_content_#{horizontal}"
-        else
-          "align_items_#{horizontal}" if align == "none"
+        elsif align == "none"
+          "align_items_#{horizontal}"
         end
       end
 
@@ -102,8 +102,8 @@ module Playbook
       def vertical_class
         if orientation == "row"
           "align_items_#{vertical}"
-        else
-          "justify_content_#{vertical}" if justify == "none"
+        elsif justify == "none"
+          "justify_content_#{vertical}"
         end
       end
 

data/app/pb_kits/playbook/pb_form/form.rb

@@ -23,7 +23,7 @@ module Playbook
           class: classname,
           data: data,
           validate: validate,
-        }.merge(prop(:options) || prop(:form_system_options) || { })
+        }.merge(prop(:options) || prop(:form_system_options) || {})
       end
     end
   end

data/app/pb_kits/playbook/pb_form_pill/_form_pill.jsx

@@ -14,6 +14,7 @@ type FormPillProps = {
   onClick?: EventHandler,
   avatar?: boolean,
   avatarUrl?: string,
+  size?: string,
   closeProps?: {
     onClick?: EventHandler,
     onMouseDown?: EventHandler,
@@ -21,11 +22,20 @@ type FormPillProps = {
   },
 }
 const FormPill = (props: FormPillProps) => {
-  const { className, text, name, onClick = () => {}, avatarUrl, closeProps = {} } = props
+  const {
+    className,
+    text,
+    name,
+    onClick = () => {},
+    avatarUrl,
+    closeProps = {},
+    size = '',
+  } = props
   const css = classnames(
     `pb_form_pill_kit_${'primary'}`,
     globalProps(props),
-    className
+    className,
+    size === 'small' ? 'small' : null,
   )
   return (
     <div className={css}>

data/app/pb_kits/playbook/pb_form_pill/_form_pill.scss

@@ -51,4 +51,23 @@ $form_pill_colors: (
       }
     }
   }
+  &.small {
+    height: fit-content;
+    height: -moz-fit-content;
+    .pb_form_pill_text, .pb_form_pill_close, .pb_form_pill_tag {
+      font-size: 16px;
+      font-weight: 400;
+    }
+    .pb_form_pill_text, .pb_form_pill_tag {
+      line-height: 1.7;
+      padding-left: $space_xs;
+      padding-right: 2px;
+    }
+    [class^=pb_avatar_kit], [class^=pb_avatar_kit] .avatar_wrapper {
+      width: 20px;
+      height: 20px;
+      flex-basis: 20px;
+      &::before { line-height: 21px; }
+    }
+  }
 }

data/app/pb_kits/playbook/pb_form_pill/docs/_form_pill_size.html.erb

@@ -0,0 +1,13 @@
+<%= pb_rails("form_pill", props: {
+   name: "Anna Black",
+   avatar_url: "https://randomuser.me/api/portraits/women/44.jpg",
+   size: "small",
+}) %>
+
+<br />
+<br />
+
+<%= pb_rails("form_pill", props: {
+   name: "Anna Black",
+   size: "small",
+}) %>

data/app/pb_kits/playbook/pb_form_pill/docs/_form_pill_size.jsx

@@ -0,0 +1,25 @@
+import React from 'react'
+import FormPill from '../_form_pill.jsx'
+
+const FormPillSize = (props) => {
+  return (
+
+    <div>
+      <FormPill
+          avatarUrl="https://randomuser.me/api/portraits/women/44.jpg"
+          name="Anna Black"
+          size="small"
+          {...props}
+      />
+      <br />
+      <br />
+      <FormPill
+          name="Anna Black"
+          size="small"
+          {...props}
+      />
+    </div>
+  )
+}
+
+export default FormPillSize