Merge pull request #138 from froboy/main

fix: Count length of alerts with markup

js/openy-alerts/src/components/FooterAlertItem/FooterAlertItem.js

@@ -25,7 +25,18 @@ class AlertItem extends Component {
     description = description.concat(parse(this.props.description));
     description.forEach( el => {
       if(el.props) {
-        length += el.props.children.length;
+        // If the description has markup, we need to count each element.
+        if(el.props.children instanceof Object) {
+          el.props.children.forEach( el => {
+            // If the element has markup it's an object.
+            if(el instanceof Object){
+              length += el.props.children.length;
+            }
+            // If not an object we just get the length.
+            else length += el.length;
+          })
+        }
+        else length += el.props.children.length;
       }
     });
 

js/openy-alerts/src/components/HeaderAlertItem/HeaderAlertItem.js

@@ -25,7 +25,18 @@ class AlertItem extends Component {
     description = description.concat(parse(this.props.description));
     description.forEach( el => {
       if(el.props) {
-        length += el.props.children.length;
+        // If the description has markup, we need to count each element.
+        if(el.props.children instanceof Object) {
+          el.props.children.forEach( el => {
+            // If the element has markup it's an object.
+            if(el instanceof Object){
+              length += el.props.children.length;
+            }
+            // If not an object we just get the length.
+            else length += el.length;
+          })
+        }
+        else length += el.props.children.length;
       }
     });
     this.setState({

openy_node_alert.libraries.yml

@@ -1,5 +1,5 @@
 alert:
-  version: 1.4
+  version: 1.5
   js:
     js/openy-alerts/build/main.js: {preprocess: false}
     js/openy_node_alert.js: {preprocess: false}