playbook_ui 16.2.0.pre.rc.2 → 16.2.0.pre.rc.3

data/app/pb_kits/playbook/pb_multi_level_select/docs/_multi_level_select_label.html.erb

@@ -65,7 +65,7 @@
 }] %>
 
 <%= pb_rails("multi_level_select", props: {
-  id: "select_a_department",
-  label: "Select a Department",
-  tree_data: treeData
+    id: "multi-level-select-label-rails",
+    label: "Select a department",
+    tree_data: treeData
 }) %>

data/app/pb_kits/playbook/pb_multi_level_select/docs/_multi_level_select_label.jsx

@@ -73,11 +73,14 @@ const MultiLevelSelectDefault = (props) => {
   return (
     <div>
       <MultiLevelSelect
-          id="select_a_department"
+          id='multiselect-label'
           label="Select a Department"
           onSelect={(selectedNodes) =>
-            console.log("Selected Items", selectedNodes)
-          }
+          console.log(
+            "Selected Items",
+            selectedNodes
+          )
+        }
           treeData={treeData}
           {...props}
       />
@@ -85,4 +88,4 @@ const MultiLevelSelectDefault = (props) => {
   )
 };
 
-export default MultiLevelSelectDefault;
+export default MultiLevelSelectDefault;

data/app/pb_kits/playbook/pb_multi_level_select/multi_level_select.test.jsx

@@ -192,7 +192,7 @@ describe('MultiLevelSelect multi variant', () => {
       />
     )
     const kit = screen.getByTestId(testId)
-    const input = kit.querySelector('#multi-disabled-test_input')
+    const input = kit.querySelector('#multiselect_input')
     fireEvent.click(input)
     
     const disabledCheckbox = kit.querySelector('input[type="checkbox"][disabled]')
@@ -227,7 +227,7 @@ describe('MultiLevelSelect single variant', () => {
       />
     )
     const kit = screen.getByTestId(testId)
-    const input = kit.querySelector('#single-disabled-test_input')
+    const input = kit.querySelector('#multiselect_input')
     fireEvent.click(input)
     
     const disabledRadio = kit.querySelector('input[type="radio"][disabled]')
@@ -246,7 +246,7 @@ describe('MultiLevelSelect single variant', () => {
       />
     )
     const kit = screen.getByTestId(testId)
-    const input = kit.querySelector('#single-disabled-click-test_input')
+    const input = kit.querySelector('#multiselect_input')
     fireEvent.click(input)
     
     const disabledRadio = kit.querySelector('input[type="radio"][disabled]')
@@ -267,7 +267,7 @@ describe('MultiLevelSelect single variant', () => {
       />
     )
     const kit = screen.getByTestId(testId)
-    const input = kit.querySelector('#single-enabled-click-test_input')
+    const input = kit.querySelector('#multiselect_input')
     fireEvent.click(input)
     
     const enabledRadio = kit.querySelector('input[type="radio"]:not([disabled])')

data/app/pb_kits/playbook/pb_text_input/text_input.html.erb

@@ -1,16 +1,15 @@
-<%= pb_content_tag(:div, id: nil) do %>
+<%= pb_content_tag(:div, id: nil ) do  %>
   <% if object.label.present? %>
-    <label for="<%= object.input_options[:id] || object.id %>">
-      <% if object.required_indicator %>
-        <%= pb_rails("caption", props: { dark: object.dark, classname: "pb_text_input_kit_label" }) do %>
-          <%= object.label %><span style="color: #DA0014;"> *</span>
-        <% end %>
-      <% else %>
-        <%= pb_rails("caption", props: { text: object.label, dark: object.dark, classname: "pb_text_input_kit_label" }) %>
+  <label for="<%= object.input_options[:id] || object.id %>" >
+    <% if object.required_indicator %>
+      <%= pb_rails("caption", props: { dark: object.dark, classname: "pb_text_input_kit_label" }) do %>
+        <%= object.label %><span style="color: #DA0014;"> *</span>
       <% end %>
-    </label>
+    <% else %>
+    <%= pb_rails("caption", props: { text: object.label, dark: object.dark, classname: "pb_text_input_kit_label" }) %>
+    <% end %>
+  </label>
   <% end %>
-
   <%= content_tag(:div, class: "#{add_on_class} text_input_wrapper") do %>
     <% if content.present? %>
       <%= content %>
@@ -27,3 +26,4 @@
     <%= pb_rails("body", props: {dark: object.dark, status: "negative", text: object.error, id: object.error_id, aria: { atomic: "true", live: "polite" }, html_options: { role: "alert" }}) if object.error %>
   <% end %>
 <% end %>
+