playbook_ui 8.0.0.pre.alpha5 → 8.1.1

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

@@ -0,0 +1,9 @@
+examples:
+
+  rails:
+  - collapsible_default: Light
+
+  react:
+  - collapsible_default: Light
+  
+  

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

@@ -0,0 +1 @@
+export { default as CollapsibleDefault } from './_collapsible_default.jsx'

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

@@ -0,0 +1,79 @@
+import PbEnhancedElement from '../pb_enhanced_element'
+
+const MAIN_SELECTOR = '[data-collapsible-main]'
+const CONTENT_SELECTOR = '[data-collapsible-content]'
+const DOWN_ARROW_SELECTOR = '.fa-chevron-down'
+const UP_ARROW_SELECTOR = '.fa-chevron-up'
+
+export default class PbCollapsible extends PbEnhancedElement {
+  static get selector() {
+    return MAIN_SELECTOR
+  }
+
+  connect() {
+    this.element.addEventListener('click', () => {
+      this.toggleElement(this.target)
+    })
+    this.displayDownArrow()
+  }
+
+  get target() {
+    return this.element.parentNode.querySelector(CONTENT_SELECTOR)
+  }
+
+  showElement(elem) {
+  // Get the natural height of the element
+    const getHeight = () => {
+      elem.style.display = 'block'
+      const height = elem.scrollHeight + 'px' // Get it's height
+      elem.style.display = '' //  Hide it again
+      return height
+    }
+
+    const height = getHeight()
+    elem.classList.add('is-visible')
+    elem.style.height = height // Update the max-height
+
+    // Once the transition is complete, remove the inline max-height so the content can scale responsively
+    window.setTimeout(() => {
+      elem.style.height = ''
+    }, 300)
+  }
+
+  hideElement(elem) {
+    // Give the element a height to change from
+    elem.style.height = elem.scrollHeight + 'px'
+
+    window.setTimeout(() => {
+      elem.style.height = '0'
+      elem.style.paddingTop = '0'
+      elem.style.paddingBottom = '0'
+    }, 1)
+
+    // When the transition is complete, hide it
+    window.setTimeout(() => {
+      elem.classList.remove('is-visible')
+    }, 300)
+  }
+
+  toggleElement(elem) {
+    if (elem.classList.contains('is-visible')) {
+      this.hideElement(elem)
+      this.displayDownArrow()
+      return
+    }
+    // Otherwise, show it
+    this.showElement(elem)
+    this.displayUpArrow()
+  }
+
+  displayDownArrow() {
+    this.element.querySelector(DOWN_ARROW_SELECTOR).style.display = 'inline-block'
+    this.element.querySelector(UP_ARROW_SELECTOR).style.display = 'none'
+  }
+
+  displayUpArrow() {
+    this.element.querySelector(UP_ARROW_SELECTOR).style.display = 'inline-block'
+    this.element.querySelector(DOWN_ARROW_SELECTOR).style.display = 'none'
+  }
+}

data/app/pb_kits/playbook/pb_date_time_stacked/_date_time_stacked.jsx

@@ -2,6 +2,7 @@
 
 import React from 'react'
 import { DateStacked, Flex, FlexItem, TimeStacked } from '../'
+import { buildDataProps } from '../utilities/props'
 import { globalProps } from '../utilities/globalProps.js'
 
 type DateTimeStackedProps = {
@@ -12,12 +13,15 @@ type DateTimeStackedProps = {
 }
 
 const DateTimeStacked = (props: DateTimeStackedProps) => {
-  const { date, dark } = props
+  const { date, data = {}, dark } = props,
+    dataProps = buildDataProps(data)
+
   return (
     <Flex
         className={globalProps(props)}
         orientation="row"
         vertical="center"
+        {...dataProps}
     >
       <FlexItem>
         <DateStacked
@@ -32,8 +36,8 @@ const DateTimeStacked = (props: DateTimeStackedProps) => {
         <TimeStacked
             className="pb_date_time_stacked_kit"
             dark={dark}
-            date={date}
             tag="caption"
+            time={date}
         />
       </FlexItem>
     </Flex>

data/app/pb_kits/playbook/pb_date_time_stacked/date_time_stacked.html.erb

@@ -7,7 +7,7 @@
         <%= pb_rails("date_stacked", props: { align: "right", date: object.date, reverse: true, size: "sm", dark: object.dark }) %>
       <% end %>
       <%= pb_rails("flex/flex_item") do %>
-        <%= pb_rails("time_stacked", props: { classname: "pb_date_time_stacked_kit", date: object.date, tag: "caption" }) %>
+        <%= pb_rails("time_stacked", props: { classname: "pb_date_time_stacked_kit", time: object.date, tag: "caption" }) %>
       <% end %>
     <% end %>
 <% end %>

data/app/pb_kits/playbook/pb_docs/kit_example.rb

@@ -21,7 +21,7 @@ module Playbook
       end
 
       def description
-        @description ||= read_kit_file(kit, "_#{example_key}.md")
+        @description ||= read_kit_file("_#{example_key}.md")
       end
 
       def highlighter

data/app/pb_kits/playbook/pb_form/_form.scss

@@ -1,8 +1,12 @@
 .pb-form {
   .pb-form-actions {
     list-style: none;
-    margin: 8px;
+    margin: 0;
     padding: 0;
+    display: flex;
+    justify-content: space-between;
+    align-items: flex-start;
+    flex-direction: row;
 
     > li {
       display: inline-flex;

data/app/pb_kits/playbook/pb_form/docs/_form_form_with_validate.html.erb

@@ -13,8 +13,8 @@
   ]
 %>
 
-<%= pb_rails("form", props: {
-    form_system_options: { scope: :example, method: :get, url: "" },
+<%= pb_rails("form/form_with", props: {
+    scope: :example, method: :get, url: "",
     validate: true
   }) do |form| %>
   <%= form.text_field :example_text_field, props: { label: true, required: true } %>

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

@@ -19,8 +19,17 @@ module Playbook
         end
       end
 
-      def render_in(view_context, &_block)
-        super(view_context, &nil)
+      def render_in(view_context, &block)
+        form_kit.render_in(view_context, &block)
+      end
+
+    private
+
+      def form_kit
+        @form_system ||= begin
+          ::Playbook::KitResolver.resolve("form/#{form_system}")
+                                 .new(options: form_system_options, validate: validate, children: children)
+        end
       end
     end
   end

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

@@ -19,9 +19,11 @@ module Playbook
         prepend(CollectionSelectField)
         prepend(CheckboxField)
         prepend(DatePickerField)
+      end
 
-        def actions(&block)
-          @template.render_component ActionArea.new(submit_default_value: submit_default_value), &block
+      def actions
+        @template.send(:view_context).content_tag :ol, class: "pb-form-actions" do
+          yield ActionArea.new(@template, submit_default_value)
         end
       end
     end

data/app/pb_kits/playbook/pb_form/form_builder/action_area.rb

@@ -3,22 +3,29 @@
 module Playbook
   module PbForm
     module FormBuilder
-      class ActionArea < Playbook::KitBase
-        prop :submit_default_value, type: Playbook::Props::String
+      class ActionArea
+        def initialize(view_context, submit_default_value)
+          self.view_context = view_context
+          self.submit_default_value = submit_default_value
+        end
 
         def submit(value = nil, props: {})
           props[:type] ||= "submit"
+          props[:text] ||= value || submit_default_value
+
           button(value, props: props)
         end
 
         def button(value = nil, props:)
-          props[:text] ||= value || submit_default_value
-
-          content_tag(:li) do
-            pb_rails("button", props: props)
+          view_context.content_tag(:li) do
+            view_context.pb_rails("button", props: props)
           end
         end
+
+      private
+
+        attr_accessor :view_context, :submit_default_value
       end
     end
   end
-end
+end

data/app/pb_kits/playbook/pb_form/form_with.html.erb

@@ -1,18 +1,11 @@
-<%= form_with(id: id,
-              aria: aria,
-              class: classname,
-              data: data,
-              builder: form_builder,
-              **options) do |form| %>
-  <% instance_exec form, &children %>
+<%= form_with(form_options, &method(:render_form)) %>
 
-  <% if validate %>
-    <% content_for :pb_js do %>
-      <%= javascript_tag do %>
-        window.addEventListener('DOMContentLoaded', function() {
-          PbFormValidation.start()
-        })
-      <% end %>
+<% if validate %>
+  <% content_for :pb_js do %>
+    <%= javascript_tag do %>
+      window.addEventListener('DOMContentLoaded', function() {
+        PbFormValidation.start()
+      })
     <% end %>
   <% end %>
 <% end %>

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

@@ -14,16 +14,23 @@ module Playbook
         prop(:data).merge("pb-form-validation" => validate)
       end
 
-      def options
-        { url: "" }.merge(Hash(prop(:options)))
+      def form_options
+        {
+          url: "",
+          id: id,
+          aria: aria,
+          class: classname,
+          data: data,
+          builder: ::Playbook::PbForm::FormWith::FormBuilder,
+        }.merge(options)
       end
 
-      def classname
-        [generate_classname("pb-form"), options[:class]].join(" ")
+      def render_form(builder)
+        view_context.capture(builder, &children)
       end
 
-      def form_builder
-        ::Playbook::PbForm::FormWith::FormBuilder
+      def classname
+        [generate_classname("pb-form"), options[:class]].join(" ")
       end
 
       def render_in(view_context, &_block)

data/app/pb_kits/playbook/pb_form/simple_form.html.erb

@@ -1,6 +1,4 @@
-<%= simple_form_for model, options do |form| %>
-  <% instance_exec form, &children %>
-<% end %>
+<%= simple_form_for model, options, &method(:render_form) %>
 
 <% if validate %>
   <% content_for :pb_js do %>
@@ -10,4 +8,4 @@
       })
     <% end %>
   <% end %>
-<% end %>
+<% end %>

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

@@ -48,6 +48,10 @@ module Playbook
         prop(:options).fetch(:html, {})
       end
 
+      def render_form(builder)
+        view_context.capture(builder, &children)
+      end
+
       def render_in(view_context, &_block)
         super(view_context, &nil)
       end

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