playbook_ui 6.2.1 → 6.3.0

data/app/pb_kits/playbook/pb_pill/_pill.html.erb

@@ -1,6 +1,7 @@
 <%= content_tag(:div,
-    id: object.id,
+    aria: object.aria,
+    class: object.classname,
     data: object.data,
-    class: object.classname) do %>
+    id: object.id) do %>
   <%= pb_rails("title", props: { text: object.display_text, size: 4, classname: "pb_pill_text" }) %>
 <% end %>

data/app/pb_kits/playbook/pb_pill/_pill.jsx

@@ -4,21 +4,39 @@ import React from 'react'
 
 import classnames from 'classnames'
 import Title from '../pb_title/_title.jsx'
+import { buildAriaProps, buildCss, buildDataProps } from '../utilities/props'
 import { globalProps } from '../utilities/globalProps.js'
 
 type PillProps = {
+  aria?: object,
   className?: string,
+  data?: object,
   id?: string,
   text: string,
   variant?: "success" | "warning" | "error" | "info" | "neutral",
 }
 
 const Pill = (props: PillProps) => {
-  const { className, text, variant = 'neutral' } = props
-  const css = classnames([`pb_pill_kit_${variant}`, className], globalProps(props))
+  const {
+    aria = {},
+    className,
+    data = {},
+    id,
+    text,
+    variant = 'neutral',
+  } = props
+
+  const ariaProps = buildAriaProps(aria)
+  const dataProps = buildDataProps(data)
+  const classes = classnames(buildCss('pb_pill_kit', variant), globalProps(props), className)
 
   return (
-    <div className={css}>
+    <div
+        {...ariaProps}
+        {...dataProps}
+        className={classes}
+        id={id}
+    >
       <Title
           className="pb_pill_text"
           size={4}

data/app/pb_kits/playbook/pb_pill/_pill.scss

@@ -15,7 +15,7 @@ $pb_pill_height: 22px;
 
   @each $color_name, $color_value in $status_color_text {
     &[class*=_#{$color_name}]  {
-      background: rgba($color_value, $opacity-1);
+        background: rgba($color_value, $opacity-1);
 
       .pb_pill_text {
         color: $color_value !important;

data/app/pb_kits/playbook/pb_pill/docs/_pill_dark.html.erb

@@ -0,0 +1,52 @@
+<%= pb_rails("pill", props: {
+  dark: true,
+  text: "Default"
+}) %>
+
+<br><br>
+
+<%= pb_rails("pill", props: {
+  dark: true,
+  text: "success",
+  variant: "success"
+}) %>
+
+<br><br>
+
+<%= pb_rails("pill", props: {
+  dark: true,
+  text: "error",
+  variant: "error"
+}) %>
+
+<br><br>
+
+<%= pb_rails("pill", props: {
+  dark: true,
+  text: "warning",
+  variant: "warning"
+}) %>
+
+<br><br>
+
+<%= pb_rails("pill", props: {
+  dark: true,
+  text: "info",
+  variant: "info"
+}) %>
+
+<br><br>
+
+<%= pb_rails("pill", props: {
+  dark: true,
+  text: "neutral",
+  variant: "neutral"
+}) %>
+
+<br><br>
+
+<%= pb_rails("pill", props: {
+  dark: true,
+  text: "primary",
+  variant: "primary"
+}) %>

data/app/pb_kits/playbook/pb_pill/docs/_pill_dark.jsx

@@ -0,0 +1,69 @@
+import React from 'react'
+import Pill from '../_pill.jsx'
+
+const PillDark = () => {
+  return (
+    <div>
+      <Pill
+          dark
+          text="default"
+      />
+
+      <br />
+      <br />
+
+      <Pill
+          dark
+          text="success"
+          variant="success"
+      />
+
+      <br />
+      <br />
+
+      <Pill
+          dark
+          text="error"
+          variant="error"
+      />
+
+      <br />
+      <br />
+
+      <Pill
+          dark
+          text="warning"
+          variant="warning"
+      />
+
+      <br />
+      <br />
+
+      <Pill
+          dark
+          text="info"
+          variant="info"
+      />
+
+      <br />
+      <br />
+
+      <Pill
+          dark
+          text="neutral"
+          variant="neutral"
+      />
+
+      <br />
+      <br />
+
+      <Pill
+          dark
+          text="primary"
+          variant="primary"
+      />
+    </div>
+  )
+}
+
+export default PillDark

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

@@ -2,8 +2,10 @@ examples:
   
   rails:
   - pill_default: Default
+  - pill_dark: Dark
   
   
   react:
   - pill_default: Default
-  
+  - pill_dark: Dark
+  

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

@@ -1 +1,2 @@
 export { default as PillDefault } from './_pill_default.jsx'
+export { default as PillDark } from './_pill_dark.jsx'

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

@@ -1,7 +1,8 @@
 <%= content_tag(:div,
-  id: object.id,
+  aria: object.aria,
+  class: object.classname,
   data: object.data,
-  class: object.classname) do %>
+  id: object.id) do %>
     <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 %>">
         <%= capture(&object.children) %>

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

@@ -10,14 +10,17 @@ import {
   Reference as PopperReference,
 } from 'react-popper'
 
-import { buildCss, noop } from '../utilities/props'
+import { buildAriaProps, buildCss, buildDataProps, noop } from '../utilities/props'
 
 import classnames from 'classnames'
 import { globalProps } from '../utilities/globalProps.js'
 
 type PbPopoverProps = {
+  aria?: object,
   className?: string,
   closeOnClick?: "outside" | "inside",
+  data?: object,
+  id?: String,
   offset?: boolean,
   reference: PopperReference,
   show?: boolean,
@@ -45,8 +48,11 @@ const popoverModifiers = ({ modifiers, offset }) => {
 
 const Popover = (props: PbPopoverProps) => {
   const {
-    children,
+    aria = {},
     className,
+    children,
+    data = {},
+    id,
     modifiers,
     offset,
     placement,
@@ -58,7 +64,7 @@ const Popover = (props: PbPopoverProps) => {
     minWidth,
   } = props
 
-  const popoverSpacing = globalProps(props) ? globalProps(props) : 'p_sm'
+  const popoverSpacing = globalProps(props).includes('dark') || !globalProps(props) ? 'p_sm' : globalProps(props)
   const overflowHandling = maxHeight || maxWidth ? 'overflow_handling' : ''
   const zIndexStyle = zIndex ? { zIndex: zIndex } : {}
   const widthHeightStyles = () => {
@@ -70,6 +76,9 @@ const Popover = (props: PbPopoverProps) => {
       minWidth ? { minWidth: minWidth } : {}
     )
   }
+  const ariaProps = buildAriaProps(aria)
+  const dataProps = buildDataProps(data)
+  const classes = classnames(buildCss('pb_popover_kit'), className, globalProps(props))
 
   return (
     <Popper
@@ -80,8 +89,11 @@ const Popover = (props: PbPopoverProps) => {
       {({ placement, ref, style }) => {
         return (
           <div
-              className={`${buildCss('pb_popover_kit')} ${className}`}
+              {...ariaProps}
+              {...dataProps}
+              className={classes}
               data-placement={placement}
+              id={id}
               ref={ref}
               style={Object.assign(
                 {},

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

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

data/app/pb_kits/playbook/pb_popover/docs/_popover_close_dark.jsx

@@ -0,0 +1,102 @@
+import React, { useState } from 'react'
+import {
+  Button,
+  Flex,
+  PbReactPopover,
+} from '../..'
+
+const PopoverCloseDark = () => {
+  const [showInsidePopover, setInsideShowPopover] = useState(false)
+  const [showOutsidePopover, setOutsideShowPopover] = useState(false)
+  const [showAnyPopover, setAnyShowPopover] = useState(false)
+
+  const handleInsideShouldClosePopover = (shouldClosePopover) => {
+    setInsideShowPopover(!shouldClosePopover)
+  }
+
+  const handleInsideTogglePopover = () => {
+    setInsideShowPopover(!showInsidePopover)
+  }
+
+  const handleOutsideShouldClosePopover = (shouldClosePopover) => {
+    setOutsideShowPopover(!shouldClosePopover)
+  }
+
+  const handleOutsideTogglePopover = () => {
+    setOutsideShowPopover(!showOutsidePopover)
+  }
+
+  const handleAnyShouldClosePopover = (shouldClosePopover) => {
+    setAnyShowPopover(!shouldClosePopover)
+  }
+
+  const handleAnyTogglePopover = () => {
+    setAnyShowPopover(!showOutsidePopover)
+  }
+
+  const insidePopoverTrigger = (
+    <Button
+        dark
+        onClick={handleInsideTogglePopover}
+        text="Click Inside"
+        variant="primary"
+    />
+  )
+
+  const outsidePopoverTrigger = (
+    <Button
+        dark
+        onClick={handleOutsideTogglePopover}
+        text="Click Outside"
+        variant="primary"
+    />
+  )
+
+  const anyPopoverTrigger = (
+    <Button
+        dark
+        onClick={handleAnyTogglePopover}
+        text="Click Anywhere"
+        variant="primary"
+    />
+  )
+
+  return (
+    <Flex spacing="between">
+      <PbReactPopover
+          closeOnClick="inside"
+          offset
+          placement="bottom"
+          reference={insidePopoverTrigger}
+          shouldClosePopover={handleInsideShouldClosePopover}
+          show={showInsidePopover}
+      >
+        {'Click on me!'}
+      </PbReactPopover>
+
+      <PbReactPopover
+          closeOnClick="outside"
+          offset
+          placement="top"
+          reference={outsidePopoverTrigger}
+          shouldClosePopover={handleOutsideShouldClosePopover}
+          show={showOutsidePopover}
+      >
+        {'Click anywhere but me!'}
+      </PbReactPopover>
+
+      <PbReactPopover
+          closeOnClick="any"
+          offset
+          placement="right"
+          reference={anyPopoverTrigger}
+          shouldClosePopover={handleAnyShouldClosePopover}
+          show={showAnyPopover}
+      >
+        {'Click anything!'}
+      </PbReactPopover>
+    </Flex>
+  )
+}
+
+export default PopoverCloseDark

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

@@ -0,0 +1,25 @@
+<%= pb_rails("flex", props: {
+    dark: true,
+    orientation: "row",
+    vertical: "center"
+    }) do %>
+    <%= pb_rails("body", props: { 
+        dark: true, 
+        text: "Click info for more details" 
+    }) %>
+    &nbsp;
+    <%= pb_rails("circle_icon_button", props: {
+        dark: true,
+        variant: "secondary",
+        icon: "info",
+        id: "regular-popover-5"
+    }) %>
+    <%= pb_rails("popover", props: {
+        trigger_element_id: "regular-popover-5",
+        tooltip_id: "tooltip-5",
+        offset: true,
+        position: "top",
+    }) do %>
+        I'm a popover. I can show content of any size.
+    <% end %>
+<% end %>