playbook_ui 7.0.1.pre.alpha8 → 7.0.1.pre.alpha14

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

@@ -3,10 +3,8 @@ examples:
   rails:
   - selectable_icon_default: Default
   - selectable_icon_single_select: Single Select
-  - selectable_icon_dark: Dark
+
   
   react:
   - selectable_icon_default: Default
   - selectable_icon_single_select: Single Select
-  - selectable_icon_dark: Dark
-  

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

@@ -1,3 +1,2 @@
 export { default as SelectableIconDefault } from './_selectable_icon_default.jsx'
 export { default as SelectableIconSingleSelect } from './_selectable_icon_single_select.jsx'
-export { default as SelectableIconDark } from './_selectable_icon_dark.jsx'

data/app/pb_kits/playbook/pb_stat_change/_stat_change.jsx

@@ -33,9 +33,9 @@ const StatChange = (props: StatChangeProps) => {
     <If condition={value}>
       <div
           className={classnames(
-          className,
           buildCss('pb_stat_change_kit', status),
-          globalProps(props)
+          globalProps(props),
+          className
         )}
           id={id}
       >

data/app/pb_kits/playbook/pb_stat_value/_stat_value.jsx

@@ -21,11 +21,11 @@ const StatValue = (props: StatValueProps) => {
   } = props
 
   const displayValue = function(value) {
-    if (value) {
+    if (value || value === 0) {
       return (
         <Title
             size={1}
-            text={value}
+            text={`${value}`}
         />
       )
     }
@@ -44,7 +44,7 @@ const StatValue = (props: StatValueProps) => {
 
   return (
     <div
-        className={classnames('pb_stat_value_kit', className, globalProps(props))}
+        className={classnames('pb_stat_value_kit', globalProps(props), className)}
         id={id}
     >
       <div className="pb_stat_value_wrapper">

data/app/pb_kits/playbook/pb_stat_value/docs/_stat_value_default.jsx

@@ -3,7 +3,12 @@ import { StatValue } from '../../'
 
 const StatValueDefault = () => {
   return (
-    <StatValue value={1048} />
+    <>
+      <StatValue value={1048} />
+      <br />
+      <br />
+      <StatValue value={0} />
+    </>
   )
 }
 

data/app/pb_kits/playbook/pb_table/_table.jsx

@@ -44,7 +44,6 @@ const Table = (props: TableProps) => {
         {...ariaProps}
         {...dataProps}
         className={classnames(
-        className,
         'pb_table',
         `table-${size}`,
         `table-responsive-${responsive}`,
@@ -55,7 +54,8 @@ const Table = (props: TableProps) => {
           'single-line': singleLine,
           'no-hover': disableHover,
         },
-        globalProps(props)
+        globalProps(props),
+        className
       )}
         id={id}
     >

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

@@ -10,8 +10,7 @@ examples:
     - table_single_line: Force Single Line
     - table_data_table: Data Table
     - table_responsive_table: Responsive Tables
-    - table_dark: Table Dark
-    - table_container_dark: Table Container Off Dark
+ 
   react:
     - table_sm: Small
     - table_md: Medium
@@ -23,5 +22,3 @@ examples:
     - table_single_line: Force Single Line
     - table_data_table: Data Table
     - table_responsive_table: Responsive Tables
-    - table_dark: Table Dark
-    - table_container_dark: Table Container Off Dark

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

@@ -5,8 +5,6 @@ export { default as TableSideHighlight } from './_table_side_highlight.jsx'
 export { default as TableContainer } from './_table_container.jsx'
 export { default as TableDataTable } from './_table_data_table.jsx'
 export { default as TableResponsiveTable } from './_table_responsive_table.jsx'
-export { default as TableDark } from './_table_dark.jsx'
-export { default as TableContainerDark } from './_table_container_dark.jsx'
 export { default as TableDisableHover } from './_table_disable_hover.jsx'
 export { default as TableSingleLine } from './_table_single_line.jsx'
 export { default as TableMultiline } from './_table_multiline.jsx'

data/app/pb_kits/playbook/pb_table/styles/_hover.scss

@@ -19,14 +19,16 @@ $transition-speed: 0.2s;
             transition: all $transition-speed ease;
           }
 
-          &:hover {
-            box-shadow: 0 2px 10px 0 rgba($slate, $opacity-6);
+          @media (hover:hover) {
+            &:hover {
+              box-shadow: 0 2px 10px 0 rgba($slate, $opacity-6);
 
-            td {
-              background: $white;
-              border-color: darken($border_light, 10%);
-              border-top-width: 1px;
-              border-top-color: transparent;
+              td {
+                background: $white;
+                border-color: darken($border_light, 10%);
+                border-top-width: 1px;
+                border-top-color: transparent;
+              }
             }
           }
         }
@@ -38,11 +40,12 @@ $transition-speed: 0.2s;
               border-top-color: $border_light;
               border-top-width: 1px;
             }
-
-            &:hover {
-              td {
-                border-top-width: 1px;
-                border-top-color: $border_light;
+            @media (hover:hover) {
+              &:hover {
+                td {
+                  border-top-width: 1px;
+                  border-top-color: $border_light;
+                }
               }
             }
           }

data/app/pb_kits/playbook/pb_table/styles/_mobile.scss

@@ -91,10 +91,12 @@
           }
         }
       }
-      &:hover {
-        tbody {
-          tr {
-            box-shadow: 0 0 0 $white !important;
+      @media (hover:hover) {
+        &:hover {
+          tbody {
+            tr {
+              box-shadow: 0 0 0 $white !important;
+            }
           }
         }
       }

data/app/pb_kits/playbook/pb_table/styles/_table-dark.scss

@@ -42,10 +42,12 @@ $table-dark-card-bg: $bg_dark;
       &:not(.no-hover) {
         tbody {
           tr {
-            &:hover {
-              td {
-                border-color: $border_dark !important;
-                background: $bg_dark;
+            @media (hover:hover) {
+              &:hover {
+                td {
+                  border-color: $border_dark !important;
+                  background: $bg_dark;
+                }
               }
             }
           }

data/app/pb_kits/playbook/pb_table/table.rb

@@ -33,9 +33,9 @@ module Playbook
 
     private
 
-      def dark_class	
-        dark ? "table-dark" : nil	
-      end	
+      def dark_class
+        dark ? "table-dark" : nil
+      end
 
       def data_table_class
         data_table ? "data_table" : nil

data/app/pb_kits/playbook/pb_text_input/_text_input.html.erb

@@ -14,6 +14,7 @@
       <% end %>
     <% else %>
       <%= tag(:input,
+          autocomplete: object.autocomplete ? nil : "off",
           class: "text_input",
           data: object.validation_data,
           disabled: object.disabled,

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

@@ -13,6 +13,7 @@ type TextInputProps = {
   aria?: object,
   className: string,
   data?: object,
+  dark?: boolean,
   disabled?: boolean,
   error?: string,
   id?: string,
@@ -26,30 +27,32 @@ type TextInputProps = {
   children: Node,
 }
 
-const TextInput = ({
-  aria = {},
-  className,
-  data = {},
-  disabled,
-  error,
-  id,
-  name,
-  label,
-  onChange = () => {},
-  placeholder,
-  required,
-  type = 'text',
-  value,
-  children = null,
-  ...props
-}: TextInputProps) => {
+const TextInput = (props: TextInputProps) => {
+  const {
+    aria = {},
+    className,
+    data = {},
+    dark = false,
+    disabled,
+    error,
+    id,
+    name,
+    label,
+    onChange = () => {},
+    placeholder,
+    required,
+    type = 'text',
+    value,
+    children = null,
+  } = props
+
   const ariaProps = buildAriaProps(aria)
   const dataProps = buildDataProps(data)
   const css = classnames([
     'pb_text_input_kit',
-    className,
     error ? 'error' : null,
     globalProps(props),
+    className,
   ])
 
   return (
@@ -60,6 +63,7 @@ const TextInput = ({
     >
       <Caption
           className="pb_text_input_kit_label"
+          dark={dark}
           text={label}
       />
       <div className="text_input_wrapper">

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

@@ -13,17 +13,22 @@
     margin-bottom: $space_sm;
     display: block;
 
-    input::placeholder {
+    input::placeholder,
+    .text_input .placeholder {
       @include pb_body_light;
     }
 
-    > input:first-child {
+    input,
+    .text_input {
       @include pb_textarea_light;
       @include pb_title_4;
       overflow: hidden;
     }
 
-    input:focus, input:-webkit-autofill:focus {
+    input:focus,
+    .text_input:focus,
+    input:-webkit-autofill:focus,
+    .text_input:-webkit-autofill:focus {
       @include pb_textarea_focus;
       -webkit-box-shadow: 0 0 0px 1000px $focus_input_light inset;
       transition: background-color 5000s ease-in-out 0s;
@@ -34,18 +39,22 @@
     .text_input_wrapper {
       margin-bottom: 1rem;
 
-      input::placeholder {
+      input::placeholder,
+      .text_input .placeholder {
         @include pb_body_light_dark;
       }
 
-      > input:first-child {
+      input, .text_input {
         @include pb_textarea_dark;
         @include pb_title_4;
         @include pb_title_dark;
         overflow: hidden;
       }
 
-      input:focus, input:-webkit-autofill:focus {
+      input:focus,
+      .text_input:focus,
+      input:-webkit-autofill:focus,
+      .text_input:-webkit-autofill:focus {
         @include pb_textarea_focus_dark;
         -webkit-box-shadow: 0 0 0px 1000px $focus_input_dark inset;
         transition: background-color 5000s ease-in-out 0s;
@@ -54,7 +63,7 @@
 
     &.error {
       .text_input_wrapper {
-        > input:first-child {
+        input, .text_input {
           border-color: $error_dark;
         }
       }
@@ -66,7 +75,7 @@
       [class*=pb_body_kit] {
         margin-top: $space_xs / 2;
       }
-      > input:first-child {
+      input, .text_input {
         border-color: $error;
       }
     }

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

@@ -1,16 +1,11 @@
 examples:
   rails:
   - text_input_default: Default
-  - text_input_dark: Dark
   - text_input_error: With Error
-  - text_input_dark_error: Dark With Error
   - text_input_custom: Custom
   - text_input_disabled: Disabled
   react:
   - text_input_default: Default
-  - text_input_dark: Dark
   - text_input_error: With Error
-  - text_input_dark_error: Dark With Error
   - text_input_custom: Custom
   - text_input_disabled: Disabled
-

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

@@ -1,6 +1,4 @@
 export { default as TextInputDefault } from './_text_input_default.jsx'
-export { default as TextInputDark } from './_text_input_dark.jsx'
-export { default as TextInputDarkError } from './_text_input_dark_error.jsx'
 export { default as TextInputCustom } from './_text_input_custom.jsx'
 export { default as TextInputError } from './_text_input_error.jsx'
 export { default as TextInputDisabled } from './_text_input_disabled.jsx'

data/app/pb_kits/playbook/pb_text_input/text_input.rb

@@ -7,6 +7,8 @@ module Playbook
 
       partial "pb_text_input/text_input"
 
+      prop :autocomplete, type: Playbook::Props::Boolean,
+                          default: true
       prop :disabled, type: Playbook::Props::Boolean,
                       default: false
       prop :error

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

@@ -42,7 +42,7 @@ const Textarea = ({
 }: TextareaProps) => {
   const errorClass = error ? 'error' : null
   const resizeClass = `resize_${resize}`
-  const classes = classnames('pb_textarea_kit', className, errorClass, resizeClass, globalProps(props))
+  const classes = classnames('pb_textarea_kit', errorClass, resizeClass, globalProps(props), className)
 
   return (
     <div className={classes}>

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

@@ -3,17 +3,14 @@ examples:
   rails:
   - textarea_default: Default
   - textarea_custom: Custom Textarea Input
-  - textarea_dark: Dark
   - textarea_resize: Resize
   - textarea_error: Textarea w/ Error
-  - textarea_dark_error: Textarea w/ Error
+
 
 
 
   react:
   - textarea_default: Default
   - textarea_custom: Custom Textarea Input
-  - textarea_dark: Dark
   - textarea_resize: Resize
   - textarea_error: Textarea w/ Error
-  - textarea_dark_error: Textarea w/ Error

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

@@ -1,6 +1,4 @@
 export { default as TextareaDefault } from './_textarea_default.jsx'
 export { default as TextareaResize } from './_textarea_resize.jsx'
 export { default as TextareaCustom } from './_textarea_custom.jsx'
-export { default as TextareaDark } from './_textarea_dark.jsx'
 export { default as TextareaError } from './_textarea_error.jsx'
-export { default as TextareaDarkError } from './_textarea_dark_error.jsx'

data/app/pb_kits/playbook/pb_time/_time.html.erb

@@ -2,32 +2,38 @@
     id: object.id,
     data: object.data,
     class: object.classname) do %>
-  <% if object.size == "lg" %>
-    <%= pb_rails("title", props: { tag: "span", text: object.format_time_string, size: 3 }) %>
-    <%= content_tag(:span) do %>
+    <%
+      # convert deprecated prop values
+      size = object.size
+      size = "sm" if object.size == "xs"
+      size = "md" if object.size == "lg"
+    %>
+  <% if size == "md" %>
+    <% if object.show_icon %>
+      <%= pb_rails("body", props: { color: "light", tag: "span"}) do %>
+        <%= pb_rails("icon", props: { icon: "clock", fixed_width: true, size: "lg" }) %>
+      <% end %>
+    <% end %>
+    <%= pb_rails("caption", props: { tag: "span", text: object.format_time_string, size: "lg" }) %>
+    <% if object.show_timezone %>
       <%= content_tag(:span, class: "pb_time_timezone") do %>
         <%= object.pb_date_time.to_timezone.upcase %>
       <% end %>
     <% end %>
-  <% elsif object.size == "sm" %>
-    <%= pb_rails("body", props: { tag: "span", text: object.format_time_string }) do %>
-      <%= pb_rails("icon", props: { icon: "clock", fixed_width: true }) %>
-      <%= content_tag(:time, datetime: object.pb_date_time.to_iso) do %>
-        <%= content_tag(:span) do %>
-          <%= object.format_time_string %>
-          <%= content_tag(:span, class: "pb_time_timezone") do %>
-            <%= object.pb_date_time.to_timezone.upcase %>
-          <% end %>
-        <% end %>
+  <% else %>
+    <% if object.show_icon %>
+      <%= pb_rails("body", props: { color: "light", tag: "span"}) do %>
+        <%= pb_rails("icon", props: { icon: "clock", fixed_width: true }) %>
       <% end %>
     <% end %>
-  <% else %>
-    <%= pb_rails("body", props: { tag: "span", text: object.format_time_string }) do %>
+    <%= pb_rails("body", props: { color: "light", tag: "span", text: object.format_time_string }) do %>
       <%= content_tag(:time, datetime: object.pb_date_time.to_iso) do %>
         <%= content_tag(:span) do %>
           <%= object.format_time_string %>
-          <%= content_tag(:span, class: "pb_time_timezone") do %>
-            <%= object.pb_date_time.to_timezone.upcase %>
+          <% if object.show_timezone %>
+            <%= content_tag(:span, class: "pb_time_timezone") do %>
+              <%= object.pb_date_time.to_timezone.upcase %>
+            <% end %>
           <% end %>
         <% end %>
       <% end %>