From 047908c67a96624b1eb918cfd6203e237dc2ffcb Mon Sep 17 00:00:00 2001
From: CD Cabrera <ccabrera@redhat.com>
Date: Fri, 26 Jun 2020 14:53:27 -0400
Subject: [PATCH] fix(messageView): reposition empty state icon

---
 .../__snapshots__/authentication.test.js.snap | 20 +++++++++----------
 .../__snapshots__/messageView.test.js.snap    |  6 +++---
 src/components/messageView/messageView.js     |  2 +-
 3 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/src/components/authentication/__tests__/__snapshots__/authentication.test.js.snap b/src/components/authentication/__tests__/__snapshots__/authentication.test.js.snap
index 1dc0a1f0a..0e5c2ccf3 100644
--- a/src/components/authentication/__tests__/__snapshots__/authentication.test.js.snap
+++ b/src/components/authentication/__tests__/__snapshots__/authentication.test.js.snap
@@ -112,16 +112,6 @@ exports[`Authentication Component should render a non-connected component error:
               <div
                 className="pf-c-empty-state__content"
               >
-                <Title
-                  headingLevel="h2"
-                  size="lg"
-                >
-                  <h2
-                    className="pf-c-title pf-m-lg"
-                  >
-                    t(curiosity-auth.authorizedTitle, [object Object])
-                  </h2>
-                </Title>
                 <EmptyStateIcon
                   icon={[Function]}
                 >
@@ -154,6 +144,16 @@ exports[`Authentication Component should render a non-connected component error:
                     </svg>
                   </LockIcon>
                 </EmptyStateIcon>
+                <Title
+                  headingLevel="h2"
+                  size="lg"
+                >
+                  <h2
+                    className="pf-c-title pf-m-lg"
+                  >
+                    t(curiosity-auth.authorizedTitle, [object Object])
+                  </h2>
+                </Title>
                 <EmptyStateBody>
                   <div
                     className="pf-c-empty-state__body"
diff --git a/src/components/messageView/__tests__/__snapshots__/messageView.test.js.snap b/src/components/messageView/__tests__/__snapshots__/messageView.test.js.snap
index c4047c0c4..0b8ae1467 100644
--- a/src/components/messageView/__tests__/__snapshots__/messageView.test.js.snap
+++ b/src/components/messageView/__tests__/__snapshots__/messageView.test.js.snap
@@ -21,15 +21,15 @@ exports[`MessageView Component should render a non-connected component: non-conn
     className="fadein"
     variant="full"
   >
+    <EmptyStateIcon
+      icon={[Function]}
+    />
     <Title
       headingLevel="h2"
       size="lg"
     >
       lorem
     </Title>
-    <EmptyStateIcon
-      icon={[Function]}
-    />
     <EmptyStateBody>
       ipsum
     </EmptyStateBody>
diff --git a/src/components/messageView/messageView.js b/src/components/messageView/messageView.js
index bd9613f77..5e1c9aa10 100644
--- a/src/components/messageView/messageView.js
+++ b/src/components/messageView/messageView.js
@@ -27,12 +27,12 @@ const MessageView = ({ icon, message, pageTitle, title }) => (
   <PageLayout>
     <PageHeader>{pageTitle || helpers.UI_DISPLAY_NAME}</PageHeader>
     <PfEmptyState variant={EmptyStateVariant.full} className="fadein">
+      {icon && <EmptyStateIcon icon={icon} />}
       {title && (
         <Title headingLevel="h2" size="lg">
           {title}
         </Title>
       )}
-      {icon && <EmptyStateIcon icon={icon} />}
       {message && <EmptyStateBody>{message}</EmptyStateBody>}
     </PfEmptyState>
   </PageLayout>