From 0e1392c22fcc6405e4a07b9ad9fd5a193566945d Mon Sep 17 00:00:00 2001
From: Marius Vollmer <marius.vollmer@gmail.com>
Date: Fri, 13 Dec 2024 13:34:34 +0200
Subject: [PATCH] storage: Don't navigate when clicking on the mobile kebab

Let's use the same trick as with the table: Don't put the navigation
handler on the whole card, but on all its children that should
navigate, and exclude the menu button.

Fixes #19897
---
 pkg/storaged/pages.jsx | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/pkg/storaged/pages.jsx b/pkg/storaged/pages.jsx
index 07c04622ee1b..6f2306c6a893 100644
--- a/pkg/storaged/pages.jsx
+++ b/pkg/storaged/pages.jsx
@@ -587,7 +587,7 @@ export const PageTable = ({ emptyCaption, aria_label, pages, crossrefs, sorted,
 
         if (narrow) {
             rows.push(
-                <Card key={key} onClick={onClick}
+                <Card key={key}
                       className={"ct-small-table-card" +
                                  (page.location ? " ct-clickable-card" : null) +
                                  (is_new ? " ct-new-item" : "")}
@@ -595,11 +595,13 @@ export const PageTable = ({ emptyCaption, aria_label, pages, crossrefs, sorted,
                       data-test-row-location={page.columns[1]}>
                     <CardBody>
                         <Split hasGutter>
-                            { icon && <SplitItem>{icon}</SplitItem> }
-                            <SplitItem isFilled><strong><Truncate content={name} /></strong>{info}</SplitItem>
+                            { icon && <SplitItem onClick={onClick}>{icon}</SplitItem> }
+                            <SplitItem isFilled onClick={onClick}>
+                                <strong><Truncate content={name} /></strong>{info}
+                            </SplitItem>
                             <SplitItem>{actions}</SplitItem>
                         </Split>
-                        <Split hasGutter isWrappable>
+                        <Split hasGutter isWrappable onClick={onClick}>
                             <SplitItem>{type}</SplitItem>
                             <SplitItem isFilled>{location}</SplitItem>
                             <SplitItem isFilled className="pf-v5-u-text-align-right">{size}</SplitItem>