RubyGems - playbook_ui - Versions diffs - 4.17.0.pre.alpha1 → 5.0.1.pre.alpha1 - Mend

playbook_ui 4.17.0.pre.alpha1 → 5.0.1.pre.alpha1

Files changed (322) hide show

data/app/pb_kits/playbook/pb_person/_person.html.erb CHANGED

@@ -1,7 +1,14 @@
 <%= content_tag(:div,
+    aria: object.aria,
     id: object.id,
     data: object.data,
     class: object.classname) do %>
-      <%= pb_rails("body", props: { tag: "span", classname: "pb_person_first", children: -> { object.first_name } }) %>
-      <%= pb_rails("title", props: { text: object.last_name, size: 4 }) if object.last_name %>
+      <%= pb_rails("body", props: {
+        tag: "span",
+        classname: "pb_person_first",
+        children: -> { object.first_name
+      } }) %>
+      <%= pb_rails("title", props: {
+        text: object.last_name,
+        size: 4 }) if object.last_name %>
 <% end %>

data/app/pb_kits/playbook/pb_person/_person.jsx CHANGED

@@ -2,40 +2,58 @@
 import React from 'react'
 import classnames from 'classnames'
+import { spacing } from '../utilities/spacing.js'
 import {
-  Body,
-  Title,
-} from '../'
+  buildAriaProps,
+  buildCss,
+  buildDataProps,
+} from '../utilities/props'
+import { Body, Title } from '../'
 type PersonProps = {
+  aria?: object,
   className?: String | Array<String>,
-  dark?: Boolean,
+  data?: object,
   firstName: String,
+  id?: String,
   lastName: String,
 }
-const Person = ({
-  className,
-  dark = false,
-  firstName,
-  lastName,
-}: PersonProps) => {
+const Person = (props: PersonProps) => {
+  const {
+    aria = {},
+    className,
+    data = {},
+    firstName,
+    id,
+    lastName } = props
+  const ariaProps = buildAriaProps(aria)
+  const dataProps = buildDataProps(data)
+  const classes = classnames(buildCss('pb_person_kit'), className, spacing(props))
   return (
-    <div className={classnames('pb_person_kit', className)}>
+    <div
+        {...ariaProps}
+        {...dataProps}
+        className={classes}
+        id={id}
+    >
       <Body
           className="pb_person_first"
-          dark={dark}
           tag="span"
       >
         {firstName}
       </Body>
-      <Title
-          className="pb_person_first"
-          dark={dark}
-          size={4}
-          text={` ${lastName}`}
-      />
+      <If condition={lastName}>
+        <Title
+            className="pb_person_first"
+            size={4}
+            text={` ${lastName}`}
+        />
+      </If>
     </div>
   )
 }

data/app/pb_kits/playbook/pb_person/docs/_person_default.html.erb CHANGED

@@ -1 +1,5 @@
-<%= pb_rails("person", props: { first_name: "Kyle", last_name: "Fadigan" }) %>
+<%= pb_rails("person",
+    props: {
+      first_name: "Kyle",
+      last_name: "Fadigan"
+      }) %>

data/app/pb_kits/playbook/pb_person_contact/_person_contact.html.erb CHANGED

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

data/app/pb_kits/playbook/pb_person_contact/_person_contact.jsx CHANGED

@@ -3,7 +3,15 @@
 import React from 'react'
 import classnames from 'classnames'
-import { Contact, Person } from '../'
+import {
+  buildAriaProps,
+  buildCss,
+  buildDataProps,
+} from '../utilities/props'
+import { spacing } from '../utilities/spacing.js'
+import { Caption, Contact, Person } from '../'
 type ContactItem = {
   contactType: String,
@@ -12,33 +20,77 @@ type ContactItem = {
 }
 type PersonContactProps = {
+  aria?: object,
   className?: String | Array<String>,
   dark?: Boolean,
+  data?: object,
   firstName: String,
+  id?: String,
   lastName: String,
   contacts?: Array<ContactItem>,
 }
-const PersonContact = ({
-  className,
-  firstName,
-  lastName,
-  contacts,
-}: PersonContactProps) => (
-  <div className={classnames('pb_person_contact_kit', className)}>
-    <Person
-        firstName={firstName}
-        lastName={lastName}
-    />
-    {contacts.map((contactObject, index) => (
-      <Contact
-          contactDetail={contactObject.contactDetail}
-          contactType={contactObject.contactType}
-          contactValue={contactObject.contactValue}
-          key={index}
+const PersonContact = (props: PersonContactProps) => {
+  const {
+    aria = {},
+    className,
+    contacts = [],
+    data = {},
+    firstName,
+    id,
+    lastName,
+  } = props
+  const ariaProps = buildAriaProps(aria)
+  const dataProps = buildDataProps(data)
+  const classes = classnames(buildCss('pb_person_contact_kit'), className, spacing(props))
+  const wrongContacts = () => (
+    contacts.filter((contactObject) => (
+      contactObject.contactType === 'wrong-phone'
+    ))
+  )
+  const validContacts = () => (
+    contacts.filter((contactObject) => (
+      contactObject.contactType !== 'wrong-phone'
+    ))
+  )
+  return (
+    <div
+        {...ariaProps}
+        {...dataProps}
+        className={classes}
+        id={id}
+    >
+      <Person
+          firstName={firstName}
+          lastName={lastName}
       />
-    ))}
-  </div>
-)
+      {validContacts().map((contactObject, index) => (
+        <Contact
+            contactDetail={contactObject.contactDetail}
+            contactType={contactObject.contactType}
+            contactValue={contactObject.contactValue}
+            key={index}
+        />
+            ))}
+      {wrongContacts().map((contactObject, index) => (
+        <>
+          <Caption
+              className="wrong_numbers"
+              text="wrong number"
+          />
+          <Contact
+              contactType={contactObject.contactType}
+              contactValue={contactObject.contactValue}
+              key={index}
+          />
+        </>
+          ))}
+    </div>
+  )
+}
 export default PersonContact

data/app/pb_kits/playbook/pb_person_contact/docs/_person_contact_with_wrong_numbers.jsx ADDED

@@ -0,0 +1,32 @@
+import React from 'react'
+import { PersonContact } from '../../'
+const PersonContactWithWrongNumbers = () => {
+  return (
+    <>
+      <PersonContact
+          contacts={[
+            {
+              contactType: 'email',
+              contactValue: 'email@example.com',
+            },
+            {
+              contactValue: '5555555555',
+            },
+            {
+              contactType: 'wrong-phone',
+              contactValue: '3245627482',
+            },
+            {
+              contactType: 'phone',
+              contactValue: '3048615385',
+            },
+          ]}
+          firstName="Pauline"
+          lastName="Smith"
+      />
+    </>
+  )
+}
+export default PersonContactWithWrongNumbers

data/app/pb_kits/playbook/pb_person_contact/docs/example.yml CHANGED

@@ -10,3 +10,5 @@ examples:
   - person_contact_default: Default
   - person_contact_multiple: Multiple People
   - person_contact_with_detail: With Detail
+  - person_contact_with_wrong_numbers: With Wrong Numbers

data/app/pb_kits/playbook/pb_person_contact/docs/index.js CHANGED

@@ -1,3 +1,4 @@
 export { default as PersonContactDefault } from './_person_contact_default.jsx'
 export { default as PersonContactMultiple } from './_person_contact_multiple.jsx'
 export { default as PersonContactWithDetail } from './_person_contact_with_detail.jsx'
+export { default as PersonContactWithWrongNumbers } from './_person_contact_with_wrong_numbers.jsx'

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

@@ -4,23 +4,18 @@ import React from 'react'
 import classnames from 'classnames'
 import Title from '../pb_title/_title.jsx'
+import { spacing } from '../utilities/spacing.js'
 type PillProps = {
   className?: String,
   id?: String,
   text: String,
-  variant?: 'success' | 'warning' | 'error' | 'info' | 'neutral',
+  variant?: "success" | "warning" | "error" | "info" | "neutral",
 }
-const Pill = ({
-  className,
-  text,
-  variant = 'neutral',
-}: PillProps) => {
-  const css = classnames([
-    `pb_pill_kit_${variant}`,
-    className,
-  ])
+const Pill = (props: PillProps) => {
+  const { className, text, variant = 'neutral' } = props
+  const css = classnames([`pb_pill_kit_${variant}`, className], spacing(props))
   return (
     <div className={css}>

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

@@ -2,9 +2,9 @@
   id: object.id,
   data: object.data,
   class: object.classname) do %>
-    <div class="popover_tooltip" id="<%= object.tooltip_id %>" role="tooltip">
-      <%= pb_rails("card", props: {shadow: "deeper"}) do %>
+    <div class="popover_tooltip" id="<%= object.tooltip_id %>" role="tooltip" style="<%= object.z_index_helper %>">
+      <div class="popover_body <%= object.popover_spacing_helper %>" style="<%= object.width_height_helper %>">
         <%= capture(&object.children) %>
-      <% end %>
+      </div>
     </div>
-<% end %>
+<% end %>

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

@@ -10,16 +10,14 @@ import {
   Reference as PopperReference,
 } from 'react-popper'
-import {
-  buildCss,
-  noop,
-} from '../utilities/props'
+import { buildCss, noop } from '../utilities/props'
-import { Card } from '../'
+import classnames from 'classnames'
+import { spacing } from '../utilities/spacing.js'
 type PbPopoverProps = {
   className?: String,
-  closeOnClick?: 'outside' | 'inside',
+  closeOnClick?: "outside" | "inside",
   offset?: Boolean,
   reference: PopperReference,
   show?: Boolean,
@@ -30,11 +28,12 @@ type PbPopoverProps = {
 // https://popper.js.org/docs/v2/modifiers
 const POPOVER_MODIFIERS = {
-  offset: { //https://popper.js.org/docs/v2/modifiers/offset/
+  offset: {
+    //https://popper.js.org/docs/v2/modifiers/offset/
     enabled: true,
     name: 'offset',
     options: {
-      offset: [0, 8],
+      offset: [0, 20],
     },
     phase: 'main',
   },
@@ -44,38 +43,72 @@ const popoverModifiers = ({ modifiers, offset }) => {
   return offset ? modifiers.concat([POPOVER_MODIFIERS.offset]) : modifiers
 }
-const Popover = ({
-  children,
-  className,
-  modifiers,
-  offset,
-  placement,
-  referenceElement,
-}: PbPopoverProps) => (
-  <Popper
-      modifiers={popoverModifiers({ modifiers, offset })}
-      placement={placement}
-      referenceElement={referenceElement}
-  >
-    {({ placement, ref, style }) => {
-      return (
-        <div
-            className={`${buildCss('pb_popover_kit')} ${className}`}
-            data-placement={placement}
-            ref={ref}
-            style={style}
-        >
-          <div className={`${buildCss('popover_tooltip')} show`}>
-            <Card shadow="deeper">
-              { children }
-            </Card>
+const Popover = (props: PbPopoverProps) => {
+  const {
+    children,
+    className,
+    modifiers,
+    offset,
+    placement,
+    referenceElement,
+    zIndex,
+    maxHeight,
+    maxWidth,
+    minHeight,
+    minWidth,
+  } = props
+  const popoverSpacing = spacing(props) ? spacing(props) : 'p_sm'
+  const zIndexStyle = zIndex ? { zIndex: zIndex } : {}
+  const widthHeightStyles = () => {
+    return Object.assign(
+      {},
+      maxHeight ? { maxHeight: maxHeight } : {},
+      maxWidth ? { maxWidth: maxWidth } : {},
+      minHeight ? { minHeight: minHeight } : {},
+      minWidth ? { minWidth: minWidth } : {}
+    )
+  }
+  return (
+    <Popper
+        modifiers={popoverModifiers({ modifiers, offset })}
+        placement={placement}
+        referenceElement={referenceElement}
+    >
+      {({ placement, ref, style }) => {
+        return (
+          <div
+              className={`${buildCss('pb_popover_kit')} ${className}`}
+              data-placement={placement}
+              ref={ref}
+              style={Object.assign(
+                {},
+                style,
+                zIndexStyle
+              )}
+          >
+            <div
+                className={classnames(
+                `${buildCss('popover_tooltip')} show`
+              )}
+            >
+              <div
+                  className={classnames(
+                  'popover_body',
+                  popoverSpacing
+                )}
+                  style={widthHeightStyles()}
+              >
+                {children}
+              </div>
+            </div>
           </div>
-        </div>
         )
-      }
-    }
-  </Popper>
-)
+      }}
+    </Popper>
+  )
+}
 export default class PbReactPopover extends React.Component<PbPopoverProps> {
   static defaultProps = {
@@ -94,8 +127,10 @@ export default class PbReactPopover extends React.Component {
     if (!closeOnClick) return
     document.body.addEventListener('click', ({ target }) => {
-      const targetIsPopover = target.closest('[class^=popover_tooltip]') !== null
-      const targetIsReference = target.closest('.pb_popover_reference_wrapper') !== null
+      const targetIsPopover =
+        target.closest('[class^=popover_tooltip]') !== null
+      const targetIsReference =
+        target.closest('.pb_popover_reference_wrapper') !== null
       if (targetIsReference) return
@@ -131,15 +166,26 @@ export default class PbReactPopover extends React.Component {
       referenceElement,
       show,
       usePortal,
+      zIndex,
+      maxHeight,
+      maxWidth,
+      minHeight,
+      minWidth,
     } = this.props
     const popoverComponent = (
       <Popover
+          {...this.props}
           className={className}
+          maxHeight={maxHeight}
+          maxWidth={maxWidth}
+          minHeight={minHeight}
+          minWidth={minWidth}
           modifiers={modifiers}
           offset={offset}
           placement={placement}
           referenceElement={referenceElement}
+          zIndex={zIndex}
       >
         {children}
       </Popover>
@@ -154,9 +200,7 @@ export default class PbReactPopover extends React.Component {
                   className="pb_popover_reference_wrapper"
                   ref={ref}
               >
-                <reference.type
-                    {...reference.props}
-                />
+                <reference.type {...reference.props} />
               </span>
             )}
           </PopperReference>