feat(images): removed change source icon tooltip

canonical · Jan 28, 2025 · faa7423 · faa7423
1 parent 97dd52a
commit faa7423
Showing 1 changed file with 32 additions and 20 deletions.
diff --git 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>