playbook_ui 8.0.1 → 8.2.0.pre.alpha2

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_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_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

data/app/pb_kits/playbook/pb_time_stacked/time_stacked.test.js

@@ -0,0 +1,33 @@
+import { renderKit, screen } from '../utilities/test-utils'
+
+import TimeStacked from './_time_stacked'
+
+/*eslint no-multiple-empty-lines: 0*/
+
+test('returns the namespaced class', () => {
+  const props = {
+    data: { testid: 'default' },
+    time: new Date,
+  }
+
+  renderKit(TimeStacked, props)
+  expect(screen.getByTestId('default')).toHaveClass('pb_time_stacked_kit_left')
+
+  renderKit(TimeStacked, props, {
+    align: 'center',
+    data: { testid: 'center' },
+  })
+  expect(screen.getByTestId('center')).toHaveClass('pb_time_stacked_kit_center')
+
+  renderKit(TimeStacked, props, {
+    align: 'right',
+    data: { testid: 'right' },
+  })
+  expect(screen.getByTestId('right')).toHaveClass('pb_time_stacked_kit_right')
+
+  renderKit(TimeStacked, props, {
+    dark: true,
+    data: { testid: 'dark' },
+  })
+  expect(screen.getByTestId('dark')).toHaveClass('pb_time_stacked_kit_left dark')
+})

data/app/pb_kits/playbook/pb_tooltip/_tooltip.scss

@@ -46,7 +46,7 @@ $tooltip_shadow: rgba(60, 106, 172, 0.18);
       content: " ";
       position: absolute;
       top: 100%;
-      left: 50%;
+      left: calc(50% - 10px);
       border-color: $white transparent transparent transparent;
       border-style: solid;
       border-width: 10px;
@@ -95,9 +95,10 @@ $tooltip_shadow: rgba(60, 106, 172, 0.18);
     box-shadow: -8px 0 28px 0 $tooltip_shadow;
     margin: 0 0 0 $space_sm;
     .arrow {
-      left: -#{$space_xs};
-      top: $arrow_vertical_offset;
+      left: -18px;
+      right: auto;
       margin-bottom: 0;
+      top: $arrow_vertical_offset;
       transform: rotate(90deg);
     }
     &.flipped .arrow {

data/app/pb_kits/playbook/plugins/pb_chart.js

@@ -1,7 +1,7 @@
 import Highcharts from 'highcharts'
 
 import { highchartsTheme } from '../pb_dashboard/pbChartsLightTheme.js'
-import colors from '../tokens/_colors.scss'
+import colors from '../tokens/exports/_colors.scss'
 
 require('highcharts/modules/variable-pie')(Highcharts)
 import highchartsMore from 'highcharts/highcharts-more.js'

data/app/pb_kits/playbook/tokens/_border_radius.scss

@@ -32,13 +32,3 @@ $border_radius: (
   border_radius_none:    $border_radius_none,
   border_radius_rounded: $border_radius_rounded
 );
-
-
-:export {
-  @mixin export_border_radius($border_radius_list) {
-    @each $name, $size in $border_radius_list {
-      #{$name}: $size;
-    }
-  }
-  @include export_border_radius($border_radius);
-}

data/app/pb_kits/playbook/tokens/_colors.scss

@@ -250,28 +250,3 @@ $transparent: transparent;
   background: linear-gradient(135deg,  $start 0%, $end 100%);
   filter: progid:DXImageTransform.Microsoft.gradient(startColorstr='#{$start}', endColorstr='#{$end}', GradientType=1);
 }
-
-:export {
-  @mixin export-colors($colors-list) {
-    @each $name, $color in $colors-list {
-      #{$name}: $color;
-    }
-  }
-  @include export-colors($product_colors);
-  @include export-colors($status_colors);
-  @include export-colors($data_colors);
-  @include export-colors($shadow_colors);
-  @include export-colors($colors);
-  @include export-colors($interface_colors);
-  @include export-colors($main_colors);
-  @include export-colors($background_colors);
-  @include export-colors($card_colors);
-  @include export-colors($active_colors);
-  @include export-colors($action_colors);
-  @include export-colors($hover_colors);
-  @include export-colors($border_colors);
-  @include export-colors($text_colors);
-  @include export-colors($category_colors);
-  gradient_start: $gradient_start;
-  gradient_end: $gradient_end;
-}