playbook_ui 14.18.0.pre.alpha.dropdownautocomplete7493 → 14.18.0.pre.alpha.play1736highchartslinegraphdefaultrebuild7431

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

@@ -9,7 +9,6 @@ examples:
   - phone_number_input_clear_field: Clearing the Input Field
   - phone_number_input_access_input_element: Accessing the Input Element
   - phone_number_input_format: Format as You Type
-  - phone_number_input_country_search: Country Search
 
   rails:
   - phone_number_input_default: Default
@@ -19,5 +18,4 @@ examples:
   - phone_number_input_validation: Form Validation
   - phone_number_input_format: Format as You Type
   - phone_number_input_hidden_inputs: Hidden Inputs
-  - phone_number_input_country_search: Country Search
   

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

@@ -6,4 +6,3 @@ export { default as PhoneNumberInputValidation } from './_phone_number_input_val
 export { default as PhoneNumberInputClearField } from './_phone_number_input_clear_field'
 export { default as PhoneNumberInputAccessInputElement } from './_phone_number_input_access_input_element'
 export { default as PhoneNumberInputFormat } from './_phone_number_input_format'
-export { default as PhoneNumberInputCountrySearch } from './_phone_number_input_country_search'

data/app/pb_kits/playbook/pb_phone_number_input/phone_number_input.rb

@@ -25,8 +25,6 @@ module Playbook
                                 default: false
       prop :hidden_inputs, type: Playbook::Props::Boolean,
                            default: false
-      prop :country_search, type: Playbook::Props::Boolean,
-                            default: false
 
       def classname
         generate_classname("pb_phone_number_input")
@@ -47,7 +45,6 @@ module Playbook
           preferredCountries: preferred_countries,
           required: required,
           value: value,
-          countrySearch: country_search,
         }
       end
     end

data/app/pb_kits/playbook/pb_phone_number_input/phone_number_input.test.js

@@ -139,22 +139,3 @@ test("should format phone number as '555-555-5555' with formatAsYouType and 'us'
 
     expect(input.value).toBe("555-555-5555");
 });
-
-test("should pass countrySearch prop to component", () => {
-    window.intlTelInput = jest.fn(() => ({
-      getSelectedCountryData: jest.fn(() => ({})),
-      isValidNumber: jest.fn(() => true),
-      getValidationError: jest.fn(() => 0),
-    }));
-    
-    const props = {
-      id: testId,
-      countrySearch: true,
-      data: { testid: 'phone-input-with-search' }
-    };
-  
-    render(<PhoneNumberInput {...props} />);
-    
-    const wrapper = screen.getByTestId('phone-input-with-search');
-    expect(wrapper).toBeInTheDocument();
-});

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

@@ -6,7 +6,6 @@ examples:
     - popover_z_index: Set Z-Index
     - popover_scroll_height: Scroll and Height Settings
     - popover_actionable_content: With Actionable Content
-    - popover_append_to: Append To
 
   react:
     - popover_default: Default

data/app/pb_kits/playbook/pb_popover/index.ts

@@ -13,15 +13,7 @@ export default class PbPopover extends PbEnhancedElement {
   }
 
   moveTooltip() {
-    let container: HTMLElement | null;
-
-    if (this.appendTo === "parent") {
-      container = this.element.parentElement;
-    } else if (this.appendTo) {
-      container = document.querySelector(this.appendTo);
-    }
-
-    (container || document.body).appendChild(this.tooltip);
+    document.querySelector('body').appendChild(this.tooltip)
   }
 
   connect() {
@@ -116,8 +108,4 @@ export default class PbPopover extends PbEnhancedElement {
   get closeOnClick() {
     return this.element.dataset.pbPopoverCloseOnClick
   }
-
-  get appendTo() {
-    return this.element.dataset.pbPopoverAppendTo;
-  }
 }

data/app/pb_kits/playbook/pb_popover/popover.rb

@@ -3,7 +3,6 @@
 module Playbook
   module PbPopover
     class Popover < Playbook::KitBase
-      prop :append_to, type: Playbook::Props::String, default: "body"
       prop :position, type: Playbook::Props::Enum,
                       values: %w[top bottom left right top-start top-end bottom-start bottom-end right-start right-end left-start left-end],
                       default: "left"
@@ -48,7 +47,6 @@ module Playbook
 
       def data
         Hash(values[:data]).merge(
-          pb_popover_append_to: append_to,
           pb_popover_kit: true,
           pb_popover_position: position,
           pb_popover_trigger_element_id: trigger_element_id,

data/app/pb_kits/playbook/pb_select/docs/_select_error.html.erb

@@ -1,5 +1,5 @@
 <%= pb_rails("select", props: {
-  error: raw(pb_rails("icon", props: { icon: "warning" }) + " Please make a valid selection"),
+  error: "Please make a valid selection",
   label: "Favorite Food",
   name: "food",
   options: [

data/app/pb_kits/playbook/pb_text_input/docs/_text_input_error.html.erb

@@ -4,7 +4,7 @@
     border: true,
     icon: "user"
   },
-  error: raw(pb_rails("icon", props: { icon: "warning" }) + " Please enter a valid email address"),
+  error: "Please enter a valid email address",
   label: "Email Address",
   placeholder: "Enter email address",
   type: "email"

data/app/pb_kits/playbook/pb_textarea/docs/_textarea_error.html.erb

@@ -1,5 +1 @@
-<%= pb_rails("textarea", props: {
-  error: raw(pb_rails("icon", props: { icon: "warning" }) + " This field has an error!"),
-  label: "Label",
-  rows: 4
-}) %>
+<%= pb_rails("textarea", props: { error: "This field has an error!", label: "Label", rows: 4}) %>

data/app/pb_kits/playbook/pb_typeahead/docs/_typeahead_error_state.html.erb

@@ -8,14 +8,7 @@
 
 %>
 
-<%= pb_rails("typeahead", props: {
-  id: "typeahead-error-example",
-  options: options,
-  error: "Please make a valid selection",
-  label: "Products",
-  name: :foo,
-  is_multi: false
-}) %>
+<%= pb_rails("typeahead", props: { id: "typeahead-error-example", options: options, error: "Please make a valid selection", label: "Products", name: :foo, is_multi: false }) %>
 
 <!-- This section is an example of the available JavaScript event hooks -->
 <%= javascript_tag defer: "defer" do %>