playbook_ui 14.3.0.pre.rc.8 → 14.3.0.pre.rc.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 87f159ff1ca1720cfe5d67393dee393d6ba2c8ba9cc5265c0c90803bf06298ab
-  data.tar.gz: 19e3c47ccac8485a2a65965817a9ab4c0035bf22c0eaaffc0f5eaaa7d0a35905
+  metadata.gz: 802ad05cc819086feec002c5a2528144dd9996654e668eccec3bc02beb2d6521
+  data.tar.gz: c0b4e44c40893d27244309b0e24ee3c8708746a91d15dffe565fe41e07d14b03
 SHA512:
-  metadata.gz: 6d517ea8d227655b4ce33b0ef06f9f85e53820cc58f4048e24ba9fc8ab8a38e82ee3b09553e9ec3a44782f97da429f441891cebe400313aeb975eb26cdf56926
-  data.tar.gz: d99e3d9ae1b4317a1bfb97e91488d4ec3fb4c92c90376e6093e67097663974b6f81c446ec4e0c34d42a93db70ee2f6a4a07feb0b4716d96df56325bc00cee910
+  metadata.gz: 8e7a1200079126289d171fa6e6a26bda62b629aeef8ebcc880a28ca3a494d23810dc27a70a12bdb9a4081a98a8a37c7ae648b7f0c6a3d2552d7ee343c9b9fb5a
+  data.tar.gz: 3b89ba5afc4d031b92e19cae2ef8e3517e081d44784b135e49c0fff824f0ba576e505a57f230302ddee822c7428a1bbf4188853cf32d19b4273b5c6e52dcbd0c

data/app/pb_kits/playbook/pb_hashtag/_hashtag.tsx

@@ -1,5 +1,5 @@
 /* eslint-disable react/jsx-no-target-blank */
-/* eslint-disable react/no-multi-comp, flowtype/space-before-type-colon */
+/* eslint-disable react/no-multi-comp */
 
 import React from 'react'
 import classnames from 'classnames'
@@ -18,6 +18,7 @@ type HashtagProps = {
   id?: string,
   newWindow?: boolean,
   rel?: string,
+  target?: string,
   text?: string,
   type: "default" | "home" | "project" | "appointment",
   url?: string,
@@ -40,6 +41,7 @@ const Hashtag = (props: HashtagProps) => {
     id,
     newWindow,
     rel,
+    target,
     text,
     type = 'default',
     url,
@@ -50,6 +52,18 @@ const Hashtag = (props: HashtagProps) => {
   const htmlProps = buildHtmlProps(htmlOptions)
   const classes = classnames(buildCss('pb_hashtag_kit'), globalProps(props), className)
 
+  const getTargetAttribute = () => {
+    if (target && url) {
+      return target
+    } else if (newWindow) {
+      return '_blank'
+    } else {
+      return '_self'
+    }
+
+    return null
+  }
+
   return (
     <span
         {...ariaProps}
@@ -61,7 +75,7 @@ const Hashtag = (props: HashtagProps) => {
       <a
           href={url}
           rel={(newWindow ? "noreferrer" : rel)}
-          target={(newWindow ? '_blank' : '_self')}
+          target={getTargetAttribute()}
       >
         <Badge
             dark={dark}

data/app/pb_kits/playbook/pb_hashtag/docs/_hashtag_link.html.erb

@@ -2,4 +2,4 @@
 
 <br/><br/>
 
-<%= pb_rails("hashtag", props: {text: "Open in a new Window", url: "https://google.com", type: "project", new_window: true}) %>
+<%= pb_rails("hashtag", props: {text: "Open in a new Window", url: "https://google.com", type: "project", target: "_blank"}) %>

data/app/pb_kits/playbook/pb_hashtag/docs/_hashtag_link.jsx

@@ -13,7 +13,7 @@ const HashtagLink = (props) => {
       <br />
       <br />
       <Hashtag
-          newWindow
+          target="_blank"
           text="Open in a new window"
           type="project"
           url="https://google.com"

data/app/pb_kits/playbook/pb_hashtag/docs/_hashtag_link.md

@@ -1 +1 @@
-Use the newWindow/new_window prop to control whether the link opens on the same page or a new tab (same page is the default behavior)
+Use the `target` prop to control whether the link opens on the same or a new tab (same page is the default behavior). You can use any [web/standard](https://www.w3schools.com/tags/att_a_target.asp) values or a custom string to specify your link target.

data/app/pb_kits/playbook/pb_hashtag/hashtag.html.erb

@@ -1,5 +1,5 @@
 <%= pb_content_tag(:span) do %>
-  <%= link_to object.url, target: object.link_option do %>
+  <%= link_to object.url, target: object.target_option do %>
     <%= pb_rails("badge", props: { dark: object.dark, variant: "primary", text: object.hashtag_text }) %>
   <% end %>
 <% end %>

data/app/pb_kits/playbook/pb_hashtag/hashtag.rb

@@ -10,6 +10,7 @@ module Playbook
                   values: %w[default project home appointment],
                   default: "default"
       prop :url
+      prop :target
 
       def classname
         generate_classname("pb_hastag_kit")
@@ -19,8 +20,14 @@ module Playbook
         type_text + text
       end
 
-      def link_option
-        new_window ? "_blank" : "_self"
+      def target_option
+        if target && url
+          target
+        elsif new_window
+          "_blank"
+        else
+          "_self"
+        end
       end
 
     private

data/app/pb_kits/playbook/pb_home_address_street/_home_address_street.tsx

@@ -23,6 +23,7 @@ type HomeAddressStreetProps = {
   homeId: string,
   houseStyle: string,
   homeUrl: string,
+  target: string,
   newWindow: boolean,
   state: string,
   zipcode: string,
@@ -42,6 +43,7 @@ const HomeAddressStreet = (props: HomeAddressStreetProps): React.ReactElement =>
     htmlOptions = {},
     homeId,
     homeUrl,
+    target,
     newWindow,
     houseStyle,
     state,
@@ -62,6 +64,19 @@ const HomeAddressStreet = (props: HomeAddressStreetProps): React.ReactElement =>
   const dataProps: GenericObject = buildDataProps(data)
   const ariaProps: GenericObject = buildAriaProps(aria)
   const htmlProps = buildHtmlProps(htmlOptions)
+
+  const getTargetAttribute = () => {
+    if (target && homeUrl) {
+      return target
+    } else if (newWindow) {
+      return '_blank'
+    } else {
+      return '_self'
+    }
+
+    return null
+  }
+
   return (
     <div
         className={classes(className, dark)}
@@ -120,6 +135,7 @@ const HomeAddressStreet = (props: HomeAddressStreetProps): React.ReactElement =>
             dark={dark}
             marginRight="xxs"
             newWindow={newWindow}
+            target={getTargetAttribute()}
             text={homeId}
             type="home"
             url={homeUrl || '#'}

data/app/pb_kits/playbook/pb_home_address_street/city_emphasis.html.erb

@@ -33,6 +33,7 @@
     type: "home",
     dark: object.dark,
     classname: "home-hashtag",
+    target: object.target_option,
     new_window: object.new_window}) %>
 <% end %>
 

data/app/pb_kits/playbook/pb_home_address_street/city_emphasis.rb

@@ -10,9 +10,20 @@ module Playbook
       prop :home_url
       prop :new_window, type: Playbook::Props::Boolean,
                         default: false
+      prop :target
       prop :territory
       prop :zip
       prop :dark, type: Playbook::Props::Boolean, default: false
+
+      def target_option
+        if target && home_url
+          target
+        elsif new_window
+          "_blank"
+        else
+          "_self"
+        end
+      end
     end
   end
 end

data/app/pb_kits/playbook/pb_home_address_street/docs/_home_address_street_link.html.erb

@@ -5,7 +5,7 @@
   home_id: 8250263,
   home_url: "https://powerhrg.com/",
   house_style: "Colonial",
-  new_window: true,
+  target: "_blank",
   state: "PA",
   zipcode: "19382",
   territory: "PHL",

data/app/pb_kits/playbook/pb_home_address_street/docs/_home_address_street_link.jsx

@@ -11,8 +11,8 @@ const HomeAddressStreetLink= (props) => {
         homeId={8250263}
         homeUrl="https://powerhrg.com/"
         houseStyle="Colonial"
-        newWindow
         state="PA"
+        target="_blank"
         territory="PHL"
         zipcode="19382"
         {...props}

data/app/pb_kits/playbook/pb_home_address_street/docs/_home_address_street_link.md

@@ -1 +1 @@
-Use the `newWindow` / `new_window` prop to control whether the link opens on the same page or a new tab (same page is the default behavior)
+Use the `target` prop to control whether the link opens on the same or a new tab (same page is the default behavior). You can use any [web/standard](https://www.w3schools.com/tags/att_a_target.asp) values or a custom string to specify your link target.

data/app/pb_kits/playbook/pb_home_address_street/home_address_street.rb

@@ -14,6 +14,7 @@ module Playbook
       prop :house_style
       prop :new_window, type: Playbook::Props::Boolean,
                         default: false
+      prop :target
       prop :state
       prop :zipcode
       prop :territory
@@ -55,6 +56,7 @@ module Playbook
           dark: dark,
           home_id: home_id,
           home_url: home_url,
+          target: target_option,
           new_window: new_window,
           territory: territory,
           zip: zip,
@@ -69,11 +71,22 @@ module Playbook
           dark: dark,
           home_id: home_id,
           home_url: home_url,
+          target: target_option,
           new_window: new_window,
           territory: territory,
         }
       end
 
+      def target_option
+        if target && home_url
+          target
+        elsif new_window
+          "_blank"
+        else
+          "_self"
+        end
+      end
+
     private
 
       def dark_class

data/app/pb_kits/playbook/pb_home_address_street/home_adress_street.test.js

@@ -23,10 +23,10 @@ function HomeAdressStreetTest(props) {
             territory="PHL"
             zipcode="19382"
             {...props}
-    />
+        />
     );
   }
-  
+
   test("renders the component", () => {
     render(<HomeAdressStreetTest />);
     const kit = screen.getByTestId("primary-test");

data/app/pb_kits/playbook/pb_home_address_street/street_emphasis.html.erb

@@ -23,7 +23,8 @@
     type: "home",
     dark: object.dark,
     classname: "home-hashtag",
-    new_window: object.new_window}) %>
+    new_window: object.new_window,
+    target: object.target_option}) %>
 <% end %>
 
 <%= pb_rails "body", props: { color: "light", tag: "span", dark: object.dark } do %>

data/app/pb_kits/playbook/pb_home_address_street/street_emphasis.rb

@@ -10,9 +10,20 @@ module Playbook
       prop :home_url
       prop :new_window, type: Playbook::Props::Boolean,
                         default: false
+      prop :target
       prop :territory
       prop :dark, type: Playbook::Props::Boolean,
                   default: false
+
+      def target_option
+        if target && home_url
+          target
+        elsif new_window
+          "_blank"
+        else
+          "_self"
+        end
+      end
     end
   end
 end

data/app/pb_kits/playbook/pb_icon/icon.rb

@@ -102,6 +102,11 @@ module Playbook
       def render_svg
         doc = Nokogiri::XML(URI.open(asset_path || icon || custom_icon)) # rubocop:disable Security/Open
         svg = doc.at_css "svg"
+
+        unless svg
+          return "" # Return an empty string if SVG element is not found
+        end
+
         svg["class"] = %w[pb_custom_icon svg-inline--fa].concat([object.custom_icon_classname]).join(" ")
         svg["id"] = object.id
         svg["data"] = object.data
@@ -112,6 +117,9 @@ module Playbook
         fill_color = object.color || "currentColor"
         doc.at_css("path")["fill"] = fill_color
         raw doc
+      rescue OpenURI::HTTPError, StandardError
+        # Handle any exceptions and return an empty string
+        ""
       end
 
       def is_svg?