playbook_ui 10.19.0.pre.popover.alpha1 → 10.19.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '090e97cafaabff3f06f58cca0ce4a1416d09f6baea9f828e51c93059f279f292'
-  data.tar.gz: 580359a862a8567240f9759e2ab80b708688d528ddc054789e6f658352cc21e1
+  metadata.gz: 2aee1f2c048d8521387c720ec1de805f4b141ac36d596a51cd785cda18964425
+  data.tar.gz: c66e833a6eae63f445b75cb3c89845a9096e57c45771c085102d1f2ee1b5dac0
 SHA512:
-  metadata.gz: 85c349d66339b000e6d7cb9077621b9e8d40852538d6e9eb7111947fabd0df9e11b3d2448a67609727c9fb915393ddf903140f43375be0947d5d749ffb6fb85a
-  data.tar.gz: b3774dfcbf0315c3b2fde5e145e7e0e33c3ceb23e5371aa01120ccca6f7294e7883e1f19c89dc73cdc3af2664d7b3484f0d9de5835119e862424902ee153bbc9
+  metadata.gz: e232f695466a3fa606ef20b3416d2c4684e613ae6a278c3d7c957e502dcebecb6ad28109db1422a89928451bee0df24728d2e0bdfc30143ca5422e314bfc6c45
+  data.tar.gz: 45904dac97847a942877f532d363552702f762bee32c97e096a346efeb32d0fb8e55e7fc7b81ebcf767a0fb1b95ec722669eb989fe85b7f4f13606d130e2a16a

data/app/pb_kits/playbook/pb_button/_button.jsx

@@ -22,7 +22,7 @@ type ButtonPropTypes = {
   loading?: boolean,
   newWindow?: boolean,
   onClick?: EventHandler,
-  size?: 'sm' | 'md' | 'lg',
+  size: 'large' | 'medium' | 'small',
   text?: string,
   type: 'inline' | null,
   htmlType: string | 'button',
@@ -36,16 +36,16 @@ const buttonClassName = (props: ButtonPropTypes) => {
     disabled = false,
     fullWidth = false,
     loading = false,
+    size = null,
     type = 'inline',
     variant = 'primary',
-    size = null,
   } = props
 
   let className = 'pb_button_kit'
 
-  className += `${size !== null ? `_${size}` : ''}`
   className += `${variant !== null ? `_${variant}` : ''}`
   className += `${type !== null ? `_${type}` : ''}`
+  className += `${size !== null ? `_${size}` : ''}`
   className += `${fullWidth ? '_block' : ''}`
   className += disabled ? '_disabled' : '_enabled'
   className += loading ? '_loading' : ''

data/app/pb_kits/playbook/pb_button/_button.scss

@@ -1,24 +1,7 @@
 @import "./button_mixins";
 
-$pb_button_sizes: (
-  "sm":   0.75rem,
-  "md":   0.875rem,
-  "lg":   1.125rem,
-);
-
 [class^=pb_button_kit]{
   @include pb_button;
-  // Size =================
-  @each $name, $size in $pb_button_sizes {
-    &[class*=_#{$name}] {
-      font-size: $size;
-      padding: calc(#{$size} / 2) calc(#{$size} * 2.42) !important;
-    }
-  }
-  &[class*=_sm] {
-    min-height: 0;
-  }
-
   // Variants =================
   &[class*=_primary]  {
     @include pb_button_primary;

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

@@ -18,9 +18,6 @@ module Playbook
       prop :text
       prop :type
       prop :value
-      prop :size, type: Playbook::Props::Enum,
-                  values: ["sm", "md", "lg", nil],
-                  default: nil
 
       def options
         {
@@ -47,12 +44,12 @@ module Playbook
         link ? "a" : "button"
       end
 
+    private
+
       def classname
-        generate_classname("pb_button_kit", size, variant, full_width_class, disabled_class, loading_class)
+        generate_classname("pb_button_kit", variant, full_width_class, disabled_class, loading_class)
       end
 
-    private
-
       def disabled_class
         disabled ? "disabled" : "enabled"
       end

data/app/pb_kits/playbook/pb_button/button.test.js

@@ -89,16 +89,3 @@ test('click event', async () => {
 
   expect(screen.getByText('clicked button!')).toBeInTheDocument()
 })
-
-test('size prop', () => {
-  render(
-    <Button
-        data={{ testid: 'size-test' }}
-        size="sm"
-    />
-  )
-
-  const kit = screen.getByTestId('size-test')
-
-  expect(kit).toHaveClass('pb_button_kit_sm_primary_inline_enabled')
-})

data/app/pb_kits/playbook/pb_button/docs/example.yml

@@ -7,7 +7,6 @@ examples:
   - button_block_content: Button Block Content
   - button_accessibility: Button Accessibility Options
   - button_options: Button Additional Options
-  - button_size: Button Size
   react:
   - button_default: Button Variants
   - button_full_width: Button Full Width
@@ -16,4 +15,3 @@ examples:
   - button_block_content: Button Block Content
   - button_accessibility: Button Accessibility Options
   - button_options: Button Additional Options (onClick)
-  - button_size: Button Size

data/app/pb_kits/playbook/pb_button/docs/index.js

@@ -5,4 +5,3 @@ export { default as ButtonLoading } from './_button_loading.jsx'
 export { default as ButtonBlockContent } from './_button_block_content.jsx'
 export { default as ButtonAccessibility } from './_button_accessibility.jsx'
 export { default as ButtonOptions } from './_button_options.jsx'
-export { default as ButtonSize } from './_button_size.jsx'

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

@@ -40,6 +40,7 @@ type DatePickerProps = {
   onChange: (String) => void,
   pickerId?: String,
   placeholder?: String,
+  plugins?: Boolean,
   type?: String,
   yearRange?: Array,
 }
@@ -73,6 +74,7 @@ const DatePicker = (props: DatePickerProps) => {
     onChange = () => {},
     pickerId,
     placeholder = 'Select Date',
+    plugins = false,
     yearRange = [ 1900, 2100 ],
   } = props
 
@@ -100,6 +102,7 @@ const DatePicker = (props: DatePickerProps) => {
       mode: mode,
       onChange: onChange,
       pickerId: pickerId,
+      plugins: plugins,
       yearRange: yearRange,
     })
   }, [])

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

@@ -3,6 +3,7 @@
 @import "../tokens/colors";
 @import "./sass_partials/flatpickr_styles";
 @import "./sass_partials/inline_styles";
+@import "./sass_partials/month_and_year_styles";
 
 [class^=pb_date_picker_kit] {
   .input_wrapper {

data/app/pb_kits/playbook/pb_date_picker/date_picker.rb

@@ -41,6 +41,8 @@ module Playbook
                        required: true
       prop :placeholder, type: Playbook::Props::String,
                          default: "Select Date"
+      prop :plugins, type: Playbook::Props::Boolean,
+                     default: false
       prop :required, type: Playbook::Props::Boolean,
                       default: false
       prop :year_range, type: Playbook::Props::Array,
@@ -64,6 +66,7 @@ module Playbook
           minDate: min_date,
           mode: mode,
           pickerId: picker_id,
+          plugins: plugins,
           required: required,
           yearRange: year_range,
         }.to_json.html_safe

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

@@ -1,4 +1,5 @@
 import flatpickr from 'flatpickr'
+import monthSelectPlugin from 'flatpickr/dist/plugins/monthSelect'
 
 const datePickerHelper = (config) => {
   const {
@@ -13,6 +14,7 @@ const datePickerHelper = (config) => {
     mode,
     onChange = () => {},
     pickerId,
+    plugins,
     required,
     yearRange,
   } = config
@@ -50,6 +52,10 @@ const datePickerHelper = (config) => {
     }
   }
 
+  const setMonthAndYearPlugin = () => (
+    plugins ? [ monthSelectPlugin({ shorthand: true, dateFormat: 'F Y', altFormat: 'F Y' }) ] : []
+  )
+
   // ===========================================================
   // |             Flatpickr initializer w/ config             |
   // ===========================================================
@@ -97,6 +103,7 @@ const datePickerHelper = (config) => {
     onYearChange: [() => {
       yearChangeHook()
     }],
+    plugins: setMonthAndYearPlugin(),
     prevArrow: '<i class="far fa-angle-left"></i>',
     static: true,
   })
@@ -153,8 +160,10 @@ const datePickerHelper = (config) => {
   }
 
   // Adding dropdown icons to year and month selects
-  picker.monthElements[0].insertAdjacentHTML('afterend', '<i class="far fa-angle-down month-dropdown-icon"></i>')
   dropdown.insertAdjacentHTML('afterend', '<i class="far fa-angle-down year-dropdown-icon" id="test-id"></i>')
+  if (picker.monthElements[0].parentElement) {
+    return picker.monthElements[0].insertAdjacentHTML('afterend', '<i class="far fa-angle-down month-dropdown-icon"></i>')
+  }
 
   // Remove readonly attribute for validation and or text input
   if (allowInput){

data/app/pb_kits/playbook/pb_date_picker/docs/_date_picker_month_and_year.html.erb

@@ -0,0 +1,5 @@
+<%= pb_rails("date_picker", props: {
+  label: "Date Picker",
+  plugins: true,
+  picker_id: "disabled_date"
+}) %>

data/app/pb_kits/playbook/pb_date_picker/docs/_date_picker_month_and_year.jsx

@@ -0,0 +1,18 @@
+import React from 'react'
+
+import DatePicker from '../_date_picker'
+
+const DatePickerMonthAndYear = (props) => {
+  return (
+    <div>
+      <DatePicker
+          label="Date Picker"
+          pickerId="disabled-date"
+          plugins
+          {...props}
+      />
+    </div>
+  )
+}
+
+export default DatePickerMonthAndYear

data/app/pb_kits/playbook/pb_date_picker/docs/_date_picker_month_and_year.md

@@ -0,0 +1 @@
+By default month&year plugin is disabled. To activate it set `plugins` prop to `true`.  If you're using React just pass a `plugins` prop to the kit.

data/app/pb_kits/playbook/pb_date_picker/docs/example.yml

@@ -16,6 +16,7 @@ examples:
   - date_picker_year_range: Year Range
   - date_picker_anti_patterns: Anti-Patterns
   - date_picker_inline: Inline
+  - date_picker_month_and_year: Month & Year Only
 
 
   react:
@@ -34,3 +35,4 @@ examples:
   - date_picker_hooks: Hooks
   - date_picker_year_range: Year Range
   - date_picker_inline: Inline
+  - date_picker_month_and_year: Month & Year Only

data/app/pb_kits/playbook/pb_date_picker/docs/index.js

@@ -13,3 +13,4 @@ export { default as DatePickerHooks } from './_date_picker_hooks.jsx'
 export { default as DatePickerFlatpickrMethods } from './_date_picker_flatpickr_methods.jsx'
 export { default as DatePickerYearRange } from './_date_picker_year_range.jsx'
 export { default as DatePickerInline } from './_date_picker_inline.jsx'
+export { default as DatePickerMonthAndYear } from './_date_picker_month_and_year.jsx'

data/app/pb_kits/playbook/pb_date_picker/sass_partials/_month_and_year_styles.scss

@@ -0,0 +1,127 @@
+@import "../../tokens/colors";
+
+
+.numInput {
+  border-left: none !important;
+}
+
+// Manual Import
+.flatpickr-monthSelect-months {
+  margin: 10px 1px 3px 1px;
+  flex-wrap: wrap;
+}
+
+.flatpickr-monthSelect-month {
+  background: none;
+  border: 1px solid transparent;
+  border-radius: 4px;
+  -webkit-box-sizing: border-box;
+  box-sizing: border-box;
+  color: $text_lt_default;
+  cursor: pointer;
+  display: inline-block;
+  font-weight: 400;
+  margin: 0.5px;
+  justify-content: center;
+  padding: 10px;
+  position: relative;
+  -webkit-box-pack: center;
+  -webkit-justify-content: center;
+  -ms-flex-pack: center;
+  text-align: center;
+  width: 33%;
+}
+
+.flatpickr-monthSelect-month.flatpickr-disabled {
+	color: #eee;
+}
+
+.flatpickr-monthSelect-month.flatpickr-disabled:hover,
+.flatpickr-monthSelect-month.flatpickr-disabled:focus {
+	cursor: not-allowed;
+	background: none !important;
+}
+
+.flatpickr-monthSelect-theme-dark {
+  background: #3f4458;
+}
+
+.flatpickr-monthSelect-theme-dark .flatpickr-current-month input.cur-year {
+  color: #fff;
+}
+
+.flatpickr-monthSelect-theme-dark .flatpickr-months .flatpickr-prev-month,
+.flatpickr-monthSelect-theme-dark .flatpickr-months .flatpickr-next-month {
+  color: #fff;
+  fill: #fff;
+}
+
+.flatpickr-monthSelect-theme-dark .flatpickr-monthSelect-month {
+  color: rgba(255, 255, 255, 0.95);
+}
+
+.flatpickr-monthSelect-month.today {
+  border-color: #959ea9;
+}
+
+.flatpickr-monthSelect-month.inRange,
+.flatpickr-monthSelect-month.inRange.today,
+.flatpickr-monthSelect-month:hover,
+.flatpickr-monthSelect-month:focus {
+  background: $active_light;
+  font-weight: $bold;
+  color: $text_lt_default;
+  cursor: pointer;
+  outline: 0;
+}
+
+.flatpickr-monthSelect-theme-dark .flatpickr-monthSelect-month.inRange,
+.flatpickr-monthSelect-theme-dark .flatpickr-monthSelect-month:hover,
+.flatpickr-monthSelect-theme-dark .flatpickr-monthSelect-month:focus {
+  background: #646c8c;
+  border-color: #646c8c;
+}
+
+.flatpickr-monthSelect-month.today:hover,
+.flatpickr-monthSelect-month.today:focus {
+  background: #959ea9;
+  border-color: #959ea9;
+  color: #fff;
+}
+
+.flatpickr-monthSelect-month.selected,
+.flatpickr-monthSelect-month.startRange,
+.flatpickr-monthSelect-month.endRange {
+  background-color: $primary;
+  font-weight: $bold;
+  box-shadow: none;
+  color: #fff;
+  border-color: $primary;
+}
+
+.flatpickr-monthSelect-month.startRange {
+  border-radius: 50px 0 0 50px;
+}
+
+.flatpickr-monthSelect-month.endRange {
+  border-radius: 0 50px 50px 0;
+}
+
+.flatpickr-monthSelect-month.startRange.endRange {
+  border-radius: 50px;
+}
+
+.flatpickr-monthSelect-month.inRange {
+  border-radius: 0;
+  box-shadow: -5px 0 0 #e6e6e6, 5px 0 0 #e6e6e6;
+}
+
+.flatpickr-monthSelect-theme-dark .flatpickr-monthSelect-month.selected,
+.flatpickr-monthSelect-theme-dark .flatpickr-monthSelect-month.startRange,
+.flatpickr-monthSelect-theme-dark .flatpickr-monthSelect-month.endRange {
+  background: #80cbc4;
+  -webkit-box-shadow: none;
+  box-shadow: none;
+  color: #fff;
+  border-color: #80cbc4;
+}

data/app/pb_kits/playbook/pb_image/_image.jsx

@@ -44,20 +44,18 @@ const Image = (props: ImageProps) => {
   const dataProps = buildDataProps(data)
 
   return (
-    <div>
-      <img
-          {...ariaProps}
-          {...dataProps}
-          alt={alt}
-          className={classes}
-          data-src={url}
-          id={id}
-          onError={onError}
-          rounded={+rounded}
-          src={url}
-          transition={transition}
-      />
-    </div>
+    <img
+        {...ariaProps}
+        {...dataProps}
+        alt={alt}
+        className={classes}
+        data-src={url}
+        id={id}
+        onError={onError}
+        rounded={+rounded}
+        src={url}
+        transition={transition}
+    />
   )
 }
 

data/app/pb_kits/playbook/pb_image/docs/_custom_error_image.html.erb

@@ -49,6 +49,7 @@
   props: {
     alt: "This is the alt text!",
     on_error: "this.classList.add('image-error')",
+    padding_x: "none",
     rounded: true,
     size: "xs",
     url: "not_a_picture"

data/app/pb_kits/playbook/pb_image/docs/_default_image.jsx

@@ -5,41 +5,54 @@ import Image from '../_image'
 const DefaultImage = (props) => {
   return (
     <>
-      <Image
-          alt="picture of a misty forest"
-          size="xs"
-          url="https://unsplash.it/500/400/?image=634"
-          {...props}
-      />
-      <Image
-          alt="picture of a misty forest"
-          size="sm"
-          url="https://unsplash.it/500/400/?image=634"
-          {...props}
-      />
-      <Image
-          alt="picture of a misty forest"
-          size="md"
-          url="https://unsplash.it/500/400/?image=634"
-          {...props}
-      />
-      <Image
-          alt="picture of a misty forest"
-          size="lg"
-          url="https://unsplash.it/500/400/?image=634"
-          {...props}
-      />
-      <Image
-          alt="picture of a misty forest"
-          size="xl"
-          url="https://unsplash.it/500/400/?image=634"
-          {...props}
-      />
-      <Image
-          alt="picture of a misty forest"
-          url="https://unsplash.it/500/400/?image=634"
-          {...props}
-      />
+      <br />
+      <div>
+        <Image
+            alt="picture of a misty forest"
+            size="xs"
+            url="https://unsplash.it/500/400/?image=634"
+            {...props}
+        />
+      </div>
+      <div>
+        <Image
+            alt="picture of a misty forest"
+            size="sm"
+            url="https://unsplash.it/500/400/?image=634"
+            {...props}
+        />
+      </div>
+      <div>
+        <Image
+            alt="picture of a misty forest"
+            size="md"
+            url="https://unsplash.it/500/400/?image=634"
+            {...props}
+        />
+      </div>
+      <div>
+        <Image
+            alt="picture of a misty forest"
+            size="lg"
+            url="https://unsplash.it/500/400/?image=634"
+            {...props}
+        />
+      </div>
+      <div>
+        <Image
+            alt="picture of a misty forest"
+            size="xl"
+            url="https://unsplash.it/500/400/?image=634"
+            {...props}
+        />
+      </div>
+      <div>
+        <Image
+            alt="picture of a misty forest"
+            url="https://unsplash.it/500/400/?image=634"
+            {...props}
+        />
+      </div>
     </>
   )
 }

data/app/pb_kits/playbook/pb_image/docs/_rounded_image.jsx

@@ -5,41 +5,52 @@ import Image from '../_image'
 const RoundedImage = (props) => {
   return (
     <>
-      <Image
-          alt=""
-          rounded
-          size="xs"
-          url="https://unsplash.it/500/400/?image=634"
-          {...props}
-      />
-      <Image
-          alt=""
-          rounded
-          size="sm"
-          url="https://unsplash.it/500/400/?image=634"
-          {...props}
-      />
-      <Image
-          alt=""
-          rounded
-          size="md"
-          url="https://unsplash.it/500/400/?image=634"
-          {...props}
-      />
-      <Image
-          alt=""
-          rounded
-          size="lg"
-          url="https://unsplash.it/500/400/?image=634"
-          {...props}
-      />
-      <Image
-          alt=""
-          rounded
-          size="xl"
-          url="https://unsplash.it/500/400/?image=634"
-          {...props}
-      />
+      <br />
+      <div>
+        <Image
+            alt=""
+            rounded
+            size="xs"
+            url="https://unsplash.it/500/400/?image=634"
+            {...props}
+        />
+      </div>
+      <div>
+        <Image
+            alt=""
+            rounded
+            size="sm"
+            url="https://unsplash.it/500/400/?image=634"
+            {...props}
+        />
+      </div>
+      <div>
+        <Image
+            alt=""
+            rounded
+            size="md"
+            url="https://unsplash.it/500/400/?image=634"
+            {...props}
+        />
+      </div>
+      <div>
+        <Image
+            alt=""
+            rounded
+            size="lg"
+            url="https://unsplash.it/500/400/?image=634"
+            {...props}
+        />
+      </div>
+      <div>
+        <Image
+            alt=""
+            rounded
+            size="xl"
+            url="https://unsplash.it/500/400/?image=634"
+            {...props}
+        />
+      </div>
     </>
   )
 }

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

@@ -12,13 +12,7 @@ import Title from '../pb_title/_title'
 type LegendProps = {
   aria?: object,
   className?: string,
-  color?: | "data_1"
-    | "data_2"
-    | "data_3"
-    | "data_4"
-    | "data_5"
-    | "data_6"
-    | "data_7",
+  color?: string,
   dark?: boolean,
   data?: object,
   id?: string,

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

@@ -20,5 +20,9 @@
 }
 
 [class^=pb_legend_kit] {
-  @include indicator-colors($data_colors)
-}
+  @include indicator-colors($data_colors);
+  @include indicator-colors($status_colors);
+  @include indicator-colors($product_colors);
+  @include indicator-colors($category_colors)
+
+}

data/app/pb_kits/playbook/pb_legend/docs/_legend_colors.html.erb

@@ -0,0 +1,4 @@
+<%= pb_rails("legend", props: { color: "data_8", text: "Windows" }) %>
+<%= pb_rails("legend", props: { color: "warning", text: "Windows" }) %>
+<%= pb_rails("legend", props: { color: "windows", text: "Windows" }) %>
+<%= pb_rails("legend", props: { color: "category_7", text: "Windows" }) %>

data/app/pb_kits/playbook/pb_legend/docs/_legend_colors.jsx

@@ -0,0 +1,29 @@
+import React from 'react'
+import { Legend } from '../..'
+
+const LegendColors = (props) => (
+  <div>
+    <Legend
+        color="data_8"
+        text="Windows"
+        {...props}
+    />
+    <Legend
+        color="warning"
+        text="Windows"
+        {...props}
+    />
+    <Legend
+        color="windows"
+        text="Windows"
+        {...props}
+    />
+    <Legend
+        color="category_7"
+        text="Windows"
+        {...props}
+    />
+  </div>
+)
+
+export default LegendColors

data/app/pb_kits/playbook/pb_legend/docs/_legend_colors.md

@@ -0,0 +1,2 @@
+By default legend kit uses `data_1` color.
+Pass the color prop and use any desired value from `$data_colors`, `$status_colors`, `$product_colors` and `$category_colors` list.

data/app/pb_kits/playbook/pb_legend/docs/_legend_default.html.erb

@@ -1,5 +1,5 @@
 <% labels = %W[Windows Doors Roofing Siding Solar Gutters Insulation Other] %>
 
-<% (1..7).each do |n, i| %>
-   <%= pb_rails("legend", props: { color: "data_#{n}", text: "#{labels[n]}" }) %>
+<% (0..7).each do |n, i| %>
+   <%= pb_rails("legend", props: { color: "data_#{n + 1}", text: "#{labels[n]}" }) %>
 <% end %>

data/app/pb_kits/playbook/pb_legend/docs/_legend_default.jsx

@@ -2,7 +2,7 @@ import React from 'react'
 
 import Legend from '../_legend'
 
-const products = ['Windows', 'Doors', 'Roofing', 'Siding', 'Solar Gutters', 'Insulation', 'Other']
+const products = ['Windows', 'Doors', 'Roofing', 'Siding', 'Solar', 'Gutters', 'Insulation', 'Other']
 
 const LegendDefault = (props) => (
   <div>

data/app/pb_kits/playbook/pb_legend/docs/example.yml

@@ -1,11 +1,11 @@
 examples:
-  
+
   rails:
   - legend_default: Default
   - legend_prefix: With Prefix Text
- 
-  
-  
+  - legend_colors: Colors
+
   react:
   - legend_default: Default
   - legend_prefix: With Prefix Text
+  - legend_colors: Colors

data/app/pb_kits/playbook/pb_legend/docs/index.js

@@ -1,2 +1,3 @@
 export { default as LegendDefault } from './_legend_default.jsx'
 export { default as LegendPrefix } from './_legend_prefix.jsx'
+export { default as LegendColors } from './_legend_colors'

data/app/pb_kits/playbook/pb_legend/legend.rb

@@ -3,8 +3,7 @@
 module Playbook
   module PbLegend
     class Legend < Playbook::KitBase
-      prop :color, type: Playbook::Props::Enum,
-                   values: (1..7).map { |n| "data_#{n}" },
+      prop :color, type: Playbook::Props::String,
                    default: "data_1"
       prop :prefix_text
       prop :text, required: true

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

@@ -0,0 +1,29 @@
+import React from 'react'
+import { render, screen } from '../utilities/test-utils'
+
+import Legend from './_legend'
+
+test('returns namespaced class name', () => {
+  render(
+    <Legend
+        data={{ testid: 'primary-test' }}
+        text="Test colors"
+    />
+  )
+
+  const kit = screen.getByTestId('primary-test')
+  expect(kit).toHaveClass('pb_legend_kit_data_1')
+})
+
+test('color prop', () => {
+  render(
+    <Legend
+        color="category_17"
+        data={{ testid: 'primary-test' }}
+        text="Test colors"
+    />
+  )
+
+  const kit = screen.getByTestId('primary-test')
+  expect(kit).toHaveClass('pb_legend_kit_category_17')
+})

data/app/pb_kits/playbook/pb_popover/_popover.jsx

@@ -150,14 +150,16 @@ export default class PbReactPopover extends React.Component<PbPopoverProps> {
       const targetIsReference =
         target.closest('.pb_popover_reference_wrapper') !== null
 
+      if (targetIsReference) return
+
       switch (closeOnClick) {
       case 'outside':
-        if (!targetIsPopover || targetIsReference) {
+        if (!targetIsPopover) {
           shouldClosePopover(true)
         }
         break
       case 'inside':
-        if (targetIsPopover || targetIsReference) {
+        if (targetIsPopover) {
           shouldClosePopover(true)
         }
         break

data/app/pb_kits/playbook/pb_popover/docs/_popover_close.html.erb

@@ -1,23 +1,23 @@
 <%= pb_rails("flex", props: {classname: "flex-container", spacing: "between"}) do %>
   <span>
-    <%= pb_rails("button", props: { text: "Click Inside", variant: "secondary", id: "inside-popover-1" }) %>
+    <%= pb_rails("button", props: { text: "Click Inside", variant: "secondary", id: 'inside-popover-1' }) %>
     <%= pb_rails("popover", props: {
       close_on_click: "inside",
       trigger_element_id: "inside-popover-1",
       tooltip_id: "inside-tooltip-1",
-      position: "bottom",
+      position: 'bottom',
       offset: true
     }) do %>
       Click on me!
     <% end %>
   </span>
   <span>
-    <%= pb_rails("button", props: { text: "Click Outside", variant: "secondary", id: "outside-popover-1" }) %>
+    <%= pb_rails("button", props: { text: "Click Outside", variant: "secondary", id: 'outside-popover-1' }) %>
     <%= pb_rails("popover", props: {
       close_on_click: "outside",
       trigger_element_id: "outside-popover-1",
       tooltip_id: "outside-tooltip-1",
-      position: "left",
+      position: 'left',
       offset: true
     }) do %>
         Click anywhere but me!
@@ -27,16 +27,16 @@
     <%= pb_rails("button", props: {
       text: "Click Anywhere",
       variant: "secondary",
-      id: "any-popover-1"
+      id: 'any-popover-1'
     }) %>
     <%= pb_rails("popover", props: {
       close_on_click: "any",
       trigger_element_id: "any-popover-1",
       tooltip_id: "any-tooltip-1",
-      position: "top",
+      position: 'top',
       offset: true
     }) do %>
       Click anything!
     <% end %>
   </span>
-<% end %>
+<% end %>

data/app/pb_kits/playbook/pb_popover/index.js

@@ -36,14 +36,15 @@ export default class PbPopover extends PbEnhancedElement {
       }
 
       setTimeout(() => {
-        this.toggleTooltip()
         this.popper.update()
+        this.toggleTooltip()
       }, 0)
     })
   }
 
   checkCloseTooltip() {
     document.querySelector('body').addEventListener('click', ({ target }) => {
+      const isTriggerElement = target.closest(`#${this.triggerElementId}`) !== null
       const isTooltipElement = target.closest(`#${this.tooltipId}`) !== null
 
       switch (this.closeOnClick) {
@@ -51,17 +52,21 @@ export default class PbPopover extends PbEnhancedElement {
         this.hideTooltip()
         break
       case 'outside':
-        if (!isTooltipElement) {
+        if (isTooltipElement) {
+          this.checkCloseTooltip()
+        } else {
           this.hideTooltip()
         }
         break
       case 'inside':
-        if (isTooltipElement) {
+        if (isTooltipElement || isTriggerElement) {
           this.hideTooltip()
+        } else {
+          this.checkCloseTooltip()
         }
         break
       }
-    }, { once: true, capture: true })
+    }, { once: true })
   }
 
   hideTooltip() {

data/app/pb_kits/playbook/pb_popover/popover.html.erb

@@ -3,7 +3,7 @@
   class: object.classname,
   data: object.data,
   id: object.id) do %>
-  <div class="pb_popover_tooltip hide" id="<%= object.tooltip_id %>" role="tooltip" style="<%= object.z_index_helper %>">
+  <div class="pb_popover_tooltip" id="<%= object.tooltip_id %>" role="tooltip" style="<%= object.z_index_helper %>">
     <div class="pb_popover_body <%= object.width_height_class_helper %> <%= object.popover_spacing_helper %>" style="<%= object.width_height_helper %>">
       <%= content.presence %>
     </div>

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

@@ -36,13 +36,15 @@ $section_colors_dark: (
     justify-content: center;
   }
   &[class*=_vertical] {
+    margin-left: $space_xs;
+    margin-right: $space_xs;
     &::after {
       @include section_separator_vertical(false);
     }
   }
-  
+
   // Dark =========================
-  
+
   &.dark {
     &::after {
       @include section_separator_horizontal(true);
@@ -59,6 +61,8 @@ $section_colors_dark: (
       }
     }
     &[class*=_vertical] {
+      margin-left: $space_xs;
+      margin-right: $space_xs;
       &::after {
         @include section_separator_vertical(true);
       }

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

@@ -10,8 +10,6 @@
     content: "";
     height: 100%;
     width: 1px;
-    margin-left: $space_xs;
-    margin-right: $space_xs;
     position: initial;
     z-index: 0;
 }

data/app/pb_kits/playbook/pb_text_input/_text_input.scss

@@ -92,13 +92,13 @@
       .add-on {
         &-right {
           .add-on-card {
-            border: 1px solid red;
+            border: 1px solid $error;
             border-left: 0;
           }
         }
         &-left {
           .add-on-card {
-            border: 1px solid red;
+            border: 1px solid $error;
             border-right: 0;
           }
         }
@@ -169,7 +169,7 @@
     &-card {
       height: 45px;
       margin: 0;
-      padding: 0;
+      padding: 0 !important;
       align-items: center;
       justify-content: center;
       width: 45px;

data/app/pb_kits/playbook/pb_title/title.html.erb

@@ -2,6 +2,5 @@
     aria: object.aria,
     id: object.id,
     data: object.data,
-    class: object.classname) do %>
-        <%= content.presence || object.text %>
-<% end %>
+    class: object.classname) do %><%= content.presence || object.text %><% end %>
+    

data/lib/playbook/version.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
 module Playbook
-  PREVIOUS_VERSION = "10.18.1"
-  VERSION = "10.19.0.pre.popover.alpha1"
+  PREVIOUS_VERSION = "10.18.0"
+  VERSION = "10.19.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: playbook_ui
 version: !ruby/object:Gem::Version
-  version: 10.19.0.pre.popover.alpha1
+  version: 10.19.0
 platform: ruby
 authors:
 - Power UX
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-01-24 00:00:00.000000000 Z
+date: 2022-01-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack
@@ -426,9 +426,6 @@ files:
 - app/pb_kits/playbook/pb_button/docs/_button_loading.md
 - app/pb_kits/playbook/pb_button/docs/_button_options.html.erb
 - app/pb_kits/playbook/pb_button/docs/_button_options.jsx
-- app/pb_kits/playbook/pb_button/docs/_button_size.html.erb
-- app/pb_kits/playbook/pb_button/docs/_button_size.jsx
-- app/pb_kits/playbook/pb_button/docs/_button_size.md
 - app/pb_kits/playbook/pb_button/docs/_footer.md
 - app/pb_kits/playbook/pb_button/docs/example.yml
 - app/pb_kits/playbook/pb_button/docs/index.js
@@ -669,6 +666,9 @@ files:
 - app/pb_kits/playbook/pb_date_picker/docs/_date_picker_label.md
 - app/pb_kits/playbook/pb_date_picker/docs/_date_picker_min_max.html.erb
 - app/pb_kits/playbook/pb_date_picker/docs/_date_picker_min_max.jsx
+- app/pb_kits/playbook/pb_date_picker/docs/_date_picker_month_and_year.html.erb
+- app/pb_kits/playbook/pb_date_picker/docs/_date_picker_month_and_year.jsx
+- app/pb_kits/playbook/pb_date_picker/docs/_date_picker_month_and_year.md
 - app/pb_kits/playbook/pb_date_picker/docs/_date_picker_on_change.jsx
 - app/pb_kits/playbook/pb_date_picker/docs/_date_picker_on_change.md
 - app/pb_kits/playbook/pb_date_picker/docs/_date_picker_range.html.erb
@@ -684,6 +684,7 @@ files:
 - app/pb_kits/playbook/pb_date_picker/sass_partials/_flatpickr_styles.scss
 - app/pb_kits/playbook/pb_date_picker/sass_partials/_header_styles.scss
 - app/pb_kits/playbook/pb_date_picker/sass_partials/_inline_styles.scss
+- app/pb_kits/playbook/pb_date_picker/sass_partials/_month_and_year_styles.scss
 - app/pb_kits/playbook/pb_date_picker/sass_partials/_overrides.scss
 - app/pb_kits/playbook/pb_date_range_inline/_date_range_inline.jsx
 - app/pb_kits/playbook/pb_date_range_inline/_date_range_inline.scss
@@ -1163,6 +1164,9 @@ files:
 - app/pb_kits/playbook/pb_legend/_legend.scss
 - app/pb_kits/playbook/pb_legend/docs/_description.md
 - app/pb_kits/playbook/pb_legend/docs/_footer.md
+- app/pb_kits/playbook/pb_legend/docs/_legend_colors.html.erb
+- app/pb_kits/playbook/pb_legend/docs/_legend_colors.jsx
+- app/pb_kits/playbook/pb_legend/docs/_legend_colors.md
 - app/pb_kits/playbook/pb_legend/docs/_legend_default.html.erb
 - app/pb_kits/playbook/pb_legend/docs/_legend_default.jsx
 - app/pb_kits/playbook/pb_legend/docs/_legend_prefix.html.erb
@@ -1171,6 +1175,7 @@ files:
 - app/pb_kits/playbook/pb_legend/docs/index.js
 - app/pb_kits/playbook/pb_legend/legend.html.erb
 - app/pb_kits/playbook/pb_legend/legend.rb
+- app/pb_kits/playbook/pb_legend/legend.test.js
 - app/pb_kits/playbook/pb_line_graph/_line_graph.jsx
 - app/pb_kits/playbook/pb_line_graph/_line_graph.scss
 - app/pb_kits/playbook/pb_line_graph/docs/_description.md
@@ -2125,9 +2130,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubygems_version: 3.1.6
 signing_key:

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

@@ -1,3 +0,0 @@
-<%= pb_rails("button", props: { text: "Button sm size", size: "sm" }) %>
-<%= pb_rails("button", props: { text: "Button md size", size: "md" }) %>
-<%= pb_rails("button", props: { text: "Button lg size", size: "lg" }) %>

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

@@ -1,26 +0,0 @@
-import React from 'react'
-import { Button } from '../../'
-
-const ButtonSize = (props) => (
-  <div>
-    <Button
-        size="sm"
-        text="Button sm size"
-        {...props}
-    />
-    {' '}
-    <Button
-        size="md"
-        text="Button md size"
-        {...props}
-    />
-    {' '}
-    <Button
-        size="lg"
-        text="Button lg size"
-        {...props}
-    />
-  </div>
-)
-
-export default ButtonSize

data/app/pb_kits/playbook/pb_button/docs/_button_size.md

@@ -1 +0,0 @@
-By default button has the `md` size style, even if you don't explicitly pass a size prop.