From faa74237d3dc147bc8db964643f88b31756fa2e5 Mon Sep 17 00:00:00 2001
From: Ahmet Can Buyukyilmaz <ahmet.buyukyilmaz@canonical.com>
Date: Tue, 28 Jan 2025 19:31:17 +0300
Subject: [PATCH] feat(images): removed change source icon tooltip

---
 .../ImageListHeader/ImageListHeader.tsx       | 52 ++++++++++++-------
 1 file changed, 32 insertions(+), 20 deletions(-)

diff --git a/src/app/images/views/ImageList/ImageListHeader/ImageListHeader.tsx b/src/app/images/views/ImageList/ImageListHeader/ImageListHeader.tsx
index 8bdef19435..cf9c91c95d 100644
--- a/src/app/images/views/ImageList/ImageListHeader/ImageListHeader.tsx
+++ b/src/app/images/views/ImageList/ImageListHeader/ImageListHeader.tsx
@@ -181,27 +181,39 @@ const ImageListHeader = ({
             <i className="p-icon--begin-downloading" />
             <span>Select upstream images</span>
           </Button>
-          <Button
-            data-testid="change-source-button"
-            disabled={!canChangeSource}
-            onClick={() =>
-              setSidePanelContent({
-                view: ImageSidePanelViews.CHANGE_SOURCE,
-                extras: { hasSources },
-              })
-            }
-          >
-            Change source
-            {!canChangeSource && (
-              <Tooltip
-                className="u-nudge-right--small"
-                message="Cannot change source while images are downloading."
-                position="top-right"
+
+          {!canChangeSource ? (
+            <Tooltip
+              message="Cannot change source while images are downloading."
+              position="top-right"
+            >
+              <Button
+                data-testid="change-source-button"
+                disabled={!canChangeSource}
+                onClick={() =>
+                  setSidePanelContent({
+                    view: ImageSidePanelViews.CHANGE_SOURCE,
+                    extras: { hasSources },
+                  })
+                }
               >
-                <Icon name="information" />
-              </Tooltip>
-            )}
-          </Button>
+                Change source
+              </Button>
+            </Tooltip>
+          ) : (
+            <Button
+              data-testid="change-source-button"
+              disabled={!canChangeSource}
+              onClick={() =>
+                setSidePanelContent({
+                  view: ImageSidePanelViews.CHANGE_SOURCE,
+                  extras: { hasSources },
+                })
+              }
+            >
+              Change source
+            </Button>
+          )}
         </MainToolbar.Controls>
       )}
     </MainToolbar>