playbook_ui 13.16.0.pre.alpha.PLAY11641992 → 13.16.0.pre.alpha.PLAY12002127

data/app/pb_kits/playbook/pb_map/_map_custom_button.tsx

@@ -3,13 +3,15 @@ import Button from "../pb_button/_button";
 import Icon from "../pb_icon/_icon";
 
 type MapCustomButtonTypes = {
-  onClick?: () => {};
+  onClick?: () => void;
   icon?: string;
 };
 
 const MapCustomButton = ({ onClick, icon }: MapCustomButtonTypes) => {
   return (
-    <Button className="pb_map-custom-button" onClick={onClick}>
+    <Button className="pb_map-custom-button"
+        onClick={onClick}
+    >
       <Icon icon={icon} />
     </Button>
   );

data/app/pb_kits/playbook/pb_message/_message.tsx

@@ -20,7 +20,7 @@ type MessageProps = {
   avatarUrl?: string,
   children?: React.ReactChild[] | React.ReactChild,
   className?: string,
-  data?: object,
+  data?: Record<string, unknown>,
   htmlOptions?: {[key: string]: string | number | boolean | (() => void)},
   id?: string,
   label?: string,

data/app/pb_kits/playbook/pb_message/_message_mention.tsx

@@ -7,7 +7,7 @@ type MessageMentionProps = {
   aria: { [key: string]: string },
   children?: React.ReactChild[] | React.ReactChild,
   className?: string,
-  data?: object,
+  data?: Record<string, unknown>,
   htmlOptions?: {[key: string]: string | number | boolean | (() => void)},
   id?: string,
   variant: 'user' | 'self',
@@ -34,11 +34,11 @@ const MessageMention = (props: MessageMentionProps) => {
 
   return (
     <div
-      {...ariaProps}
-      {...dataProps}
-      {...htmlProps}
-      className={classes}
-      id={id}
+        {...ariaProps}
+        {...dataProps}
+        {...htmlProps}
+        className={classes}
+        id={id}
     >
       {children}
     </div>

data/app/pb_kits/playbook/pb_multi_level_select/_multi_level_select.tsx

@@ -86,6 +86,50 @@ const MultiLevelSelect = (props: MultiLevelSelectProps) => {
     item: []
   })
 
+  const modifyRecursive = (tree: { [key: string]: any }[], check: boolean) => {
+    if (!Array.isArray(tree)) {
+      return
+    }
+    return tree.map((item: { [key: string]: any }) => {
+      item.checked = check
+      item.children = modifyRecursive(item.children, check)
+      return item
+    })
+  }
+
+   // Function to map over data and add parent_id + depth property to each item
+  const addCheckedAndParentProperty = (
+    treeData: { [key: string]: any }[],
+    selectedIds: string[],
+    parent_id: string = null,
+    depth = 0,
+  ) => {
+    if (!Array.isArray(treeData)) {
+      return
+    }
+    return treeData.map((item: { [key: string]: any } | any) => {
+      const newItem = {
+        ...item,
+        checked: Boolean(selectedIds && selectedIds.length && selectedIds.includes(item.id)),
+        parent_id,
+        depth,
+      }
+      if (newItem.children && newItem.children.length > 0) {
+        const children =
+          item.checked && !returnAllSelected
+            ? modifyRecursive(item.children, true)
+            : item.children
+        newItem.children = addCheckedAndParentProperty(
+          children,
+          selectedIds,
+          newItem.id,
+          depth + 1
+        )
+      }
+      return newItem
+    })
+  }
+
   useEffect(() => {
     const formattedData = addCheckedAndParentProperty(
       treeData,
@@ -139,16 +183,7 @@ const MultiLevelSelect = (props: MultiLevelSelectProps) => {
     }
   }, [])
 
-  const modifyRecursive = (tree: { [key: string]: any }[], check: boolean) => {
-    if (!Array.isArray(tree)) {
-      return
-    }
-    return tree.map((item: { [key: string]: any }) => {
-      item.checked = check
-      item.children = modifyRecursive(item.children, check)
-      return item
-    })
-  }
+
 
   // Iterate over tree, find item and set checked or unchecked
   const modifyValue = (
@@ -206,38 +241,7 @@ const MultiLevelSelect = (props: MultiLevelSelectProps) => {
     return tree
   }
 
-  // Function to map over data and add parent_id + depth property to each item
-  const addCheckedAndParentProperty = (
-    treeData: { [key: string]: any }[],
-    selectedIds: string[],
-    parent_id: string = null,
-    depth = 0,
-  ) => {
-    if (!Array.isArray(treeData)) {
-      return
-    }
-    return treeData.map((item: { [key: string]: any } | any) => {
-      const newItem = {
-        ...item,
-        checked: Boolean(selectedIds && selectedIds.length && selectedIds.includes(item.id)),
-        parent_id,
-        depth,
-      }
-      if (newItem.children && newItem.children.length > 0) {
-        const children =
-          item.checked && !returnAllSelected
-            ? modifyRecursive(item.children, true)
-            : item.children
-        newItem.children = addCheckedAndParentProperty(
-          children,
-          selectedIds,
-          newItem.id,
-          depth + 1
-        )
-      }
-      return newItem
-    })
-  }
+ 
 
   // Click event for x on form pill
   const handlePillClose = (event: any, clickedItem: { [key: string]: any }) => {

data/app/pb_kits/playbook/pb_multiple_users_stacked/_multiple_users_stacked.tsx

@@ -44,11 +44,11 @@ const MultipleUsersStacked = (props: MultipleUsersStackedProps) => {
     return users.slice(0, 1).map((userObject, index) => {
       return (
         <Avatar
-          {...userObject}
-          className="pb_multiple_users_stacked_item"
-          dark={dark}
-          key={index}
-          size="xs"
+            {...userObject}
+            className="pb_multiple_users_stacked_item"
+            dark={dark}
+            key={index}
+            size="xs"
         />
       )
     })
@@ -59,11 +59,11 @@ const MultipleUsersStacked = (props: MultipleUsersStackedProps) => {
       return users.slice(1, 2).map((userObject, index) => {
         return (
           <Avatar
-            {...userObject}
-            className="pb_multiple_users_stacked_item second_item"
-            dark={dark}
-            key={index}
-            size="xs"
+              {...userObject}
+              className="pb_multiple_users_stacked_item second_item"
+              dark={dark}
+              key={index}
+              size="xs"
           />
         )
       })
@@ -74,11 +74,11 @@ const MultipleUsersStacked = (props: MultipleUsersStackedProps) => {
     if (moreThanTwo === true) {
       return (
         <Badge
-          className="pb_multiple_users_stacked_item second_item"
-          dark={dark}
-          rounded
-          text={`+${users.length - displayCount()}`}
-          variant="primary"
+            className="pb_multiple_users_stacked_item second_item"
+            dark={dark}
+            rounded
+            text={`+${users.length - displayCount()}`}
+            variant="primary"
         />
       )
     }
@@ -86,11 +86,11 @@ const MultipleUsersStacked = (props: MultipleUsersStackedProps) => {
 
   return (
     <div
-      {...ariaProps}
-      {...dataProps}
-      {...htmlProps}
-      className={classes}
-      id={id}
+        {...ariaProps}
+        {...dataProps}
+        {...htmlProps}
+        className={classes}
+        id={id}
     >
       {firstUser()}
       {secondUser()}

data/app/pb_kits/playbook/pb_nav/_item.tsx

@@ -18,7 +18,7 @@ type NavItemProps = {
   children?: React.ReactNode[] | React.ReactNode;
   className?: string;
   collapsible?: boolean;
-  data?: object;
+  data?: Record<string, unknown>;
   dark?: boolean;
   fontSize?: "normal" | "small";
   htmlOptions?: {[key: string]: string | number | boolean | (() => void)},
@@ -76,7 +76,7 @@ const NavItem = (props: NavItemProps) => {
     id,
     imageUrl,
     link,
-    onClick = () => {},
+    onClick,
     target = "_self",
     text = "",
     collapsibleTrail,
@@ -205,46 +205,49 @@ const { filteredPadding, filteredMargin } = filterItemSpacing(itemSpacing);
       {collapsible ? (
         <>
           <Collapsible
-            className={collapsibleClasses}
-            icon={iconRight && iconRight}
-            iconSize="xs"
-            id={id}
-            collapsed={collapsed}
-            onIconClick={onIconRightClick}
-            onClick={onClick}
+              className={collapsibleClasses}
+              collapsed={collapsed}
+              icon={iconRight && iconRight}
+              iconSize="xs"
+              id={id}
+              onClick={onClick}
+              onIconClick={onIconRightClick}
           >
             <Collapsible.Main
-            className={globalProps({ ...finalItemSpacing })}
-            dark={dark}>
+                className={globalProps({ ...finalItemSpacing })}
+                dark={dark}
+            >
               <Tag
-                {...ariaProps}
-                {...dataProps}
-                {...htmlProps}
-                className={classes}
-                id={id}
-                href={link}
-                target={target}
+                  {...ariaProps}
+                  {...dataProps}
+                  {...htmlProps}
+                  className={classes}
+                  href={link}
+                  id={id}
+                  target={target}
               >
                 {imageUrl && (
                   <div
-                    className="pb_nav_list_item_icon_section_collapsible"
-                    key={imageUrl}
-                    onClick={(e) => handleIconClick(e)}
+                      className="pb_nav_list_item_icon_section_collapsible"
+                      key={imageUrl}
+                      onClick={(e) => handleIconClick(e)}
                   >
-                    <Image className="pb_nav_img_wrapper" url={imageUrl} />
+                    <Image className="pb_nav_img_wrapper"
+                        url={imageUrl}
+                    />
                   </div>
                 )}
 
                 {iconLeft && (
                   <div
-                    className="pb_nav_list_item_icon_section_collapsible"
-                    key={iconLeft}
-                    onClick={(e) => handleIconClick(e)}
+                      className="pb_nav_list_item_icon_section_collapsible"
+                      key={iconLeft}
+                      onClick={(e) => handleIconClick(e)}
                   >
                     <Icon
-                      className="pb_nav_list_item_icon_left_collapsible"
-                      fixedWidth
-                      icon={iconLeft}
+                        className="pb_nav_list_item_icon_left_collapsible"
+                        fixedWidth
+                        icon={iconLeft}
                     />
                   </div>
                 )}
@@ -258,27 +261,33 @@ const { filteredPadding, filteredMargin } = filterItemSpacing(itemSpacing);
         </>
       ) : (
         <Tag
-          {...ariaProps}
-          {...dataProps}
-          {...htmlProps}
-          className={classes}
-          id={id}
-          href={link}
-          onClick={onClick}
-          target={target}
+            {...ariaProps}
+            {...dataProps}
+            {...htmlProps}
+            className={classes}
+            href={link}
+            id={id}
+            onClick={onClick}
+            target={target}
         >
           {imageUrl && (
-            <div className="pb_nav_list_item_icon_section" key={imageUrl}>
-              <Image className="pb_nav_img_wrapper" url={imageUrl} />
+            <div className="pb_nav_list_item_icon_section"
+                key={imageUrl}
+            >
+              <Image className="pb_nav_img_wrapper"
+                  url={imageUrl}
+              />
             </div>
           )}
 
           {iconLeft && (
-            <div className="pb_nav_list_item_icon_section" key={iconLeft}>
+            <div className="pb_nav_list_item_icon_section"
+                key={iconLeft}
+            >
               <Icon
-                className="pb_nav_list_item_icon_left"
-                fixedWidth
-                icon={iconLeft}
+                  className="pb_nav_list_item_icon_left"
+                  fixedWidth
+                  icon={iconLeft}
               />
             </div>
           )}
@@ -287,13 +296,13 @@ const { filteredPadding, filteredMargin } = filterItemSpacing(itemSpacing);
 
           {iconRight && (
             <div
-              className="pb_nav_list_item_icon_section"
-              key={iconRight as string}
+                className="pb_nav_list_item_icon_section"
+                key={iconRight as string}
             >
               <Icon
-                className="pb_nav_list_item_icon_right"
-                fixedWidth
-                icon={iconRight as string}
+                  className="pb_nav_list_item_icon_right"
+                  fixedWidth
+                  icon={iconRight as string}
               />
             </div>
           )}

data/app/pb_kits/playbook/pb_nav/_nav.tsx

@@ -12,12 +12,12 @@ type NavProps = {
   borderless?: boolean,
   children?: React.ReactNode[] | React.ReactNode,
   className?: string | string[],
-  data?: object,
+  data?: Record<string, unknown>,
   dark?: boolean,
   highlight?: boolean,
   htmlOptions?: {[key: string]: string | number | boolean | (() => void)},
   id?: string,
-  onClick?: React.MouseEventHandler<HTMLElement>,
+  onClick?: () => void,
   orientation?: "vertical" | "horizontal",
   link?: string,
   title?: string,
@@ -25,7 +25,7 @@ type NavProps = {
   itemSpacing?: SpacingObject
 } & GlobalProps
 
-const Nav = (props: NavProps) => {
+const Nav = (props: NavProps): React.ReactElement => {
   const {
     aria = {},
     borderless = false,
@@ -37,7 +37,7 @@ const Nav = (props: NavProps) => {
     htmlOptions = {},
     id,
     link = '#',
-    onClick = () => { },
+    onClick,
     orientation = 'vertical',
     title = '',
     variant = 'normal',
@@ -71,23 +71,23 @@ const childrenWithProps = React.Children.map(children, (child) => {
 
   return (
     <nav
-      {...ariaProps}
-      {...dataProps}
-      {...htmlProps}
-      className={cardCss}
-      id={id}
+        {...ariaProps}
+        {...dataProps}
+        {...htmlProps}
+        className={cardCss}
+        id={id}
     >
       {title &&
         <div className="pb_nav_list_title">
           <a
-            className="pb_nav_list_item_link_text"
-            href={link}
-            onClick={onClick}
+              className="pb_nav_list_item_link_text"
+              href={link}
+              onClick={onClick}
           >
             <Caption
-              dark={dark}
-              size="md"
-              text={`${title}`}
+                dark={dark}
+                size="md"
+                text={`${title}`}
             />
           </a>
         </div>

data/app/pb_kits/playbook/pb_phone_number_input/docs/_phone_number_input_validation.html.erb

@@ -3,12 +3,10 @@
     <%= pb_rails("button", props: {html_type: "submit", text: "Save Phone Number"}) %>
 </form>
 
-<% content_for(:pb_js) do %>
-    <%= javascript_tag do %>
-        document.addEventListener('DOMContentLoaded', function () {
-            document.querySelector('#example-form-validation').addEventListener('submit', function (e) {
-                if (e.target.querySelectorAll('[error]:not([error=""])').length > 0) e.preventDefault();
-            })
+<%= javascript_tag do %>
+    document.addEventListener('DOMContentLoaded', function () {
+        document.querySelector('#example-form-validation').addEventListener('submit', function (e) {
+            if (e.target.querySelectorAll('[error]:not([error=""])').length > 0) e.preventDefault();
         })
-    <% end %>
+    })
 <% end %>

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

@@ -171,29 +171,27 @@ const PbReactPopover = (props: PbPopoverProps): React.ReactElement => {
 
     document.body.addEventListener(
       "click",
-      ({ target }) => {
+      (e: MouseEvent) => {
+        const target = e.target as HTMLElement
+
         const targetIsPopover =
-          (target as HTMLElement).closest("#" + targetId) !==
-          null;
+          target.closest("#" + targetId) !== null;
         const targetIsReference =
-          (target as HTMLElement).closest("#reference-" + targetId) !==
-          null;
+          target.closest("#reference-" + targetId) !== null;
+
+        const shouldClose = () => {
+          setTimeout(() => shouldClosePopover(true), 0);
+        }
 
         switch (closeOnClick) {
           case "outside":
-            if (!targetIsPopover && !targetIsReference) {
-              shouldClosePopover(true);
-            }
+            if (!targetIsPopover && !targetIsReference) shouldClose();
             break;
           case "inside":
-            if (targetIsPopover) {
-              shouldClosePopover(true);
-            }
+            if (targetIsPopover) shouldClose();
             break;
           case "any":
-            if (targetIsPopover || !targetIsPopover && !targetIsReference) {
-              shouldClosePopover(true);
-            }
+            if (targetIsPopover || !targetIsPopover && !targetIsReference) shouldClose();
             break;
         }
       },

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

@@ -0,0 +1,30 @@
+<%= pb_rails("flex", props: {
+    dark: true,
+    orientation: "row",
+    vertical: "center"
+    }) do %>
+    <%= pb_rails("body", props: { text: "Click info for more details" }) %>
+    &nbsp;
+    <%= pb_rails("circle_icon_button", props: {
+        variant: "secondary",
+        icon: "info",
+        id: "actionable-popover-1"
+    }) %>
+    <%= pb_rails("popover", props: {
+        close_on_click: "any",
+        trigger_element_id: "actionable-popover-1",
+        tooltip_id: "actionable-tooltip-1",
+        offset: true,
+        position: "top"
+    }) do %>
+        <%= pb_rails("button", props: { id: "actionable-tooltip-button", text: "Learn more" }) %>
+    <% end %>
+<% end %>
+
+<%= javascript_tag do %>
+    document.addEventListener('DOMContentLoaded', function () {
+        document.querySelector('#actionable-tooltip-button').addEventListener('click', function (e) {
+            alert("Let's do this")
+        }, { once: true })
+    })
+<% end %>

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

@@ -0,0 +1,57 @@
+import React, { useState } from 'react'
+import {
+  Body,
+  Button,
+  CircleIconButton,
+  Flex,
+  PbReactPopover,
+} from '../..'
+
+const PopoverActionableContent = (props) => {
+  const [showPopover, setShowPopover] = useState(false)
+
+  const handleTogglePopover = () => {
+    setShowPopover(!showPopover)
+  }
+
+  const handleShouldClosePopover = (shouldClose) => {
+    setShowPopover(!shouldClose)
+  }
+
+  const popoverReference = (
+    <CircleIconButton
+        icon="info"
+        onClick={handleTogglePopover}
+        variant="secondary"
+    />
+  )
+
+  return (
+    <Flex
+        orientation="row"
+        vertical="center"
+        {...props}
+    >
+      <Body text="Click info for more details" />
+      &nbsp;
+      <PbReactPopover
+          closeOnClick="inside"
+          offset
+          placement="top"
+          reference={popoverReference}
+          shouldClosePopover={handleShouldClosePopover}
+          show={showPopover}
+          {...props}
+      >
+        <Body textAlign="center">
+          <Button
+              onClick={() => {alert("Let's do this!")}}
+              text="Learn More"
+          />
+        </Body>
+      </PbReactPopover>
+    </Flex>
+  )
+}
+
+export default PopoverActionableContent

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

@@ -5,6 +5,7 @@ examples:
     - popover_close: Close Options
     - popover_z_index: Set Z-Index
     - popover_scroll_height: Scroll and Height Settings
+    - popover_actionable_content: With Actionable Content
 
   react:
     - popover_default: Default
@@ -12,4 +13,4 @@ examples:
     - popover_close: Close Options
     - popover_z_index: Set Z-Index
     - popover_scroll_height: Scroll and Height Settings
-    
+    - popover_actionable_content: With Actionable Content

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

@@ -3,3 +3,4 @@ export { default as PopoverList } from './_popover_list.jsx'
 export { default as PopoverClose } from './_popover_close.jsx'
 export { default as PopoverZIndex } from './_popover_z_index.jsx'
 export { default as PopoverScrollHeight } from './_popover_scroll_height.jsx'
+export { default as PopoverActionableContent } from './_popover_actionable_content.jsx'