[arvados-workbench2] updated: 2.7.0-2-g0ccf8be1

git repository hosting git at public.arvados.org
Thu Sep 28 16:03:11 UTC 2023


Summary of changes:
 .../main-app-bar/notifications-menu.tsx                | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

       via  0ccf8be1713e2a21ad4214d6f64913d1a9d8c8e7 (commit)
      from  f79ee597833b880551baff9aef059919d80d5cd7 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 0ccf8be1713e2a21ad4214d6f64913d1a9d8c8e7
Author: Lisa Knox <lisaknox83 at gmail.com>
Date:   Thu Sep 28 12:03:05 2023 -0400

    21002: fixed notif menu clickboxes Arvados-DCO-1.1-Signed-off-by: Lisa Knox <lisa.knox at curii.com>

diff --git a/src/views-components/main-app-bar/notifications-menu.tsx b/src/views-components/main-app-bar/notifications-menu.tsx
index f415c7d5..89fd2e91 100644
--- a/src/views-components/main-app-bar/notifications-menu.tsx
+++ b/src/views-components/main-app-bar/notifications-menu.tsx
@@ -40,8 +40,8 @@ export const NotificationsMenuComponent = (props: NotificationsMenuComponentProp
 
     if (!isOpen && bannerResult) {
         menuItems.push(
-            <MenuItem>
-                <span onClick={openBanner}>Restore Banner</span>
+            <MenuItem onClick={openBanner}>
+                <span>Restore Banner</span>
             </MenuItem>
         );
     }
@@ -57,14 +57,14 @@ export const NotificationsMenuComponent = (props: NotificationsMenuComponentProp
 
     if (tooltipResult) {
         menuItems.push(
-            <MenuItem>
-                <span onClick={toggleTooltips}>Enable tooltips</span>
+            <MenuItem onClick={toggleTooltips}>
+                <span>Enable tooltips</span>
             </MenuItem>
         );
     } else {
         menuItems.push(
-            <MenuItem>
-                <span onClick={toggleTooltips}>Disable tooltips</span>
+            <MenuItem onClick={toggleTooltips}>
+                <span>Disable tooltips</span>
             </MenuItem>
         );
     }
@@ -78,12 +78,14 @@ export const NotificationsMenuComponent = (props: NotificationsMenuComponentProp
             icon={
                 <Badge
                     badgeContent={0}
-                    color="primary">
+                    color="primary"
+                >
                     <NotificationIcon />
                 </Badge>
             }
             id="account-menu"
-            title="Notifications">
+            title="Notifications"
+        >
             {menuItems.map((item, i) => (
                 <div key={i}>{item}</div>
             ))}

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list