playbook_ui 13.27.0.pre.alpha.PLAY888Cardkithighlightoverflowtooltiperror2930 → 13.27.0.pre.alpha.PLAY1122fixMultipleUserspaddingoverflow2844

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 84d66eed5b978ecd745e826ec17f388a4dbe57dfa989b153aca53faf75cded75
-  data.tar.gz: 5b9618559b5d5a6dd1ddae3523df8e9f8e690af6ac0e0cd711ccc6bb41756601
+  metadata.gz: 2c75da57f5a97adfc38df188be0c6a4de3e10c2577d811d1d47a491b4074e77c
+  data.tar.gz: 6c33e4eb580a12b2de7301c97fc447b2c003e87d95ce192f9b56116eef84edce
 SHA512:
-  metadata.gz: 605001caf6eb9108ec8456543bebb0123a7897a3a9c60a708db790712ddcbdd2ad064ae8520c80fd4eb545b61cbb11e53f06e338ddff87a85f766cb9db69c290
-  data.tar.gz: ac4df765d7761099f29b4e7c57d538473f52e6b0b0727e8e5783abcbfeee2cbf4e28a8c1bca55ac5f8ef98ad29550308ff339ab3f39b47713e1bee52b2721d95
+  metadata.gz: 2f0e697e201ac42ec4d36612b22c5891b877f25b2bf60ed86000023a5baca4fe18b4abae54fa9a0c133455c4a4383dfea7ec39ea3926ed38a728a5d86b2d369f
+  data.tar.gz: 26064871c9351cb46067a246824f534267328937e7de3bc47c38f9b891135e1ca8ecad0b1650f42f282e9b85ba8861ed1b950fa7d73988b70d609d5eb6124429

data/app/pb_kits/playbook/pb_advanced_table/advanced_table.html.erb

@@ -1,4 +1,8 @@
-<%= pb_content_tag do %>
+<%= content_tag(:div,
+      aria: object.aria,
+      class: object.classname,
+      data: object.data,
+      id: object.id) do %>
         <%= pb_rails("table", props: { size: "sm", tag:"div", data_table: true, number_spacing:"tabular", responsive:"none" }) do %>
           <% if content.present? %>
             <% content.presence %>

data/app/pb_kits/playbook/pb_advanced_table/table_body.html.erb

@@ -1,4 +1,8 @@
-<%= pb_content_tag do %>
+<%= content_tag(:div,
+      aria: object.aria,
+      class: object.classname,
+      data: object.data,
+      id: object.id) do %>
         <% object.table_data.each do |row| %>
           <%= render_row_and_children(row, object.column_definitions) %>
         <% end %>

data/app/pb_kits/playbook/pb_advanced_table/table_header.html.erb

@@ -1,4 +1,8 @@
-<%= pb_content_tag do %>
+<%= content_tag(:div,
+      aria: object.aria,
+      class: object.classname,
+      data: object.data,
+      id: object.id) do %>
         <%= pb_rails("table/table_row", props: {tag:"div"}) do %>
             <% object.column_definitions.each_with_index do |item, index| %>
                 <%= pb_rails("table/table_header", props: { tag:"div", id:item[:accessor], classname:object.th_classname}) do %>

data/app/pb_kits/playbook/pb_advanced_table/table_row.html.erb

@@ -1,4 +1,8 @@
-<%= pb_content_tag do %>
+<%= content_tag(:div,
+      aria: object.aria,
+      class: object.classname,
+      data: object.data,
+      id: object.id) do %>
         <% object.column_definitions.each_with_index do |column, index| %>
             <%= pb_rails("table/table_cell", props: { tag:"div", classname:object.td_classname}) do %>
                 <%= pb_rails("flex", props:{ align: "center", justify: index.zero? ? "start" : "end" }) do %>

data/app/pb_kits/playbook/pb_avatar/Utilities/GetPlacementPropsHelper.tsx

@@ -9,10 +9,6 @@ export const getPlacementProps = (placement: string, size?: "md" | "lg" | "sm" |
               'bottom-left': { bottom: 'xs', left: 'xs' },
               'top-left': { top: 'xs', left: 'xs' },
               'bottom-right': { bottom: 'xs', right: 'xs' },
-              'bottom-center': { bottom: "xs", className: "overlay_bottom_center" },
-              'top-center': { top: "xs", className: "overlay_top_center" },
-              'left-center': { left: "sm", className: "overlay_left_center" },
-              'right-center': { right: "sm", className: "overlay_right_center" }
           };
           break;
       
@@ -22,10 +18,6 @@ export const getPlacementProps = (placement: string, size?: "md" | "lg" | "sm" |
               'bottom-left': { bottom: '0', left: 'xs' },
               'top-left': { top: '0', left: 'xs' },
               'bottom-right': { bottom: '0', right: 'xs' },
-              'bottom-center': { bottom: "xs", className: "overlay_bottom_center" },
-              'top-center': { top: "xs", className: "overlay_top_center" },
-              'left-center': { left: "sm", className: "overlay_left_center" },
-              'right-center': { right: "sm", className: "overlay_right_center" }
           };
           break;
       
@@ -36,10 +28,6 @@ export const getPlacementProps = (placement: string, size?: "md" | "lg" | "sm" |
               'bottom-left': { bottom: '0', left: '0' },
               'top-left': { top: '0', left: '0' },
               'bottom-right': { bottom: '0', right: '0' },
-              'bottom-center': { bottom: "xs", className: "overlay_bottom_center" },
-              'top-center': { top: "xs", className: "overlay_top_center" },
-              'left-center': { left: "sm", className: "overlay_left_center" },
-              'right-center': { right: "sm", className: "overlay_right_center" }
           };
           break;
 
@@ -49,10 +37,6 @@ export const getPlacementProps = (placement: string, size?: "md" | "lg" | "sm" |
               'bottom-left': { bottom: { value: "xxs", inset: true }, left: { value: "xxs", inset: true } },
               'top-left': { top: { value: "xxs", inset: true }, left: { value: "xxs", inset: true } },
               'bottom-right': { bottom: { value: "xxs", inset: true }, right: { value: "xxs", inset: true } },
-              'bottom-center': { bottom: "xs", className: "overlay_bottom_center" },
-              'top-center': { top: "xs", className: "overlay_top_center" },
-              'left-center': { left: "sm", className: "overlay_left_center" },
-              'right-center': { right: "sm", className: "overlay_right_center" }
           };
           break;
   }

data/app/pb_kits/playbook/pb_avatar/_avatar.scss

@@ -26,22 +26,6 @@ $avatar-sizes: (
       flex-grow: 0;
       flex-basis: $size;
 
-      & > [class^=pb_flex_kit] {
-        [class^=pb_card_kit].overlay_bottom_center,
-        [class^=pb_card_kit].overlay_top_center {
-          left: 50%;
-          transform: translateX(-50%);
-          padding: 2px !important;
-        }
-
-        [class^=pb_card_kit].overlay_left_center,
-        [class^=pb_card_kit].overlay_right_center {
-          top: 50%;
-          transform: translateY(-50%);
-          padding: 2px !important;
-        }
-      }
-
       .avatar_wrapper {
         width: $size;
         height: $size;

data/app/pb_kits/playbook/pb_avatar/avatar.html.erb

@@ -7,7 +7,7 @@
       <% end %>
       <%= pb_rails("card", props: { border_none: true, border_radius: "rounded", html_options: { style: "padding: 2px" }, position: "absolute" }.merge(specific_placement_style)) do %>
         
-        <%= pb_rails("icon_circle", props: { size: "xxs", icon: object.component_overlay[:icon], variant: object.component_overlay[:variant] }) %>
+        <%= pb_rails("icon_circle", props: { size: "xxs", icon: object.component_overlay[:icon] }) %>
       <% end %>  
   <% end %>
   <% elsif object.component_overlay && object.component_overlay[:component] == "badge" %>

data/app/pb_kits/playbook/pb_avatar/avatar.rb

@@ -71,10 +71,6 @@ module Playbook
           "bottom-left" => { bottom: top_bottom_offset, left: offset },
           "top-left" => { top: top_bottom_offset, left: offset },
           "bottom-right" => { bottom: top_bottom_offset, right: offset },
-          "bottom-center" => { bottom: "xs", classname: "overlay_bottom_center" },
-          "top-center" => { top: "xs", classname: "overlay_top_center" },
-          "left-center" => { left: "sm", classname: "overlay_left_center" },
-          "right-center" => { right: "sm", classname: "overlay_right_center" },
         }
       end
     end

data/app/pb_kits/playbook/pb_avatar/avatar.test.js

@@ -42,50 +42,3 @@ test('uses the name prop as the alt property if no imageAlt prop is passed in',
 
   expect(image).toHaveAttribute('alt', name)
 })
-
-test('renders with badge overlay', () => {
-  const componentOverlay = {
-    component: "badge",
-    placement: "top-right",
-    text: "New",
-    variant: "error"
-  };
-  render(
-    <Avatar
-        componentOverlay={componentOverlay}
-        data={{ testid: testId }}
-        imageAlt={imageAlt}
-        imageUrl={imageUrl}
-        name={name}
-    />
-  );
-
-  const badgeOverlay = screen.getByTestId(testId);
- 
-  expect(badgeOverlay).toBeInTheDocument();
-  expect(badgeOverlay).toHaveClass('pb_avatar_kit_size_md');
-  expect(badgeOverlay).toHaveTextContent('New');
-});
-
-test('renders with iconCircle overlay', () => {
-  const componentOverlay = {
-    component: "iconCircle",
-    placement: "bottom-left",
-    icon: "plus",
-    variant: "blue"
-  };
-  render(
-    <Avatar
-        componentOverlay={componentOverlay}
-        data={{ testid: testId }}
-        imageAlt={imageAlt}
-        imageUrl={imageUrl}
-        name={name}
-    />
-  );
-
-  const iconCircleOverlay = screen.getByTestId(testId);
- 
-  expect(iconCircleOverlay).toBeInTheDocument();
-  expect(iconCircleOverlay).toHaveClass('pb_avatar_kit_size_md');
-});

data/app/pb_kits/playbook/pb_avatar/docs/_avatar_badge_component_overlay.html.erb

@@ -1,51 +1,71 @@
+<%= pb_rails("avatar", props: {
+    component_overlay: {
+        component: "badge",
+        text: "12",
+        placement: "top-left"
+    },
+    name: "Terry Johnson",
+    size: "xxs",
+    image_url: "https://randomuser.me/api/portraits/men/44.jpg",
+    margin_bottom: "sm"
+  }) %>
 
- <%= pb_rails("avatar", props: {
-      component_overlay: {
-          component: "badge",
-          text: "12",
-          placement: "bottom-right"
-      },
-      name: "Terry Johnson",
-      size: "sm",
-      image_url: "https://randomuser.me/api/portraits/men/44.jpg",
-      margin_bottom: "sm"
-    }) %>
+  <%= pb_rails("avatar", props: {
+    component_overlay: {
+        component: "badge",
+        text: "12",
+        placement: "top-right"
+    },
+    name: "Terry Johnson",
+    size: "xs",
+    image_url: "https://randomuser.me/api/portraits/men/44.jpg",
+    margin_bottom: "sm"
+  }) %>
 
-    <%= pb_rails("avatar", props: {
-      component_overlay: {
-          component: "badge",
-          text: "12",
-          placement: "top-left"
-      },
-      name: "Terry Johnson",
-      size: "md",
-      image_url: "https://randomuser.me/api/portraits/men/44.jpg",
-      margin_bottom: "sm"
-    }) %>
+  <%= pb_rails("avatar", props: {
+    component_overlay: {
+        component: "badge",
+        text: "12",
+        placement: "bottom-right"
+    },
+    name: "Terry Johnson",
+    size: "sm",
+    image_url: "https://randomuser.me/api/portraits/men/44.jpg",
+    margin_bottom: "sm"
+  }) %>
 
-    <%= pb_rails("avatar", props: {
-      component_overlay: {
-          component: "badge",
-          text: "On Roadtrip",
-          placement: "top-center",
-          variant: "info"
-      },
-      name: "Terry Johnson",
-      size: "lg",
-      image_url: "https://randomuser.me/api/portraits/men/44.jpg",
-      margin_bottom: "sm"
-    }) %>
+  <%= pb_rails("avatar", props: {
+    component_overlay: {
+        component: "badge",
+        text: "12",
+        placement: "bottom-left"
+    },
+    name: "Terry Johnson",
+    size: "md",
+    image_url: "https://randomuser.me/api/portraits/men/44.jpg",
+    margin_bottom: "sm"
+  }) %>
 
-    <%= pb_rails("avatar", props: {
-      component_overlay: {
-          component: "badge",
-          text: "Out of Office",
-          placement: "bottom-center",
-          variant: "error"
-      },
-      name: "Terry Johnson",
-      size: "xl",
-      image_url: "https://randomuser.me/api/portraits/men/44.jpg",
-      margin_bottom: "sm"
-    }) %>
+  <%= pb_rails("avatar", props: {
+    component_overlay: {
+        component: "badge",
+        text: "12",
+        placement: "top-left"
+    },
+    name: "Terry Johnson",
+    size: "lg",
+    image_url: "https://randomuser.me/api/portraits/men/44.jpg",
+    margin_bottom: "sm"
+  }) %>
 
+  <%= pb_rails("avatar", props: {
+    component_overlay: {
+        component: "badge",
+        text: "12",
+        placement: "top-right"
+    },
+    name: "Terry Johnson",
+    size: "xl",
+    image_url: "https://randomuser.me/api/portraits/men/44.jpg",
+    margin_bottom: "sm"
+  }) %>

data/app/pb_kits/playbook/pb_avatar/docs/_avatar_badge_component_overlay.jsx

@@ -4,6 +4,28 @@ import { Avatar } from '../..'
 const AvatarBadgeComponentOverlay = () => {
     return (
         <div>
+            <Avatar
+                componentOverlay={{
+                    component: "badge",
+                    placement: "top-left",
+                    text: "12"
+                }}
+                imageUrl="https://randomuser.me/api/portraits/men/44.jpg"
+                marginBottom="sm"
+                size="xxs"
+            />
+
+            <Avatar
+                componentOverlay={{
+                    component: "badge",
+                    placement: "top-right",
+                    text: "12"
+                }}
+                imageUrl="https://randomuser.me/api/portraits/men/44.jpg"
+                marginBottom="sm"
+                size="xs"
+            />
+
             <Avatar
                 componentOverlay={{
                     component: "badge",
@@ -18,7 +40,7 @@ const AvatarBadgeComponentOverlay = () => {
             <Avatar
                 componentOverlay={{
                     component: "badge",
-                    placement: "top-left",
+                    placement: "bottom-left",
                     text: "12"
                 }}
                 imageUrl="https://randomuser.me/api/portraits/men/44.jpg"
@@ -29,9 +51,8 @@ const AvatarBadgeComponentOverlay = () => {
             <Avatar
                 componentOverlay={{
                     component: "badge",
-                    placement: "top-center",
-                    text: "On Roadtip",
-                    variant: "info"
+                    placement: "top-left",
+                    text: "12"
                 }}
                 imageUrl="https://randomuser.me/api/portraits/men/44.jpg"
                 marginBottom="sm"
@@ -41,14 +62,14 @@ const AvatarBadgeComponentOverlay = () => {
             <Avatar
                 componentOverlay={{
                     component: "badge",
-                    placement: "bottom-center",
-                    text: "Out of Office",
-                    variant: "error"
+                    placement: "top-right",
+                    text: "12"
                 }}
                 imageUrl="https://randomuser.me/api/portraits/men/44.jpg"
                 marginBottom="sm"
                 size="xl"
-            /> 
+            />    
+
         </div>
     )
 }

data/app/pb_kits/playbook/pb_avatar/docs/_avatar_circle_icon_component_overlay.html.erb

@@ -1,9 +1,32 @@
+<%= pb_rails("avatar", props: {
+    component_overlay: {
+        component: "icon_circle",
+        icon: "plus",
+        placement: "top-left"
+    },
+    name: "Terry Johnson",
+    size: "xxs",
+    image_url: "https://randomuser.me/api/portraits/men/44.jpg",
+    margin_bottom: "sm"
+  }) %>
+
+  <%= pb_rails("avatar", props: {
+    component_overlay: {
+        component: "icon_circle",
+        icon: "plus",
+        placement: "top-right"
+    },
+    name: "Terry Johnson",
+    size: "xs",
+    image_url: "https://randomuser.me/api/portraits/men/44.jpg",
+    margin_bottom: "sm"
+  }) %>
+
   <%= pb_rails("avatar", props: {
     component_overlay: {
         component: "icon_circle",
-        icon: "shield",
-        placement: "bottom-right",
-        variant: "royal"
+        icon: "plus",
+        placement: "bottom-right"
     },
     name: "Terry Johnson",
     size: "sm",
@@ -14,9 +37,8 @@
   <%= pb_rails("avatar", props: {
     component_overlay: {
         component: "icon_circle",
-        icon: "check",
-        placement: "top-right",
-        variant: "green"
+        icon: "plus",
+        placement: "bottom-left"
     },
     name: "Terry Johnson",
     size: "md",
@@ -27,9 +49,8 @@
   <%= pb_rails("avatar", props: {
     component_overlay: {
         component: "icon_circle",
-        icon: "lock",
-        placement: "top-left",
-        variant: "red"
+        icon: "plus",
+        placement: "top-left"
     },
     name: "Terry Johnson",
     size: "lg",
@@ -40,12 +61,11 @@
   <%= pb_rails("avatar", props: {
     component_overlay: {
         component: "icon_circle",
-        icon: "star",
-        placement: "left-center",
-        variant: "yellow"
+        icon: "plus",
+        placement: "top-right"
     },
     name: "Terry Johnson",
     size: "xl",
     image_url: "https://randomuser.me/api/portraits/men/44.jpg",
     margin_bottom: "sm"
-  }) %>
+  }) %>

data/app/pb_kits/playbook/pb_avatar/docs/_avatar_circle_icon_component_overlay.jsx

@@ -4,54 +4,73 @@ import { Avatar } from '../..'
 const AvatarCircleIconComponentOverlay = () => {
     return (
         <div>
-        <Avatar
-            componentOverlay={{
-                component: "iconCircle",
-                placement: "bottom-right",
-                icon: "shield",
-                variant: "royal"
-            }}
-            imageUrl="https://randomuser.me/api/portraits/men/44.jpg"
-            marginBottom="sm"
-            size="sm"
-        />
-
-        <Avatar
-            componentOverlay={{
-                component: "iconCircle",
-                placement: "bottom-right",
-                icon: "check",
-                variant: "green"
-            }}
-            imageUrl="https://randomuser.me/api/portraits/men/44.jpg"
-            marginBottom="sm"
-            size="md"
-        />
-
-        <Avatar
-            componentOverlay={{
-                component: "iconCircle",
-                placement: "top-left",
-                icon: "lock",
-                variant: "red"
-            }}
-            imageUrl="https://randomuser.me/api/portraits/men/44.jpg"
-            marginBottom="sm"
-            size="lg"
-        />
-
-        <Avatar
-            componentOverlay={{
-                component: "iconCircle",
-                placement: "left-center",
-                icon: "star",
-                variant: "yellow"
-            }}
-            imageUrl="https://randomuser.me/api/portraits/men/44.jpg"
-            marginBottom="sm"
-            size="xl"
-        /> 
-    </div>
+            <Avatar
+                componentOverlay={{
+                    component: "iconCircle",
+                    icon: "plus",
+                    placement: "top-left"
+                }}
+                imageUrl="https://randomuser.me/api/portraits/men/44.jpg"
+                marginBottom="sm"
+                size="xxs"
+            />
+
+            <Avatar
+                componentOverlay={{
+                    component: "iconCircle",
+                    icon: "plus",
+                    placement: "top-right"
+                }}
+                imageUrl="https://randomuser.me/api/portraits/men/44.jpg"
+                marginBottom="sm"
+                size="xs"
+            />
+
+            <Avatar
+                componentOverlay={{
+                    component: "iconCircle",
+                    icon: "plus",
+                    placement: "bottom-right",
+                }}
+                imageUrl="https://randomuser.me/api/portraits/men/44.jpg"
+                marginBottom="sm"
+                size="sm"
+            />
+
+            <Avatar
+                componentOverlay={{
+                    component: "iconCircle",
+                    icon: "plus",
+                    placement: "bottom-left"
+                }}
+                imageUrl="https://randomuser.me/api/portraits/men/44.jpg"
+                marginBottom="sm"
+                size="md"
+            />
+
+            <Avatar
+                componentOverlay={{
+                    component: "iconCircle",
+                    icon: "plus",
+                    placement: "top-left"
+                }}
+                imageUrl="https://randomuser.me/api/portraits/men/44.jpg"
+                marginBottom="sm"
+                size="lg"
+            />
+
+            <Avatar
+                componentOverlay={{
+                    component: "iconCircle",
+                    icon: "plus",
+                    placement: "top-right"
+                }}
+                imageUrl="https://randomuser.me/api/portraits/men/44.jpg"
+                marginBottom="sm"
+                size="xl"
+            />
+
+        </div>
     )
 }
 

data/app/pb_kits/playbook/pb_avatar/docs/_avatar_default.jsx

@@ -57,6 +57,26 @@ const AvatarDefault = (props) => {
           status="offline"
           {...props}
       />
+      <br/>
+      <Avatar
+          imageAlt="Terry Johnson Standing"
+          imageUrl="https://randomuser.me/api/portraits/men/44.jpg"
+          name="Terry Johnson"
+          overlay={{component:"badge", text: "6", variant: "notification", placement:"top-right"}}
+          size="lg"
+          status="offline"
+          {...props}
+      />
+      <br/>
+      <Avatar
+          imageAlt="Terry Johnson Standing"
+          imageUrl="https://randomuser.me/api/portraits/men/44.jpg"
+          name="Terry Johnson"
+          overlay={{component:"icon", icon:"plus", variant:"royal", placement:"bottom-left"}}
+          size="xl"
+          status="offline"
+          {...props}
+      />
     </div>
   )
 }

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

@@ -4,14 +4,14 @@ examples:
     - avatar_monogram: Monogram
     - avatar_no_image: "Bad Image Link"
     - avatar_status: Status
-    - avatar_circle_icon_component_overlay: Icon Circle Component Overlay
+    - avatar_circle_icon_component_overlay: Circle Icon Component Overlay
     - avatar_badge_component_overlay: Badge Component Overlay
   react:
     - avatar_default: Default
     - avatar_monogram: Monogram
     - avatar_no_image: "Bad Image Link"
     - avatar_status: Status
-    - avatar_circle_icon_component_overlay: Icon Circle Component Overlay
+    - avatar_circle_icon_component_overlay: Circle Icon Component Overlay
     - avatar_badge_component_overlay: Badge Component Overlay
   swift:
     - avatar_default_swift: Default

data/app/pb_kits/playbook/pb_avatar_action_button/avatar_action_button.html.erb

@@ -1,4 +1,9 @@
-<%= pb_content_tag do %>
+<%= content_tag(:div,
+    aria: object.aria,
+    id: object.id,
+    data: object.data,
+    class: object.classname,
+    **combined_html_options) do %>
     <% if object.tooltip_text.present? %>
         <%= pb_rails("tooltip", props: {
           trigger_element_id: object.tooltip_id,

data/app/pb_kits/playbook/pb_badge/badge.html.erb

@@ -1,3 +1,8 @@
-<%= pb_content_tag do %>
+<%= content_tag(:div,
+    aria: object.aria,
+    id: object.id,
+    data: object.data,
+    class: object.classname,
+    **combined_html_options) do %>
   <span><%= object.text %></span>
 <% end %>