playbook_ui 8.0.3 → 8.1.0.pre.alpha1

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

@@ -0,0 +1,5 @@
+examples:
+  react:
+  - inline_default: Default
+  - inline_text_options: Text Options
+  - inline_input_options: Input Options

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

@@ -0,0 +1,3 @@
+export { default as InlineDefault } from './_inline_default.jsx'
+export { default as InlineTextOptions } from './_inline_text_options.jsx'
+export { default as InlineInputOptions } from './_inline_input_options.jsx'

data/app/pb_kits/playbook/pb_inline/inline.test.jsx

@@ -0,0 +1,21 @@
+import React from 'react'
+import { render, screen } from '../utilities/test-utils'
+import { Inline } from '../'
+
+/* See these resources for more testing info:
+  - https://github.com/testing-library/jest-dom#usage for useage and examples
+  - https://jestjs.io/docs/en/using-matchers
+*/
+
+test('generated scaffold test - update me', () => {
+  const testId = "test1"
+
+  render(
+    <Inline
+      data={{ testid: testId }}
+    />
+  )
+
+  const kit = screen.getByTestId(testId)
+  expect(kit).toBeInTheDocument()
+})

data/app/pb_kits/playbook/pb_nav/_item.jsx

@@ -75,6 +75,7 @@ const NavItem = (props: NavItemProps) => {
               key={iconLeft}
           >
             <Icon
+                aria={{ label: iconLeft }}
                 className="pb_nav_list_item_icon_left"
                 fixedWidth
                 icon={iconLeft}
@@ -90,6 +91,7 @@ const NavItem = (props: NavItemProps) => {
               key={iconRight}
           >
             <Icon
+                aria={{ label: iconRight }}
                 className="pb_nav_list_item_icon_right"
                 fixedWidth
                 icon={iconRight}

data/app/pb_kits/playbook/pb_nav/_nav.jsx

@@ -48,7 +48,7 @@ const Nav = (props: NavProps) => {
   )
 
   return (
-    <div
+    <nav
         {...ariaProps}
         {...dataProps}
         className={cardCss}
@@ -69,7 +69,7 @@ const Nav = (props: NavProps) => {
         </div>
       </If>
       <ul>{children}</ul>
-    </div>
+    </nav>
   )
 }
 

data/app/pb_kits/playbook/pb_nav/item.html.erb

@@ -9,13 +9,13 @@
       <%= pb_rails("image", props: { url: object.image_url, classname: "pb_nav_img_wrapper" }) %>
     <% end %>
     <% if object.icon_left %>
-      <%= pb_rails("icon", props: { icon: object.icon_left, classname: "pb_nav_list_item_icon_left", fixed_width: true}) %>
+      <%= pb_rails("icon", props: { aria: { label: object.icon_left }, icon: object.icon_left, classname: "pb_nav_list_item_icon_left", fixed_width: true}) %>
     <% end %>
     <span class="pb_nav_list_item_text">
       <%= object.text %><%= content.presence %>
     </span>
     <% if object.icon_right %>
-      <%= pb_rails("icon", props: { icon: object.icon_right, classname: "pb_nav_list_item_icon_right", fixed_width: true}) %>
+      <%= pb_rails("icon", props: { aria: { label: object.icon_right }, icon: object.icon_right, classname: "pb_nav_list_item_icon_right", fixed_width: true}) %>
     <% end %>
   <% end %>
 <% end %>

data/app/pb_kits/playbook/pb_nav/nav.html.erb

@@ -1,4 +1,4 @@
-<%= content_tag(:div,
+<%= content_tag(:nav,
     aria: object.aria,
     class: object.classname,
     data: object.data,

data/app/pb_kits/playbook/pb_online_status/_online_status.jsx

@@ -23,6 +23,8 @@ const OnlineStatus = (props: OnlineStatusProps) => {
     status = 'offline',
   } = props
 
+  aria.label = status
+
   const ariaProps = buildAriaProps(aria)
   const dataProps = buildDataProps(data)
   const classes = classnames(buildCss('pb_online_status_kit', status), globalProps(props), className)

data/app/pb_kits/playbook/pb_online_status/online_status.html.erb

@@ -1,5 +1,5 @@
 <%= content_tag(:div,
-    aria: object.aria,
+    aria: object.aria.merge!(label: object.status),
     class: object.classname,
     data: object.data,
     id: object.id) do %>

data/app/pb_kits/playbook/pb_progress_simple/_progress_simple.jsx

@@ -14,7 +14,7 @@ type ProgressSimpleProps = {
   muted: boolean,
   percent: string,
   value: number,
-  variant?: "default" | "positive" | "negative",
+  variant?: "default" | "positive" | "negative" | "warning",
   width: string,
 }
 

data/app/pb_kits/playbook/pb_progress_simple/_progress_simple.scss

@@ -33,6 +33,11 @@ $pb_progress_simple_height: 4px;
         background: $error;
       }
     }
+    &[class*=_warning] {
+      .progress_simple_value {
+        background: $warning;
+      }
+    }
 
   [class^=progress_simple_value] {
     width: 0%;

data/app/pb_kits/playbook/pb_progress_simple/docs/_progress_simple_variants.html.erb

@@ -7,3 +7,8 @@
 <br>
 
 <%= pb_rails("progress_simple", props: { percent: 10, variant: "negative" }) %>
+
+<br>
+
+<%= pb_rails("progress_simple", props: { percent: 40, variant: "warning" }) %>
+

data/app/pb_kits/playbook/pb_progress_simple/docs/_progress_simple_variants.jsx

@@ -19,6 +19,13 @@ const ProgressSimpleVariants = () => {
           percent={10}
           variant="negative"
       />
+
+      <br />
+
+      <ProgressSimple
+          percent={40}
+          variant="warning"
+      />
     </div>
   )
 }

data/app/pb_kits/playbook/pb_progress_simple/docs/_progress_simple_variants.md

@@ -1 +1 @@
-Progress Simple can pass colors - primary, green and red. Variants names are `default`, `positive`, and `negative` respectively.
+Progress Simple can pass colors - primary, green, red, and yellow. Variants names are `default`, `positive`, `negative`, and `warning`, respectively.

data/app/pb_kits/playbook/pb_progress_simple/progress_simple.rb

@@ -18,7 +18,7 @@ module Playbook
       # could this possibly be [sm, md, lg]?
       prop :width, default: "100%"
       prop :variant, type: Playbook::Props::Enum,
-                     values: %w[default positive negative],
+                     values: %w[default positive negative warning],
                      default: "default"
 
       def number_value

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

@@ -1,6 +1,6 @@
 /* @flow */
 
-import React, { useEffect, useRef } from 'react'
+import React, { forwardRef, useEffect, useRef } from 'react'
 import classnames from 'classnames'
 import useFocus from './useFocus.js'
 import Trix from 'trix'
@@ -22,7 +22,7 @@ type RichTextEditorProps = {
   value?: string,
 }
 
-const RichTextEditor = (props: RichTextEditorProps) => {
+const RichTextEditor = (props: RichTextEditorProps, ref: React.ElementRef<"input">) => {
   const {
     aria = {},
     className,
@@ -120,6 +120,7 @@ const RichTextEditor = (props: RichTextEditorProps) => {
       <input
           id={id}
           name={name}
+          ref={ref}
           type="hidden"
           value={value}
       />
@@ -133,4 +134,4 @@ const RichTextEditor = (props: RichTextEditorProps) => {
   )
 }
 
-export default RichTextEditor
+export default forwardRef(RichTextEditor)

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

@@ -19,6 +19,7 @@ type TextInputProps = {
   id?: string,
   name: string,
   label: string,
+  onBlur: (String) => void,
   onChange: (String) => void,
   placeholder: string,
   required?: boolean,
@@ -41,6 +42,7 @@ const TextInput = (
     id,
     name,
     label,
+    onBlur = () => {},
     onChange = () => {},
     placeholder,
     required,
@@ -79,6 +81,7 @@ const TextInput = (
               disabled={disabled}
               id={id}
               name={name}
+              onBlur={onBlur}
               onChange={onChange}
               placeholder={placeholder}
               ref={ref}

data/app/pb_kits/playbook/pb_textarea/_textarea.jsx

@@ -24,6 +24,7 @@ type TextareaProps = {
   required?: boolean,
   rows?: number,
   resize: 'none' | 'both' | 'horizontal' | 'vertical' | 'auto',
+  onBlur?: InputCallback<HTMLTextAreaElement>,
   onChange?: InputCallback<HTMLTextAreaElement>,
 }
 
@@ -37,6 +38,7 @@ const Textarea = ({
   label,
   maxCharacters,
   name,
+  onBlur = () => {},
   onChange = () => {},
   placeholder,
   required,
@@ -75,6 +77,7 @@ const Textarea = ({
             className="pb_textarea_kit"
             disabled={disabled}
             name={name}
+            onBlur={onBlur}
             onChange={onChange}
             placeholder={placeholder}
             ref={ref}

data/app/pb_kits/playbook/pb_time_stacked/_time_stacked.jsx

@@ -4,50 +4,67 @@ import React from 'react'
 import classnames from 'classnames'
 
 import DateTime from '../pb_kit/dateTime.js'
-import { buildCss } from '../utilities/props'
-import { Body, Caption } from '../'
+import { buildCss, buildDataProps } from '../utilities/props'
 import { deprecatedProps, globalProps } from '../utilities/globalProps.js'
 
+import { Body, Caption } from '../'
+
 type TimeStackedProps = {
   align?: 'left' | 'center' | 'right',
   className?: string | array<string>,
   dark?: boolean,
-  data?: string,
-  date: string,
+  data?: object,
+  date?: string,
   id?: string,
-  tag?: 'body' | 'caption',
+  time: number | Date,
+  timeZone?: string,
 }
 
-const TimeStacked = (props: TimeStackedProps) => {
-  const { align, className, dark, date } = props
-  deprecatedProps('TimeStacked', ['tag'])
+const TimeStackedDefault = (props: TimeStackedProps) => {
+  if (props.date) deprecatedProps('Time Stacked', ['date']) //date prop is deprecated, use time instead
+
+  const {
+    align = 'left',
+    className,
+    dark,
+    data = {},
+    date,
+    time,
+    timeZone,
+  } = props
+
   const classes = classnames(
     buildCss('pb_time_stacked_kit', align),
     globalProps(props),
-    className,
+    className
   )
+  const dataProps = buildDataProps(data)
 
-  const dateTimestamp = new DateTime({ value: date })
+  const dateTimestamp = new DateTime({ value: date ? date : new Date(time), zone: timeZone })
 
   return (
-    <div className={classes}>
-      <div
-          align={align}
-          className="pb_time_stacked_day_month"
+    <div
+        className={classes}
+        {...dataProps}
+    >
+      <Body
+          className={classnames('pb_time_stacked', 'time-spacing')}
+          color="light"
+          dark={dark}
       >
-        <Body
-            color="light"
-            dark={dark}
-            text={dateTimestamp.toTimeWithMeridian()}
-        />
-        <Caption
-            color="light"
-            dark={dark}
-            text={dateTimestamp.toTimezone()}
-        />
-      </div>
+        <time>
+          {dateTimestamp.toTimeWithMeridian()}
+          <Caption
+              className="pb_time_stacked"
+              color="light"
+              dark={dark}
+              tag="span"
+              text={dateTimestamp.toTimezone()}
+          />
+        </time>
+      </Body>
     </div>
   )
 }
 
-export default TimeStacked
+export default TimeStackedDefault

data/app/pb_kits/playbook/pb_time_stacked/_time_stacked.scss

@@ -1,3 +1,21 @@
 @import "../pb_title/title";
 @import "../pb_caption/caption";
 @import "../tokens/colors";
+
+[class^=pb_time_stacked_kit] {
+ &[class*=_center]  {
+   text-align: center;
+ }
+
+ &[class*=_right]  {
+   text-align: right;
+ }
+
+}
+
+.time-spacing {
+  time {
+    display: inline-grid;
+    margin-bottom: $space_xs - 6px;
+  }
+}

data/app/pb_kits/playbook/pb_time_stacked/docs/_time_stacked_default.html.erb

@@ -1,5 +1,18 @@
-<%= pb_rails("time_stacked", props: { date: DateTime.now }) %>
-<br/>
-<%= pb_rails("time_stacked", props: { align: "center", date: DateTime.now }) %>
-<br/>
-<%= pb_rails("time_stacked", props: { align: "right", date: DateTime.now }) %>
+<%= pb_rails("time_stacked", props: {
+  time: DateTime.now,
+  align: "left",
+}) %>
+
+<br>
+
+<%= pb_rails("time_stacked", props: {
+  time: Time.now,
+  align: "center"
+}) %>
+
+<br>
+
+<%= pb_rails("time_stacked", props: {
+  time: Time.now,
+  align: "right"
+}) %>

data/app/pb_kits/playbook/pb_time_stacked/docs/_time_stacked_default.jsx

@@ -5,18 +5,23 @@ const TimeStackedDefault = (props) => {
   return (
     <div>
       <TimeStacked
+          time={new Date()}
+          timeZone="America/New_York"
           {...props}
-          date={new Date()}
       />
+      <br />
       <TimeStacked
-          {...props}
           align="center"
-          date={new Date()}
+          time={new Date()}
+          timeZone="America/New_York"
+          {...props}
       />
+      <br />
       <TimeStacked
-          {...props}
           align="right"
-          date={new Date()}
+          time={new Date()}
+          timeZone="America/New_York"
+          {...props}
       />
     </div>
   )

data/app/pb_kits/playbook/pb_time_stacked/time_stacked.html.erb

@@ -1,8 +1,12 @@
 <%= content_tag(:div,
-    align: object.align,
     id: object.id,
     data: object.data,
     class: object.classname) do %>
-      <%= pb_rails("body", props: { text: object.format_time_string, color: "light" }) %>
-      <%= pb_rails("caption", props: { text: object.format_timezone, color: "light" }) %>
+
+    <%= pb_rails("body", props: { color: "light", classname: "pb_time_stacked time-spacing" }) do %>
+      <time>
+        <%= object.format_time_string %>
+        <%= pb_rails("caption", props: { color: "light", tag: "span", text: object.pb_date_time.to_timezone.upcase, classname: "pb_time_stacked" }) %>
+      </time>
+    <% end %>
 <% end %>

data/app/pb_kits/playbook/pb_time_stacked/time_stacked.rb

@@ -3,49 +3,27 @@
 module Playbook
   module PbTimeStacked
     class TimeStacked < Playbook::KitBase
+      prop :time, required: true
       prop :align, type: Playbook::Props::Enum,
                    values: %w[left center right],
                    default: "left"
-      prop :classnames, type: Playbook::Props::String,
-                        default: nil
-      prop :dark, type: Playbook::Props::Boolean,
-                  default: false
-      prop :date, type: Playbook::Props::Date, required: true
+      prop :timezone, default: "America/New_York"
 
       def classname
-        generate_classname("pb_time_stacked_kit", dark_class)
-      end
-
-      def day
-        day = Playbook::PbKit::PbDateTime.new(date)
-        content_tag(:time, datetime: day.to_iso) do
-          day.to_day.to_s
-        end
-      end
-
-      def month
-        month = Playbook::PbKit::PbDateTime.new(date)
-        content_tag(:time, datetime: month.to_iso) do
-          month.to_month.to_s
-        end
+        # convert deprecated prop values
+        generate_classname("pb_time_stacked_kit", align)
       end
 
       def format_time_string
-        "#{pb_date_time.to_full_hour}:#{pb_date_time.to_minutes}#{pb_date_time.to_meridian}"
+        "#{pb_date_time.to_hour}:#{pb_date_time.to_minutes}#{pb_date_time.to_meridian}"
       end
 
-      def format_timezone
-        pb_date_time.to_timezone
-      end
-
-    private
-
-      def dark_class
-        dark ? "dark" : nil
+      def format_timezone_string
+        pb_date_time.to_timezone.to_s
       end
 
       def pb_date_time
-        Playbook::PbKit::PbDateTime.new(date)
+        Playbook::PbKit::PbDateTime.new(time, timezone)
       end
     end
   end