playbook_ui 15.3.0.pre.alpha.PLAY260211784 → 15.3.0.pre.alpha.play2541phonenumberkitvalidation12084

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (92) hide show
  1. checksums.yaml +4 -4
  2. data/app/pb_kits/playbook/pb_advanced_table/Components/RegularTableView.tsx +2 -1
  3. data/app/pb_kits/playbook/pb_advanced_table/advanced_table.test.jsx +63 -0
  4. data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_background_control_rails.html.erb +4 -0
  5. data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_background_control_rails.md +1 -1
  6. data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_column_border_color.md +1 -1
  7. data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_column_border_color_rails.md +1 -1
  8. data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_column_styling.jsx +3 -1
  9. data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_column_styling.md +2 -0
  10. data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_column_styling_column_headers.jsx +1 -1
  11. data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_column_styling_rails.html.erb +1 -0
  12. data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_column_styling_rails.md +2 -0
  13. data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_padding_control.jsx +9 -1
  14. data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_padding_control.md +1 -1
  15. data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_sticky_columns_and_header.md +1 -1
  16. data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_table_props_sticky_header_rails.md +1 -1
  17. data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_table_props_sticky_header_react.md +1 -1
  18. data/app/pb_kits/playbook/pb_advanced_table/table_row.rb +31 -2
  19. data/app/pb_kits/playbook/pb_background/background.html.erb +10 -2
  20. data/app/pb_kits/playbook/pb_background/docs/_background_category.md +1 -1
  21. data/app/pb_kits/playbook/pb_card/docs/_card_background.md +1 -1
  22. data/app/pb_kits/playbook/pb_card/docs/_card_header.md +1 -1
  23. data/app/pb_kits/playbook/pb_card/docs/_card_highlight.md +1 -1
  24. data/app/pb_kits/playbook/pb_card/docs/_card_light.md +1 -1
  25. data/app/pb_kits/playbook/pb_currency/_currency.tsx +20 -7
  26. data/app/pb_kits/playbook/pb_currency/currency.rb +35 -8
  27. data/app/pb_kits/playbook/pb_currency/currency.test.js +47 -0
  28. data/app/pb_kits/playbook/pb_currency/docs/_currency_variants.html.erb +1 -1
  29. data/app/pb_kits/playbook/pb_currency/docs/_currency_variants.jsx +1 -1
  30. data/app/pb_kits/playbook/pb_currency/docs/_currency_variants.md +1 -0
  31. data/app/pb_kits/playbook/pb_date_picker/date_picker.rb +2 -0
  32. data/app/pb_kits/playbook/pb_distribution_bar/docs/_distribution_bar_custom_colors.md +1 -1
  33. data/app/pb_kits/playbook/pb_dropdown/docs/_dropdown_with_custom_active_style_options_react.md +1 -1
  34. data/app/pb_kits/playbook/pb_filter/docs/_filter_max_width.md +1 -1
  35. data/app/pb_kits/playbook/pb_form/pb_form_validation.js +44 -11
  36. data/app/pb_kits/playbook/pb_form_pill/docs/_form_pill_truncated_text_rails.md +1 -1
  37. data/app/pb_kits/playbook/pb_form_pill/docs/_form_pill_truncated_text_react.md +1 -1
  38. data/app/pb_kits/playbook/pb_icon/docs/_icon_color.md +1 -1
  39. data/app/pb_kits/playbook/pb_icon_circle/docs/_icon_circle_color.md +1 -1
  40. data/app/pb_kits/playbook/pb_icon_stat_value/docs/_icon_stat_value_color.html.erb +7 -14
  41. data/app/pb_kits/playbook/pb_icon_stat_value/docs/_icon_stat_value_color.jsx +6 -15
  42. data/app/pb_kits/playbook/pb_layout/docs/_layout_collection.md +1 -1
  43. data/app/pb_kits/playbook/pb_nav/docs/_collapsible_nav_item_spacing.md +1 -1
  44. data/app/pb_kits/playbook/pb_nav/docs/_nav_with_spacing_control.md +1 -1
  45. data/app/pb_kits/playbook/pb_overlay/docs/_overlay_layout.md +1 -1
  46. data/app/pb_kits/playbook/pb_phone_number_input/_phone_number_input.tsx +110 -17
  47. data/app/pb_kits/playbook/pb_pill/docs/_description.md +1 -1
  48. data/app/pb_kits/playbook/pb_popover/_popover.tsx +69 -34
  49. data/app/pb_kits/playbook/pb_popover/docs/_popover_append_to.jsx +68 -0
  50. data/app/pb_kits/playbook/pb_popover/docs/_popover_append_to_react.md +1 -0
  51. data/app/pb_kits/playbook/pb_popover/docs/example.yml +1 -0
  52. data/app/pb_kits/playbook/pb_popover/docs/index.js +1 -0
  53. data/app/pb_kits/playbook/pb_popover/popover.test.js +80 -0
  54. data/app/pb_kits/playbook/pb_rich_text_editor/docs/_description.md +1 -1
  55. data/app/pb_kits/playbook/pb_section_separator/docs/_description.md +1 -1
  56. data/app/pb_kits/playbook/pb_selectable_card/_selectable_card.scss +29 -0
  57. data/app/pb_kits/playbook/pb_skeleton_loading/docs/_skeleton_loading_border_radius_rails.md +1 -1
  58. data/app/pb_kits/playbook/pb_skeleton_loading/docs/_skeleton_loading_border_radius_react.md +1 -1
  59. data/app/pb_kits/playbook/pb_skeleton_loading/docs/_skeleton_loading_layout.md +1 -1
  60. data/app/pb_kits/playbook/pb_table/docs/_table_side_highlight.md +1 -1
  61. data/app/pb_kits/playbook/pb_table/docs/_table_sm.md +1 -1
  62. data/app/pb_kits/playbook/pb_table/docs/_table_with_dynamic_collapsible.html.erb +63 -0
  63. data/app/pb_kits/playbook/pb_table/docs/_table_with_dynamic_collapsible.jsx +89 -0
  64. data/app/pb_kits/playbook/pb_table/docs/_table_with_dynamic_collapsible_rails.md +4 -0
  65. data/app/pb_kits/playbook/pb_table/docs/_table_with_dynamic_collapsible_react.md +3 -0
  66. data/app/pb_kits/playbook/pb_table/docs/example.yml +2 -0
  67. data/app/pb_kits/playbook/pb_table/docs/index.js +1 -0
  68. data/app/pb_kits/playbook/pb_text_input/text_input.rb +2 -2
  69. data/app/pb_kits/playbook/pb_typeahead/_typeahead.test.jsx +41 -0
  70. data/app/pb_kits/playbook/pb_typeahead/_typeahead.tsx +23 -9
  71. data/app/pb_kits/playbook/pb_typeahead/docs/_typeahead_custom_options.html.erb +64 -0
  72. data/app/pb_kits/playbook/pb_typeahead/docs/_typeahead_custom_options.jsx +70 -0
  73. data/app/pb_kits/playbook/pb_typeahead/docs/_typeahead_custom_options.md +1 -0
  74. data/app/pb_kits/playbook/pb_typeahead/docs/_typeahead_default_options.html.erb +67 -1
  75. data/app/pb_kits/playbook/pb_typeahead/docs/_typeahead_default_value.jsx +68 -6
  76. data/app/pb_kits/playbook/pb_typeahead/docs/_typeahead_truncated_text.md +1 -1
  77. data/app/pb_kits/playbook/pb_typeahead/docs/example.yml +2 -0
  78. data/app/pb_kits/playbook/pb_typeahead/docs/index.js +2 -1
  79. data/dist/chunks/{_line_graph-h5H-imfn.js → _line_graph-CZlP3Ci7.js} +1 -1
  80. data/dist/chunks/_typeahead-gfPOrCeR.js +6 -0
  81. data/dist/chunks/{_weekday_stacked-Bd3ro1uX.js → _weekday_stacked-qGKMxoAo.js} +2 -2
  82. data/dist/chunks/pb_form_validation-CbtOLNdG.js +1 -0
  83. data/dist/chunks/vendor.js +1 -1
  84. data/dist/playbook-doc.js +2 -2
  85. data/dist/playbook-rails-react-bindings.js +1 -1
  86. data/dist/playbook-rails.js +1 -1
  87. data/dist/playbook.css +1 -1
  88. data/lib/playbook/version.rb +1 -1
  89. metadata +17 -7
  90. data/dist/chunks/_typeahead-U8AjZIIW.js +0 -6
  91. data/dist/chunks/pb_form_validation-DebqlUKZ.js +0 -1
  92. /data/app/pb_kits/playbook/pb_popover/docs/{_popover_append_to.md → _popover_append_to_rails.md} +0 -0
@@ -8,13 +8,13 @@ const ERROR_MESSAGE_SELECTOR = '.pb_body_kit_negative'
8
8
  // Validation selectors
9
9
  const FORM_SELECTOR = 'form[data-pb-form-validation="true"]'
10
10
  const REQUIRED_FIELDS_SELECTOR = 'input[required],textarea[required],select[required]'
11
+ const PHONE_NUMBER_VALIDATION_ERROR_SELECTOR = '[data-pb-phone-validation-error="true"]'
11
12
 
12
13
  const FIELD_EVENTS = [
13
14
  'change',
14
15
  'valid',
15
16
  'invalid',
16
17
  ]
17
-
18
18
  class PbFormValidation extends PbEnhancedElement {
19
19
  static get selector() {
20
20
  return FORM_SELECTOR
@@ -22,12 +22,27 @@ class PbFormValidation extends PbEnhancedElement {
22
22
 
23
23
  connect() {
24
24
  this.formValidationFields.forEach((field) => {
25
+ // Skip phone number inputs - they handle their own validation
26
+ const isPhoneNumberInput = field.closest('.pb_phone_number_input')
27
+ if (isPhoneNumberInput) return
28
+
25
29
  FIELD_EVENTS.forEach((e) => {
26
30
  field.addEventListener(e, debounce((event) => {
27
31
  this.validateFormField(event)
28
32
  }, 250), false)
29
33
  })
30
34
  })
35
+
36
+ // Add event listener to check for phone number validation errors
37
+ this.element.addEventListener('submit', (event) => {
38
+ // Use setTimeout to ensure React state updates have completed
39
+ setTimeout(() => {
40
+ if (this.hasPhoneNumberValidationErrors()) {
41
+ event.preventDefault()
42
+ return false
43
+ }
44
+ }, 0)
45
+ })
31
46
  }
32
47
 
33
48
  validateFormField(event) {
@@ -45,40 +60,58 @@ class PbFormValidation extends PbEnhancedElement {
45
60
 
46
61
  showValidationMessage(target) {
47
62
  const { parentElement } = target
63
+ const kitElement = parentElement.closest(KIT_SELECTOR)
64
+
65
+ // FIX: Add null check for kitElement
66
+ if (!kitElement) return
67
+
68
+ // Check if this is a phone number input
69
+ const isPhoneNumberInput = kitElement.classList.contains('pb_phone_number_input')
48
70
 
49
71
  // ensure clean error message state
50
72
  this.clearError(target)
51
- parentElement.closest(KIT_SELECTOR).classList.add('error')
73
+ kitElement.classList.add('error')
52
74
 
53
- // set the error message element
54
- const errorMessageContainer = this.errorMessageContainer
75
+ // Only add error message if it's NOT a phone number input
76
+ if (!isPhoneNumberInput) {
77
+ // set the error message element
78
+ const errorMessageContainer = this.errorMessageContainer
55
79
 
56
- if (target.dataset.message) target.setCustomValidity(target.dataset.message)
80
+ if (target.dataset.message) target.setCustomValidity(target.dataset.message)
57
81
 
58
- errorMessageContainer.innerHTML = target.validationMessage
82
+ errorMessageContainer.innerHTML = target.validationMessage
59
83
 
60
- // add the error message element to the dom tree
61
- parentElement.appendChild(errorMessageContainer)
84
+ // add the error message element to the dom tree
85
+ parentElement.appendChild(errorMessageContainer)
86
+ }
62
87
  }
63
88
 
64
89
  clearError(target) {
65
90
  const { parentElement } = target
66
- parentElement.closest(KIT_SELECTOR).classList.remove('error')
91
+ const kitElement = parentElement.closest(KIT_SELECTOR)
92
+ // Remove error class from kit element
93
+ if (kitElement) kitElement.classList.remove('error')
94
+ // Remove error message from parent element
67
95
  const errorMessageContainer = parentElement.querySelector(ERROR_MESSAGE_SELECTOR)
68
96
  if (errorMessageContainer) errorMessageContainer.remove()
69
97
  }
70
98
 
99
+ // Check if there are phone number input errors
100
+ hasPhoneNumberValidationErrors() {
101
+ const phoneNumberErrors = this.element.querySelectorAll(PHONE_NUMBER_VALIDATION_ERROR_SELECTOR)
102
+ return phoneNumberErrors.length > 0
103
+ }
104
+
71
105
  get errorMessageContainer() {
72
106
  const errorContainer = document.createElement('div')
73
107
  const kitClassName = ERROR_MESSAGE_SELECTOR.replace(/\./, '')
74
108
  errorContainer.classList.add(kitClassName)
75
109
  return errorContainer
76
110
  }
77
-
78
111
  get formValidationFields() {
79
112
  return this._formValidationFields =
80
113
  this._formValidationFields || this.element.querySelectorAll(REQUIRED_FIELDS_SELECTOR)
81
114
  }
82
115
  }
83
116
 
84
- window.PbFormValidation = PbFormValidation
117
+ window.PbFormValidation = PbFormValidation
@@ -1,3 +1,3 @@
1
- For Form Pills with longer text, the truncate global prop can be used to truncate the label within each Form Pill. See [here](https://playbook.powerapp.cloud/visual_guidelines/truncate) for more information on the truncate global prop.
1
+ For Form Pills with longer text, the truncate global prop can be used to truncate the label within each Form Pill. See [here](https://playbook.powerapp.cloud/global_props/truncate) for more information on the truncate global prop.
2
2
 
3
3
  __NOTE__: For Rails Form Pills (not ones embedded within a React-rendered Typeahead or MultiLevelSelect), a unique `id` is required to enable the Tooltip functionality displaying the text or tag section of the Form Pill.
@@ -1 +1 @@
1
- For Form Pills with longer text, the `truncate` global prop can be used to truncate the label within each Form Pill. Hover over the truncated Form Pill and a Tooltip containing the text or tag section of the Form Pill will appear. See [here](https://playbook.powerapp.cloud/visual_guidelines/truncate) for more information on the truncate global prop.
1
+ For Form Pills with longer text, the `truncate` global prop can be used to truncate the label within each Form Pill. Hover over the truncated Form Pill and a Tooltip containing the text or tag section of the Form Pill will appear. See [here](https://playbook.powerapp.cloud/global_props/truncate) for more information on the truncate global prop.
@@ -1 +1 @@
1
- Pass any text, status, data, product, or category Playbook <a href="https://playbook.powerapp.cloud/visual_guidelines/colors" target="_blank">color token</a> to the `color` prop to change any icon's color.
1
+ Pass any text, status, data, product, or category Playbook <a href="https://playbook.powerapp.cloud/tokens/colors" target="_blank">color token</a> to the `color` prop to change any icon's color.
@@ -1 +1 @@
1
- Customize your icon circle by passing one of our seven base colors to the `variant` prop: `royal` `blue` `orange` `purple` `teal` `red` `yellow` `green`
1
+ Customize your icon circle by passing one of our seven base colors to the `variant` prop: `royal` `orange` `purple` `teal` `red` `yellow` `green`
@@ -3,48 +3,41 @@
3
3
  text: "Mercury",
4
4
  unit: "AU",
5
5
  value: 0.39,
6
- variant:"blue"
6
+ variant:"royal"
7
7
  }) %>
8
8
  <br>
9
9
  <%= pb_rails("icon_stat_value", props: { icon: "planet-ringed",
10
10
  text: "Venus",
11
11
  unit: "AU",
12
12
  value: 0.723,
13
- variant:"royal"
13
+ variant:"purple"
14
14
  }) %>
15
15
  <br>
16
16
  <%= pb_rails("icon_stat_value", props: { icon: "planet-moon",
17
17
  text: "Earth",
18
18
  unit: "AU",
19
19
  value: 1.0,
20
- variant:"purple"
20
+ variant:"teal"
21
21
  }) %>
22
22
  <br>
23
23
  <%= pb_rails("icon_stat_value", props: { icon: "solar-system",
24
24
  text: "Mars",
25
25
  unit: "AU",
26
26
  value: 1.524,
27
- variant:"teal"
27
+ variant:"red"
28
28
  }) %>
29
29
  <br>
30
30
  <%= pb_rails("icon_stat_value", props: { icon: "globe-americas",
31
- text: "Jupitar",
31
+ text: "Jupiter",
32
32
  unit: "AU",
33
33
  value: 5.203,
34
- variant:"red"
34
+ variant:"yellow"
35
35
  }) %>
36
36
  <br>
37
37
  <%= pb_rails("icon_stat_value", props: { icon: "globe-africa",
38
38
  text: "Saturn",
39
39
  unit: "AU",
40
40
  value: 9.539,
41
- variant:"yellow"
42
- }) %>
43
- <br>
44
- <%= pb_rails("icon_stat_value", props: { icon: "globe",
45
- text: "Uranus",
46
- unit: "AU",
47
- value: 19.18,
48
41
  variant:"green"
49
42
  }) %>
50
43
  <br>
@@ -53,4 +46,4 @@
53
46
  unit: "AU",
54
47
  value: 19.18,
55
48
  variant:"orange"
56
- }) %>
49
+ }) %>
@@ -9,7 +9,7 @@ const IconStatValueColor = (props) => {
9
9
  text="Mercury"
10
10
  unit="AU"
11
11
  value={0.39}
12
- variant="blue"
12
+ variant="royal"
13
13
  {...props}
14
14
  />
15
15
  <br />
@@ -18,7 +18,7 @@ const IconStatValueColor = (props) => {
18
18
  text="Venus"
19
19
  unit="AU"
20
20
  value={0.723}
21
- variant="royal"
21
+ variant="purple"
22
22
  {...props}
23
23
  />
24
24
  <br />
@@ -27,7 +27,7 @@ const IconStatValueColor = (props) => {
27
27
  text="Earth"
28
28
  unit="AU"
29
29
  value={1.0}
30
- variant="purple"
30
+ variant="teal"
31
31
  {...props}
32
32
  />
33
33
  <br />
@@ -36,16 +36,16 @@ const IconStatValueColor = (props) => {
36
36
  text="Mars"
37
37
  unit="AU"
38
38
  value={1.524}
39
- variant="teal"
39
+ variant="red"
40
40
  {...props}
41
41
  />
42
42
  <br />
43
43
  <IconStatValue
44
44
  icon="globe-americas"
45
- text="Jupitar"
45
+ text="Jupiter"
46
46
  unit="AU"
47
47
  value={5.203}
48
- variant="red"
48
+ variant="yellow"
49
49
  {...props}
50
50
  />
51
51
  <br />
@@ -54,15 +54,6 @@ const IconStatValueColor = (props) => {
54
54
  text="Saturn"
55
55
  unit="AU"
56
56
  value={9.539}
57
- variant="yellow"
58
- {...props}
59
- />
60
- <br />
61
- <IconStatValue
62
- icon="globe"
63
- text="Uranus"
64
- unit="AU"
65
- value={19.18}
66
57
  variant="green"
67
58
  {...props}
68
59
  />
@@ -1 +1 @@
1
- Layout can leverage the max-width property. Learn more in our <a href="https://playbook.powerapp.cloud/visual_guidelines" target="_blank">visual guidelines.</a>
1
+ Layout can leverage the max-width property. Learn more in our <a href="https://playbook.powerapp.cloud/global_props/max_width" target="_blank">visual guidelines.</a>
@@ -1 +1 @@
1
- The `itemSpacing` prop can optionally be applied to the `Nav` parent element and accepts an object through which any of our [Spacing](https://playbook.powerapp.cloud/visual_guidelines/spacing) (padding, margin) global prop values can be passed as an object. All spacing values passed to `itemSpacing` will be applied to all navItems within the nav. Spacing (padding, margin) global props can still be used on nested navItems in conjunction with `itenSpacing` on the Nav for customized control. Any Spacing (padding,margin) global props applied directly to a navItem will override any competing `itemSpacing` value.
1
+ The `itemSpacing` prop can optionally be applied to the `Nav` parent element and accepts an object through which any of our Spacing ([padding](https://playbook.powerapp.cloud/global_props/padding), [margin](https://playbook.powerapp.cloud/global_props/margin)) global prop values can be passed as an object. All spacing values passed to `itemSpacing` will be applied to all navItems within the nav. Spacing (padding, margin) global props can still be used on nested navItems in conjunction with `itenSpacing` on the Nav for customized control. Any Spacing (padding,margin) global props applied directly to a navItem will override any competing `itemSpacing` value.
@@ -1 +1 @@
1
- All Nav variants' navItems accept our [global Spacing](https://playbook.powerapp.cloud/visual_guidelines/spacing) (padding, margin) props for custom spacing requirements. This example uses paddingY="xxs" and margin="none" on the bold variant to show this in action!
1
+ All Nav variants' navItems accept our global Spacing ([padding](https://playbook.powerapp.cloud/global_props/padding), [margin](https://playbook.powerapp.cloud/global_props/margin)) props for custom spacing requirements. This example uses paddingY="xxs" and margin="none" on the bold variant to show this in action!
@@ -2,4 +2,4 @@ The optional `layout` prop accepts the `position` and `size` of the overlay as a
2
2
 
3
3
  The `position` key accepts `bottom` (default), `top`, `y` (for both top and bottom) `right`, `left`, or `x` (for both left and right), which sets the side(s) where the `color` overlay starts. The direction of the overlay is always toward the opposite side of the position. For example, the default position of `bottom` starts the overlay on the bottom edge of your container and extends it toward the opposite side: the top.
4
4
 
5
- The `size` value is `full` (100%) by default, but accepts our [spacing tokens](https://playbook.powerapp.cloud/visual_guidelines/spacing) or a percentage value as a string, and literally translates to how much of the container is covered by the overlay(s).
5
+ The `size` value is `full` (100%) by default, but accepts our spacing tokens([padding](https://playbook.powerapp.cloud/global_props/padding), [margin](https://playbook.powerapp.cloud/global_props/margin)) or a percentage value as a string, and literally translates to how much of the container is covered by the overlay(s).
@@ -55,7 +55,6 @@ const formatToGlobalCountryName = (countryName: string) => {
55
55
 
56
56
  const formatAllCountries = () => {
57
57
  const countryData = intlTelInput.getCountryData()
58
-
59
58
  for (let i = 0; i < countryData.length; i++) {
60
59
  const country = countryData[i]
61
60
  country.name = formatToGlobalCountryName(country.name)
@@ -110,18 +109,54 @@ const PhoneNumberInput = (props: PhoneNumberInputProps, ref?: React.Ref<unknown>
110
109
 
111
110
  const inputRef = useRef<HTMLInputElement | null>(null)
112
111
  const itiRef = useRef<any>(null);
112
+ const wrapperRef = useRef<HTMLDivElement | null>(null);
113
113
  const [inputValue, setInputValue] = useState(value)
114
114
  const [error, setError] = useState(props.error || "")
115
115
  const [dropDownIsOpen, setDropDownIsOpen] = useState(false)
116
116
  const [selectedData, setSelectedData] = useState()
117
117
  const [hasTyped, setHasTyped] = useState(false)
118
+ const [formSubmitted, setFormSubmitted] = useState(false)
119
+ const [hasStartedValidating, setHasStartedValidating] = useState(false)
120
+
121
+ // Only sync initial error from props, not continuous updates
122
+ // Once validation starts, internal validation takes over
123
+ useEffect(() => {
124
+ if (props.error && !hasStartedValidating) {
125
+ setError(props.error)
126
+ // If there's an initial error from props, mark as submitted so it shows
127
+ if (props.error) {
128
+ setFormSubmitted(true)
129
+ }
130
+ }
131
+ }, [props.error, hasStartedValidating])
132
+
133
+ // Function to update validation state on the wrapper element
134
+ // Only applies when input is required
135
+ const updateValidationState = (hasError: boolean) => {
136
+ if (wrapperRef.current && required) {
137
+ if (hasError) {
138
+ wrapperRef.current.setAttribute('data-pb-phone-validation-error', 'true')
139
+ } else {
140
+ wrapperRef.current.removeAttribute('data-pb-phone-validation-error')
141
+ }
142
+ }
143
+ }
144
+
145
+ // Determine which error to display
146
+ // Show internal errors on blur (hasTyped) or on form submission (formSubmitted)
147
+ const shouldShowInternalError = (hasTyped || formSubmitted) && required && error
148
+ const displayError = shouldShowInternalError ? error : ""
118
149
 
119
150
  useEffect(() => {
120
- if ((error ?? '').length > 0) {
151
+ const hasError = (error ?? '').length > 0
152
+ if (hasError) {
121
153
  onValidate(false)
122
154
  } else {
123
155
  onValidate(true)
124
156
  }
157
+
158
+ // Update validation state whenever error changes
159
+ updateValidationState(hasError)
125
160
  }, [error, onValidate])
126
161
 
127
162
  const unformatNumber = (formattedNumber: any) => {
@@ -137,6 +172,7 @@ const PhoneNumberInput = (props: PhoneNumberInputProps, ref?: React.Ref<unknown>
137
172
 
138
173
  const validateTooLongNumber = (itiInit: any) => {
139
174
  if (!itiInit) return
175
+
140
176
  if (itiInit.getValidationError() === ValidationError.TooLong) {
141
177
  return showFormattedError('too long')
142
178
  } else {
@@ -146,13 +182,11 @@ const PhoneNumberInput = (props: PhoneNumberInputProps, ref?: React.Ref<unknown>
146
182
 
147
183
  const validateTooShortNumber = (itiInit: any) => {
148
184
  if (!itiInit) return
149
-
150
185
  // If field is empty, don't show "too short" error
151
186
  if (!inputValue || inputValue.trim() === '') {
152
187
  setError('')
153
188
  return false
154
189
  }
155
-
156
190
  if (itiInit.getValidationError() === ValidationError.TooShort) {
157
191
  return showFormattedError('too short')
158
192
  } else {
@@ -172,7 +206,7 @@ const PhoneNumberInput = (props: PhoneNumberInputProps, ref?: React.Ref<unknown>
172
206
  }
173
207
 
174
208
  const validateUnhandledError = (itiInit: any) => {
175
- if (!itiInit) return
209
+ if (!required || !itiInit) return
176
210
  if (itiInit.getValidationError() === ValidationError.SomethingWentWrong) {
177
211
  if (inputValue.length === 1) {
178
212
  return showFormattedError('too short')
@@ -184,7 +218,6 @@ const PhoneNumberInput = (props: PhoneNumberInputProps, ref?: React.Ref<unknown>
184
218
  }
185
219
  }
186
220
  }
187
-
188
221
  const validateMissingAreaCode = (itiInit: any) => {
189
222
  if (!itiInit) return
190
223
  if (itiInit.getValidationError() === ValidationError.MissingAreaCode) {
@@ -201,8 +234,9 @@ const PhoneNumberInput = (props: PhoneNumberInputProps, ref?: React.Ref<unknown>
201
234
  }
202
235
  }
203
236
 
237
+ // Validation for required empty fields
204
238
  const validateRequiredField = () => {
205
- if (!inputValue || inputValue.trim() === '') {
239
+ if (required && (!inputValue || inputValue.trim() === '')) {
206
240
  setError('Missing phone number')
207
241
  return true
208
242
  }
@@ -210,14 +244,24 @@ const PhoneNumberInput = (props: PhoneNumberInputProps, ref?: React.Ref<unknown>
210
244
  }
211
245
 
212
246
  const validateErrors = () => {
213
- // If field is empty, show error message
247
+ // Signal validation has started, so prop errors won't override internal validation
248
+ if (!hasStartedValidating) {
249
+ setHasStartedValidating(true)
250
+ }
251
+
252
+ // If field is empty, only show required field error if applicable
214
253
  if (!inputValue || inputValue.trim() === '') {
215
254
  if (validateRequiredField()) return
255
+ // Clear any existing errors if field is empty and not required
256
+ if (!required) {
257
+ setError('')
258
+ }
216
259
  return
217
260
  }
218
261
 
219
- if (!hasTyped && !error) return
262
+ if (!hasTyped && !error) return
220
263
 
264
+ // Run validation checks
221
265
  if (itiRef.current) isValid(itiRef.current.isValidNumber())
222
266
  if (validateOnlyNumbers(itiRef.current)) return
223
267
  if (validateTooLongNumber(itiRef.current)) return
@@ -227,6 +271,29 @@ const PhoneNumberInput = (props: PhoneNumberInputProps, ref?: React.Ref<unknown>
227
271
  if (validateRepeatCountryCode(itiRef.current)) return
228
272
  }
229
273
 
274
+ // Add listener for form validation to track when validation should be shown
275
+ useEffect(() => {
276
+ const handleInvalid = (event: Event) => {
277
+ const target = event.target as HTMLInputElement
278
+ const phoneNumberContainer = target.closest('.pb_phone_number_input')
279
+
280
+ if (phoneNumberContainer && phoneNumberContainer === wrapperRef.current) {
281
+ const invalidInputName = target.name || target.getAttribute('name')
282
+ if (invalidInputName === name) {
283
+ setFormSubmitted(true)
284
+ // Trigger validation when form is submitted
285
+ validateErrors()
286
+ }
287
+ }
288
+ }
289
+
290
+ document.addEventListener('invalid', handleInvalid, true)
291
+
292
+ return () => {
293
+ document.removeEventListener('invalid', handleInvalid, true)
294
+ }
295
+ }, [name, inputValue])
296
+
230
297
  /*
231
298
  useImperativeHandle exposes the kit's input element to a parent component via a ref.
232
299
  See the Playbook docs for use cases.
@@ -238,6 +305,12 @@ const PhoneNumberInput = (props: PhoneNumberInputProps, ref?: React.Ref<unknown>
238
305
  setInputValue("")
239
306
  setError("")
240
307
  setHasTyped(false)
308
+ setFormSubmitted(false)
309
+ setHasStartedValidating(false)
310
+ // Only clear validation state if field was required
311
+ if (required) {
312
+ updateValidationState(false)
313
+ }
241
314
  },
242
315
  inputNode() {
243
316
  return inputRef.current
@@ -247,6 +320,12 @@ const PhoneNumberInput = (props: PhoneNumberInputProps, ref?: React.Ref<unknown>
247
320
  // Run validation and return error message or true
248
321
  const isEmpty = !inputValue || inputValue.trim() === ''
249
322
 
323
+ if (required && isEmpty) {
324
+ setError('Missing phone number')
325
+ setFormSubmitted(true)
326
+ return 'Missing phone number'
327
+ }
328
+
250
329
  if (isEmpty) {
251
330
  // Show missing phone number error
252
331
  const errorMessage = 'Missing phone number'
@@ -266,6 +345,7 @@ const PhoneNumberInput = (props: PhoneNumberInputProps, ref?: React.Ref<unknown>
266
345
  const countryName = itiRef.current.getSelectedCountryData().name
267
346
  const errorMessage = `Invalid ${countryName} phone number (repeat country code)`
268
347
  setError(errorMessage)
348
+ setFormSubmitted(true)
269
349
  setHasTyped(true)
270
350
  return errorMessage
271
351
  }
@@ -275,6 +355,7 @@ const PhoneNumberInput = (props: PhoneNumberInputProps, ref?: React.Ref<unknown>
275
355
  const countryName = itiRef.current.getSelectedCountryData().name
276
356
  const errorMessage = `Invalid ${countryName} phone number (enter numbers only)`
277
357
  setError(errorMessage)
358
+ setFormSubmitted(true)
278
359
  setHasTyped(true)
279
360
  return errorMessage
280
361
  }
@@ -295,7 +376,9 @@ const PhoneNumberInput = (props: PhoneNumberInputProps, ref?: React.Ref<unknown>
295
376
  errorMessage = `Invalid ${countryName} phone number`
296
377
  }
297
378
 
379
+ // Set the error state so the validation attribute gets added
298
380
  setError(errorMessage)
381
+ setFormSubmitted(true)
299
382
  setHasTyped(true)
300
383
 
301
384
  return errorMessage
@@ -314,11 +397,16 @@ const PhoneNumberInput = (props: PhoneNumberInputProps, ref?: React.Ref<unknown>
314
397
 
315
398
  const handleOnChange = (evt: React.ChangeEvent<HTMLInputElement>) => {
316
399
  if (!hasTyped) setHasTyped(true)
317
-
318
400
  setInputValue(evt.target.value)
319
401
 
402
+ // Reset form submitted state when user types
403
+ if (formSubmitted) {
404
+ setFormSubmitted(false)
405
+ }
406
+
320
407
  let phoneNumberData
321
408
 
409
+ // Handle formatAsYouType with input event
322
410
  if (formatAsYouType) {
323
411
  const formattedPhoneNumberData = getCurrentSelectedData(itiRef.current, evt.target.value)
324
412
  phoneNumberData = {...formattedPhoneNumberData, number: unformatNumber(formattedPhoneNumberData.number)}
@@ -329,12 +417,15 @@ const PhoneNumberInput = (props: PhoneNumberInputProps, ref?: React.Ref<unknown>
329
417
  setSelectedData(phoneNumberData)
330
418
  onChange(phoneNumberData)
331
419
  isValid(itiRef.current.isValidNumber())
420
+
421
+ // Trigger validation after onChange for React Hook Form
422
+ // This ensures validation state is up-to-date
423
+ setTimeout(() => validateErrors(), 0)
332
424
  }
333
425
 
334
426
  // Separating Concerns as React Docs Recommend
335
427
  // This also Fixes things for our react_component rendering on the Rails Side
336
428
  useEffect(formatAllCountries, [])
337
-
338
429
  // If an initial country is not specified, the "globe" icon will show
339
430
  // Always set a country
340
431
  const fallbackCountry =
@@ -375,9 +466,9 @@ const PhoneNumberInput = (props: PhoneNumberInputProps, ref?: React.Ref<unknown>
375
466
  inputRef.current.addEventListener("open:countrydropdown", () => setDropDownIsOpen(true))
376
467
  inputRef.current.addEventListener("close:countrydropdown", () => setDropDownIsOpen(false))
377
468
 
378
- // Handle formatAsYouType with input event
379
- if (formatAsYouType) {
380
- inputRef.current.addEventListener("input", (evt: Event) => {
469
+ // Handle formatAsYouType with input event
470
+ if (formatAsYouType) {
471
+ inputRef.current.addEventListener("input", (evt: Event) => {
381
472
  const target = evt.target as HTMLInputElement
382
473
  const formattedValue = target.value
383
474
 
@@ -396,13 +487,12 @@ const PhoneNumberInput = (props: PhoneNumberInputProps, ref?: React.Ref<unknown>
396
487
  }
397
488
  }
398
489
  }, [])
399
-
400
490
  let textInputProps: {[key: string]: any} = {
401
491
  className: dropDownIsOpen ? 'dropdown_open' : '',
402
492
  dark,
403
493
  "data-phone-number": JSON.stringify(selectedData),
404
494
  disabled,
405
- error: hasTyped ? error : props.error,
495
+ error: hasTyped ? error : props.error || displayError,
406
496
  type: 'tel',
407
497
  id,
408
498
  label,
@@ -412,7 +502,10 @@ const PhoneNumberInput = (props: PhoneNumberInputProps, ref?: React.Ref<unknown>
412
502
  value: inputValue
413
503
  }
414
504
 
415
- let wrapperProps: Record<string, unknown> = { className: classes }
505
+ let wrapperProps: Record<string, unknown> = {
506
+ className: classes,
507
+ ref: wrapperRef
508
+ }
416
509
 
417
510
  if (!isEmpty(aria)) textInputProps = {...textInputProps, ...ariaProps}
418
511
  if (!isEmpty(data)) wrapperProps = {...wrapperProps, ...dataProps}
@@ -1 +1 @@
1
- A pill uses both a keyword and a specific color to categorize an item. Each pill directly corresponds to a data color <a href="https://playbook.powerapp.cloud/visual_guidelines/colors" target="_blank">here</a>.
1
+ A pill uses both a keyword and a specific color to categorize an item. Each pill directly corresponds to a data color <a href="https://playbook.powerapp.cloud/global_props/colors" target="_blank">here</a>.