Merge pull request #340 from pxblue/dev

Publish 5.3.3
etn-ccis · Dec 6, 2021 · cb90d02 · cb90d02
2 parents 4a6fec9 + a4d2daf
commit cb90d02
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 14 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Changelog
+## v5.3.3 (December 6, 2021)
 
+### Fixed
+
+-   Avoid rendering of HTML elements in DOM when `subtitle` or `info` prop is not available.
 ## v5.3.2 (November 11, 2021)
 
 ### Changed

diff --git a/components/package.json b/components/package.json
@@ -1,6 +1,6 @@
 {
     "name": "@brightlayer-ui/react-components",
-    "version": "5.3.2",
+    "version": "5.3.3",
     "description": "React components for Brightlayer UI applications",
     "scripts": {
         "test": "jest src",

diff --git a/components/src/core/InfoListItem/InfoListItem.tsx b/components/src/core/InfoListItem/InfoListItem.tsx
@@ -207,19 +207,25 @@ const InfoListItemRender: React.ForwardRefRenderFunction<unknown, InfoListItemPr
                 primary={title}
                 className={combine('listItemText')}
                 secondary={
-                    <>
-                        <Typography
-                            variant={'subtitle2'}
-                            component={'p'}
-                            noWrap={!wrapSubtitle}
-                            className={combine('subtitle')}
-                        >
-                            {getSubtitle()}
-                        </Typography>
-                        <Typography variant={'body2'} noWrap={!wrapInfo} className={combine('info')}>
-                            {getInfo()}
-                        </Typography>
-                    </>
+                    subtitle || info ? (
+                        <>
+                            {subtitle ? (
+                                <Typography
+                                    variant={'subtitle2'}
+                                    component={'p'}
+                                    noWrap={!wrapSubtitle}
+                                    className={combine('subtitle')}
+                                >
+                                    {getSubtitle()}
+                                </Typography>
+                            ) : undefined}
+                            {info ? (
+                                <Typography variant={'body2'} noWrap={!wrapInfo} className={combine('info')}>
+                                    {getInfo()}
+                                </Typography>
+                            ) : undefined}
+                        </>
+                    ) : undefined
                 }
                 primaryTypographyProps={{
                     noWrap: !wrapTitle,