playbook_ui 14.3.0 → 14.3.1.pre.alpha.dhhuynh2patch13743

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5eb649996c1fd3759b47ed9784e4189e1dba79f0a267936bb3843c2cff995d6c
-  data.tar.gz: cddd304f2aace5096dfa1035ac0e8dc304cfdb857b38d21606d7045d38d0ca97
+  metadata.gz: c4d151e6976dc38aa5b9105255c252c776e2eb50b15858a7dff6ce3c2d7eeeb1
+  data.tar.gz: c1b3c5d8a8565fcc9ed067b201d392422aa106384c11c0d85d31cc12fb936c83
 SHA512:
-  metadata.gz: 572786db824746207a14e7a5106b4da6af5d88d60aa0998aed2482a170431b477cfda472504d5b8bc9fc75c8b9fd2750148331d4a7273405a548f26bfc19b257
-  data.tar.gz: d3dd8bba1a7428ea356de7265d14cc3ef868e814e7a3f10043b5aaa3682896c998039226082a6ab063523dec461c184b1f7f0221c2224bf0c3305b6a33dd45fc
+  metadata.gz: 8034c25cd01b5c736ecad25b85e8ad8d5e8e522d578e2892077d4337667b1d20dca93d6126064517936b7c00d41f094342fc0a25ada253e2b98f2cbf026151d4
+  data.tar.gz: b41862c830e4254db3ee6f82989d9619187bbc114278066e9e963dc49373d22245584d056de7291b0738349cb5daac552ac3ca3b62ffd5317ed565f44c9dc4b1

data/app/pb_kits/playbook/pb_date_picker/sass_partials/_flatpickr_styles.scss

@@ -323,7 +323,7 @@
 }
 .flatpickr-current-month .numInputWrapper {
   width: 6ch;
-  width: 7ch\0;
+  width: 7ch;
   display: inline-block;
 }
 .flatpickr-current-month .numInputWrapper span.arrowUp:after {
@@ -774,4 +774,4 @@ span.flatpickr-weekday {
     -webkit-transform: translate3d(0, 0, 0);
             transform: translate3d(0, 0, 0);
   }
-}
+}

data/app/pb_kits/playbook/pb_radio/docs/_radio_children.jsx

@@ -18,32 +18,29 @@ const RadioChildren = (props) => {
     <div>
       <Radio
           label="Select"
-          marginBottom="sm"
           name="Group1"
           tabIndex={0}
           value="Select"
           {...props}
       >
         <Select 
-            marginBottom="none"
             minWidth="xs"
             options={options}
         />
       </Radio>
       <Radio
           label="Typeahead"
-          marginBottom="sm"
           name="Group1"
           tabIndex={0}
           value="Typeahead"
           {...props}
       >
         <Typeahead 
-            marginBottom="none"
             minWidth="xs"
             options={options}
         />
       </Radio>
+      <br />
       <Radio
           defaultChecked={false}
           label="Typography"

data/app/pb_kits/playbook/pb_text_input/_text_input.scss

@@ -45,6 +45,7 @@
       }
     }
     .text_input_wrapper {
+      margin-bottom: 1rem;
       input::placeholder,
       .text_input .placeholder {
         @include pb_body_light_dark;

data/app/pb_kits/playbook/pb_typeahead/_typeahead.test.jsx

@@ -107,17 +107,4 @@ test('should pass className prop', () => {
 
   const kit = screen.getByTestId('typeahead-test')
   expect(kit).toHaveClass(className)
-})
-
-test('typeahead textinput has mb_sm class by default', () => {
-  render(
-    <Typeahead
-        data={{ testid: 'default-mb-test' }}
-        options={options} 
-    />
-  )
-
-  const kit = screen.getByTestId('default-mb-test')
-  const textInput = kit.querySelector(".pb_text_input_kit")
-  expect(textInput).toHaveClass("mb_sm")
 })

data/app/pb_kits/playbook/pb_typeahead/_typeahead.tsx

@@ -45,7 +45,6 @@ type TypeaheadProps = {
   getOptionLabel?: string | (() => any),
   getOptionValue?: string | (() => any),
   name?: string,
-  marginBottom?: "none" | "xxs" | "xs" | "sm" | "md" | "lg" | "xl",
 } & GlobalProps
 
 export type SelectValueType = {
@@ -77,18 +76,12 @@ const Typeahead = ({
   htmlOptions = {},
   id,
   loadOptions = noop,
-  marginBottom = "sm",
   ...props
 }: TypeaheadProps) => {
   const selectProps = {
     cacheOptions: true,
     components: {
-      Control: (controlProps: any) => (
-        <Control 
-            {...controlProps} 
-            marginBottom={marginBottom} 
-        />
-      ),
+      Control,
       ClearIndicator,
       IndicatorsContainer,
       IndicatorSeparator: null as null,

data/app/pb_kits/playbook/pb_typeahead/components/Control.tsx

@@ -6,30 +6,23 @@ import TextInput from '../../pb_text_input/_text_input'
 
 type Props = {
   selectProps: any,
-  marginBottom?: string,
 }
 
-const TypeaheadControl = (props: Props) => {
-  const { selectProps, marginBottom } = props
-  const { dark, error, label } = selectProps
-
-  return (
-    <div className="pb_typeahead_wrapper">
-      <TextInput
-          dark={dark}
-          error={error}
-          label={label}
-          marginBottom={marginBottom}
-      >
-        <Flex>
-          <components.Control
-              className="text_input"
-              {...props}
-          />
-        </Flex>
-      </TextInput>
-    </div>
-  )
-}
+const TypeaheadControl = (props: Props) => (
+  <div className="pb_typeahead_wrapper">
+    <TextInput
+      dark={props.selectProps.dark}
+      error={props.selectProps.error}
+      label={props.selectProps.label}
+    >
+      <Flex>
+        <components.Control
+          className="text_input"
+          {...props}
+        />
+      </Flex>
+    </TextInput>
+  </div>
+)
 
 export default TypeaheadControl

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

@@ -9,7 +9,6 @@ examples:
     - typeahead_inline: Inline
     - typeahead_multi_kit: Multi Kit Options
     - typeahead_error_state: Error State
-    - typeahead_margin_bottom: Margin Bottom
 
   react:
     - typeahead_default: Default
@@ -24,4 +23,3 @@ examples:
     - typeahead_async_createable: Createable (+ Async Data)
     - typeahead_error_state: Error State
     - typeahead_custom_menu_list: Custom MenuList
-    - typeahead_margin_bottom: Margin Bottom

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

@@ -10,4 +10,3 @@ export { default as TypeaheadCreateable } from './_typeahead_createable.jsx'
 export { default as TypeaheadAsyncCreateable } from './_typeahead_async_createable.jsx'
 export { default as TypeaheadErrorState } from './_typeahead_error_state.jsx'
 export { default as TypeaheadCustomMenuList } from './_typeahead_custom_menu_list.jsx'
-export { default as TypeaheadMarginBottom } from './_typeahead_margin_bottom.jsx'

data/app/pb_kits/playbook/pb_typeahead/typeahead.html.erb

@@ -20,8 +20,7 @@
         label: object.label,
         name: object.name,
         value: object.value,
-        placeholder: object.placeholder,
-        margin_bottom: "none",
+        placeholder: object.placeholder
       }) %>
       <%= pb_rails("list", props: { ordered: false, borderless: false, xpadding: true, role: "status", aria: { live: "polite" }, data: { pb_typeahead_kit_results: true } }) do %>
       <% end %>

data/app/pb_kits/playbook/pb_typeahead/typeahead.rb

@@ -14,9 +14,6 @@ module Playbook
                     default: false
       prop :label
       prop :load_options
-      prop :margin_bottom, type: Playbook::Props::Enum,
-                           values: %w[none xxs xs sm md lg xl],
-                           default: "sm"
       prop :multi_kit, type: Playbook::Props::String,
                        default: ""
       prop :name
@@ -39,8 +36,7 @@ module Playbook
       prop :value
 
       def classname
-        default_margin_bottom = margin_bottom.present? ? "" : " mb_sm"
-        generate_classname("pb_typeahead_kit") + default_margin_bottom
+        generate_classname("pb_typeahead_kit")
       end
 
       def inline_class
@@ -69,7 +65,6 @@ module Playbook
           inline: inline,
           isMulti: is_multi,
           label: label,
-          marginBottom: margin_bottom,
           multiKit: multi_kit,
           name: name,
           options: options,