playbook_ui 12.12.0.pre.alpha.PLAY697sandpacklayoutstyles413 → 12.13.0.pre.alpha.PLAY705phonenumberkit438

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eac850ebbc05e1ce91f38e6377564b32fa3ba241de3ff599b5e0084a39666223
-  data.tar.gz: 2ce42692684f6598b0c22d7e763ff9bd815044bd28ea806b659746d1f73ea098
+  metadata.gz: 8ffc2f1f3d1df264f744d65f5b89a07812fe6e560e9c9137e3f91e2b1eb17816
+  data.tar.gz: a85d780b532e6930b7588a22d566023d965398cb73ea2267e8f12ab5e46f3114
 SHA512:
-  metadata.gz: 268d73b140d2e8aeb3086a4588b2a42c3f3c0dd26d7c4112b37583bcd85e9657fe59e6854042e2a8c9ca02bb139e4aef5aec926430d9da6f2b98c2c1b92d2d2e
-  data.tar.gz: f58ed28b37d12a8e3555f44612a907aabb5e60b81cba57c78c017bb86328c6d437d14999849106847bc5967f59f51daac1c3e5bca921f7e7f7958dffc1ee843a
+  metadata.gz: b346f34311944e98fb46a98c01fce27798f102ab81cc93db231ddfa570648b613a43a0bb40c8c628b3bc434366d8216f44977c657a5f9dd6fff67c655ce7edec
+  data.tar.gz: 0a807e7b97cd9f9524572abc04563d13e25fd0dff7b032e9479a0a41847502b515ee64508432ac7b0e63d48202c9a282022d678923aef7cf408ac6d1cec34b6f

data/app/pb_kits/playbook/pb_flex/_flex.scss

@@ -2,6 +2,7 @@
 @import "../tokens/spacing";
 
 $gapSpaces:(
+  xxs:  $space_xxs,
   xs:   $space_xs,
   sm:   $space_sm,
   md:   $space_md,

data/app/pb_kits/playbook/pb_flex/_flex.tsx

@@ -2,6 +2,7 @@ import React from 'react'
 import classnames from 'classnames'
 import { buildCss, buildDataProps } from '../utilities/props'
 import { GlobalProps, globalProps } from '../utilities/globalProps'
+import { Sizes } from '../types'
 
 type FlexProps = {
   children: React.ReactChild[] | React.ReactNode,
@@ -16,9 +17,9 @@ type FlexProps = {
   reverse?: boolean,
   vertical?: "top" | "center" | "bottom" | "stretch" | "baseline" | "none",
   align?: "start" | "center" | "end" | "stretch" | "baseline" | "none",
-  gap?: "xs" | "sm" | "md" | "lg" | "xl" | "none",
-  rowGap?: "xs" | "sm" | "md" | "lg" | "xl" | "none",
-  columnGap?: "xs" | "sm" | "md" | "lg" | "xl" | "none",
+  gap?: Sizes | "none",
+  rowGap?: Sizes| "none",
+  columnGap?: Sizes| "none",
   wrap?: boolean,
   alignSelf?: "start" | "end" | "center" | "stretch" | "none"
 } & GlobalProps

data/app/pb_kits/playbook/pb_flex/docs/_flex_gap.html.erb

@@ -1,7 +1,7 @@
 <%= pb_rails("title", props: {size: 4, text: "Row"}) %>
 <br/>
 <div class="flex-doc-example">
-  <%= pb_rails("flex", props:{ gap: "xs", wrap:true}) do %>
+  <%= pb_rails("flex", props:{ gap: "xxs", wrap:true}) do %>
    <%40.times do |i|%>
     <%= pb_rails("flex/flex_item") do %> <%=i%> <% end %>
    <% end %>

data/app/pb_kits/playbook/pb_flex/docs/_flex_gap.jsx

@@ -14,7 +14,7 @@ const FlexGap = (props) => {
     <>
       <div className="flex-doc-example">
         <Flex
-            gap="xs"
+            gap="xxs"
             wrap
             {...props}
         >

data/app/pb_kits/playbook/pb_flex/docs/_flex_gap.md

@@ -1,6 +1,6 @@
 ##### Prop
 
-*  `gap` | `row_gap` | `rowGap` | `column_gap` | `columnGap` | **Type**: String | **Values**: xs | sm | md | lg | xl | none
+*  `gap` | `row_gap` | `rowGap` | `column_gap` | `columnGap` | **Type**: String | **Values**: xxs | xs | sm | md | lg | xl | none
 
 - Setting the gap prop sets the row_gap || rowGap and the column_gap || columnGap props to the same size and creates equal space within a flex container.
 

data/app/pb_kits/playbook/pb_flex/flex.rb

@@ -25,15 +25,15 @@ module Playbook
                      deprecated: true
 
       prop :gap, type: Playbook::Props::Enum,
-                 values: %w[xs sm md lg xl none],
+                 values: %w[xxs xs sm md lg xl none],
                  default: "none"
 
       prop :row_gap, type: Playbook::Props::Enum,
-                     values: %w[xs sm md lg xl none],
+                     values: %w[xxs xs sm md lg xl none],
                      default: "none"
 
       prop :column_gap, type: Playbook::Props::Enum,
-                        values: %w[xs sm md lg xl none],
+                        values: %w[xxs xs sm md lg xl none],
                         default: "none"
 
       prop :reverse, type: Playbook::Props::Boolean,

data/app/pb_kits/playbook/pb_phone_number_input/_phone_number_input.tsx

@@ -89,6 +89,7 @@ const PhoneNumberInput = (props: PhoneNumberInputProps) => {
   const [itiInit, setItiInit] = useState<any>()
   const [error, setError] = useState('')
   const [dropDownIsOpen, setDropDownIsOpen] = useState(false)
+  const [selectedData, setSelectedData] = useState()
 
   const validateTooLongNumber = (itiInit: any) => {
     const error = itiInit.getValidationError()
@@ -121,10 +122,16 @@ const PhoneNumberInput = (props: PhoneNumberInputProps) => {
     validateOnlyNumbers()
   }
 
+  const getCurrentSelectedData = (itiInit: any, inputValue: string) => {
+    return { ...itiInit.getSelectedCountryData(), number: inputValue }
+  }
+
   const handleOnChange = (evt: React.ChangeEvent<HTMLInputElement>) => {
     setInputValue(evt.target.value)
     validateTooLongNumber(itiInit)
-    onChange(evt)
+    const phoneNumberData = getCurrentSelectedData(itiInit, evt.target.value)
+    setSelectedData(phoneNumberData)
+    onChange(phoneNumberData)
     isValid(itiInit.isValidNumber())
   }
 
@@ -136,15 +143,21 @@ const PhoneNumberInput = (props: PhoneNumberInputProps) => {
 
   useEffect(() => {
     const telInputInit = new intlTelInput(inputRef.current, {
-        separateDialCode: true,
-        preferredCountries,
-        allowDropdown: !disabled,
-        initialCountry,
-        onlyCountries,
-      }
+      separateDialCode: true,
+      preferredCountries,
+      allowDropdown: !disabled,
+      initialCountry,
+      onlyCountries,
+    }
     )
-    
-    inputRef.current.addEventListener("countrychange", () => validateTooLongNumber(telInputInit))
+
+    inputRef.current.addEventListener("countrychange", (evt: Event) => {
+      validateTooLongNumber(telInputInit)
+      const phoneNumberData = getCurrentSelectedData(telInputInit, (evt.target as HTMLInputElement).value)
+      setSelectedData(phoneNumberData)
+      onChange(phoneNumberData)
+    })
+
     inputRef.current.addEventListener("open:countrydropdown", () => setDropDownIsOpen(true))
     inputRef.current.addEventListener("close:countrydropdown", () => setDropDownIsOpen(false))
 
@@ -156,6 +169,7 @@ const PhoneNumberInput = (props: PhoneNumberInputProps) => {
       <TextInput
         className={dropDownIsOpen ? 'dropdown_open' : ''}
         dark={dark}
+        data-phone-number={JSON.stringify(selectedData)}
         disabled={disabled}
         error={error}
         id={id}

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

@@ -0,0 +1,107 @@
+import React, { useState } from "react";
+import classnames from "classnames";
+
+import { buildAriaProps, buildCss, buildDataProps } from "../utilities/props";
+import { globalProps } from "../utilities/globalProps";
+
+import Checkbox from "../pb_checkbox/_checkbox";
+import ListItem from "../pb_list/_list_item";
+import Radio from "../pb_radio/_radio";
+
+export type SelectableListItemProps = {
+  aria?: { [key: string]: string };
+  children: React.ReactNode[] | React.ReactNode;
+  checked?: boolean;
+  className?: string;
+  data?: object;
+  defaultChecked?: boolean;
+  id?: string;
+  label?: string;
+  text?: string;
+  name?: string;
+  value?: string;
+  variant?: string;
+  onChange: (arg: React.ChangeEvent<HTMLInputElement>  | null) => void;
+};
+
+const SelectableListItem = ({
+  aria = {},
+  checked = false,
+  children,
+  className,
+  data = {},
+  defaultChecked,
+  id,
+  label,
+  text = "",
+  name = "",
+  value = "",
+  variant = "checkbox",
+  onChange = () => {},
+  ...props
+}: SelectableListItemProps) => {
+  const ariaProps = buildAriaProps(aria);
+  const dataProps = buildDataProps(data);
+  const classes = classnames(
+    buildCss("pb_selectable_list_item_kit"),
+    globalProps(props),
+    className
+  );
+
+  const initialCheckedState = checked;
+  const [checkedState, setCheckedState] = useState(initialCheckedState);
+
+  const handleChecked = (event: React.ChangeEvent<HTMLInputElement>) => {
+    onChange(event);
+    setCheckedState(event.target.checked);
+  };
+
+  return (
+    <ListItem
+      {...props}
+      className={classnames(checkedState ? "checked_item" : "", className)}
+    >
+      <div {...ariaProps} {...dataProps} className={classes}>
+        {variant == "checkbox" && (
+          <>
+            <Checkbox
+              checked={checkedState}
+              id={id}
+              name={name}
+              onChange={handleChecked}
+              // eslint suppressor, text is needed to display on screen
+              //@ts-ignore
+              text={label || (text && false)}
+              type="checkbox"
+              value={value}
+              {...props}
+            />
+            {children}
+          </>
+        )}
+        {variant == "radio" && (
+          <>
+            <Radio
+              defaultChecked={defaultChecked}
+              id={id}
+              label={label}
+              name={name}
+              onChange={onChange}
+              //@ts-ignore
+              text={label}
+              type="radio"
+              value={value}
+              {...props}
+            />
+            {children}
+          </>
+        )}
+        {variant !== "checkbox" && variant !== "radio" && (
+          { children }
+        )}
+      </div>
+    </ListItem>
+  );
+};
+
+export default SelectableListItem;

data/app/pb_kits/playbook/pb_selectable_list/{_selectable_list.jsx → _selectable_list.tsx}

@@ -1,16 +1,16 @@
-/* @flow */
 import React, { useState } from 'react'
 import classnames from 'classnames'
 
 import { buildAriaProps, buildCss, buildDataProps } from '../utilities/props'
 import { globalProps } from '../utilities/globalProps'
+import { SelectableListItemProps } from './_item.js'
 
 import List from  '../pb_list/_list'
-import SelectableListItem from './_item.jsx'
+import SelectableListItem from './_item'
 
 type SelectableListProps = {
-  aria?: object,
-  children?: Node,
+  aria?: {[key: string]: string },
+  children?: React.ReactElement[],
   className?: string,
   data?: object,
   id?: string,
@@ -30,17 +30,18 @@ const SelectableList = (props: SelectableListProps) => {
   const classes = classnames(buildCss('pb_selectable_list_kit'), globalProps(props), className)
   const dataProps = buildDataProps(data)
   const isRadio = props.variant === "radio"
-  const defaultCheckedRadioValue = children.filter(item => item.props.defaultChecked)[0]?.props?.value
+  const defaultCheckedRadioValue = children.filter((item: {props:SelectableListItemProps} ) => item.props.defaultChecked)[0]?.props?.value
+
   const [selectedRadioValue, setSelectedRadioValue] = useState(defaultCheckedRadioValue)
   
-  const onChangeRadioValue = ({ target }) => {
+  const onChangeRadioValue = ({ target }: React.ChangeEvent<HTMLInputElement>) => {
     setSelectedRadioValue(target.value)
   }
 
   let selectableListItems = children
 
   if (isRadio) {
-    selectableListItems = children.map(({ props }) => {
+    selectableListItems = children.map(( {props}: {props:SelectableListItemProps} ) => {
       return (
         <SelectableListItem
             {...props}
@@ -51,6 +52,7 @@ const SelectableList = (props: SelectableListProps) => {
       )
     })
   }
+
   
   return (
     <div
@@ -59,13 +61,13 @@ const SelectableList = (props: SelectableListProps) => {
         className={classes}
         id={id}
     >
-      <List {...props}>
+      <List variant={props.variant}>
         {selectableListItems}
       </List>
     </div>
   )
 }
-
+ 
 SelectableList.Item = SelectableListItem
 
 export default SelectableList

data/app/pb_kits/playbook/pb_selectable_list/selectable_list.test.js

@@ -0,0 +1,68 @@
+import React from 'react'
+import { render, screen } from '../utilities/test-utils'
+import SelectableList from './_selectable_list'
+
+const testId = "selectable-list-test"
+
+const SelectableListCheckbox = () => {
+    return (
+      <SelectableList 
+          data={{ testid: testId }} 
+          variant="checkbox">
+        <SelectableList.Item
+            label="Mild"
+            name="checkbox-name-1"
+            value="1"
+        />
+        <SelectableList.Item
+            checked
+            label="Medium"
+            name="checkbox-name-2"
+            value="2"
+        />
+      </SelectableList>
+    )
+}
+
+const SelectableListRadio = () => {
+    return (
+        <SelectableList
+            data={{ testid: testId }}
+            variant="radio">
+        <SelectableList.Item
+            label="Small"
+            name="radio"
+            value="1"
+        />
+        <SelectableList.Item
+            defaultChecked
+            label="Medium"
+            name="radio"
+            value="2"
+        />
+      </SelectableList>
+    )
+}
+
+
+test("classname renders as expected", () => {
+   render(<SelectableListCheckbox />)
+
+   const kit = screen.getByTestId("selectable-list-test")
+   expect(kit).toHaveClass("pb_selectable_list_kit")
+})
+
+test("renders variant checkbox", () => {
+    render(<SelectableListCheckbox />)
+    const kit = screen.getByTestId("selectable-list-test")
+    const checkbox = kit.querySelector(".pb_checkbox_kit")
+    expect(checkbox).toBeInTheDocument()
+})
+
+test("renders variant radio", () => {
+    render(<SelectableListRadio />)
+    const kit = screen.getByTestId("selectable-list-test")
+    const checkbox = kit.querySelector(".pb_radio_kit")
+    expect(checkbox).toBeInTheDocument()
+
+})

data/lib/playbook/version.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
 module Playbook
-  PREVIOUS_VERSION = "12.12.0"
-  VERSION = "12.12.0.pre.alpha.PLAY697sandpacklayoutstyles413"
+  PREVIOUS_VERSION = "12.13.0"
+  VERSION = "12.13.0.pre.alpha.PLAY705phonenumberkit438"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: playbook_ui
 version: !ruby/object:Gem::Version
-  version: 12.12.0.pre.alpha.PLAY697sandpacklayoutstyles413
+  version: 12.13.0.pre.alpha.PLAY705phonenumberkit438
 platform: ruby
 authors:
 - Power UX
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-04-04 00:00:00.000000000 Z
+date: 2023-04-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack
@@ -1843,9 +1843,9 @@ files:
 - app/pb_kits/playbook/pb_selectable_icon/selectable_icon.html.erb
 - app/pb_kits/playbook/pb_selectable_icon/selectable_icon.rb
 - app/pb_kits/playbook/pb_selectable_icon/selectable_icon.test.js
-- app/pb_kits/playbook/pb_selectable_list/_item.jsx
-- app/pb_kits/playbook/pb_selectable_list/_selectable_list.jsx
+- app/pb_kits/playbook/pb_selectable_list/_item.tsx
 - app/pb_kits/playbook/pb_selectable_list/_selectable_list.scss
+- app/pb_kits/playbook/pb_selectable_list/_selectable_list.tsx
 - app/pb_kits/playbook/pb_selectable_list/docs/_selectable_list_checkbox.html.erb
 - app/pb_kits/playbook/pb_selectable_list/docs/_selectable_list_checkbox.jsx
 - app/pb_kits/playbook/pb_selectable_list/docs/_selectable_list_radio.html.erb
@@ -1854,6 +1854,7 @@ files:
 - app/pb_kits/playbook/pb_selectable_list/docs/index.js
 - app/pb_kits/playbook/pb_selectable_list/selectable_list.html.erb
 - app/pb_kits/playbook/pb_selectable_list/selectable_list.rb
+- app/pb_kits/playbook/pb_selectable_list/selectable_list.test.js
 - app/pb_kits/playbook/pb_selectable_list/selectable_list_item.html.erb
 - app/pb_kits/playbook/pb_selectable_list/selectable_list_item.rb
 - app/pb_kits/playbook/pb_source/_source.scss

data/app/pb_kits/playbook/pb_selectable_list/_item.jsx

@@ -1,106 +0,0 @@
-/* @flow */
-
-import React, { Node, useState } from 'react'
-import classnames from 'classnames'
-
-import { buildAriaProps, buildCss, buildDataProps } from '../utilities/props'
-import { globalProps } from '../utilities/globalProps'
-
-import Checkbox from '../pb_checkbox/_checkbox'
-import ListItem from '../pb_list/_list_item'
-import Radio from '../pb_radio/_radio'
-
-type SelectableListItemProps = {
-  aria?: object,
-  children: array<Node> | Node,
-  checked?: boolean,
-  className?: string,
-  data?: object,
-  defaultChecked?: boolean,
-  id?: string,
-  label?: string,
-  text?: string,
-  name?: string,
-  value?: string,
-  variant?: string,
-  onChange: (boolean)=>void,
-}
-
-const SelectableListItem = ({
-  aria = {},
-  checked = false,
-  children,
-  className,
-  data = {},
-  defaultChecked,
-  id,
-  label,
-  text = '',
-  name = '',
-  value = '',
-  variant = 'checkbox',
-  onChange = () => {},
-  ...props
-}: SelectableListItemProps) => {
-  const ariaProps = buildAriaProps(aria)
-  const dataProps = buildDataProps(data)
-  const classes = classnames(
-    buildCss('pb_selectable_list_item_kit'),
-    globalProps(props),
-    className
-  )
-
-  const initialCheckedState = checked
-  const [checkedState, setCheckedState] = useState(initialCheckedState)
-
-  const handleChecked = (event) => {
-    onChange(event)
-    setCheckedState(event.target.checked)
-  }
-
-  return (
-    <ListItem 
-        {...props}
-        className={classnames(checkedState ? "checked_item" : "", className)}
-    >
-      <div
-          {...ariaProps}
-          {...dataProps}
-          className={classes}
-      >
-        <Choose>
-          <When condition={variant == 'checkbox'}>
-            <Checkbox
-                checked={checkedState}
-                id={id}
-                name={name}
-                onChange={handleChecked}
-                // eslint suppressor, text is needed to display on screen
-                text={label || (text && false)}
-                type="checkbox"
-                value={value}
-                {...props}
-            />
-            {children}
-          </When>
-          <When condition={variant == 'radio'}>
-            <Radio
-                defaultChecked={defaultChecked}
-                id={id}
-                label={label}
-                name={name}
-                onChange={onChange}
-                type="radio"
-                value={value}
-                {...props}
-            />
-            {children}
-          </When>
-          <Otherwise>{children}</Otherwise>
-        </Choose>
-      </div>
-    </ListItem>
-  )
-}
-
-export default SelectableListItem