[arvados] updated: 2.7.0-5935-g1200472db4

git repository hosting git at public.arvados.org
Thu Feb 1 16:56:43 UTC 2024


Summary of changes:
 .../src/components/multiselect-toolbar/MultiselectToolbar.tsx          | 2 +-
 .../src/components/multiselect-toolbar/ms-toolbar-overflow-wrapper.tsx | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

       via  1200472db41ac1494849f894a8c345d41cd830fd (commit)
      from  84b64f2209d33aa8bc5f847812d888dd5d294747 (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 1200472db41ac1494849f894a8c345d41cd830fd
Author: Lisa Knox <lisaknox83 at gmail.com>
Date:   Thu Feb 1 11:56:35 2024 -0500

    21427: set last menu item to change only after transition Arvados-DCO-1.1-Signed-off-by: Lisa Knox <lisa.knox at curii.com>

diff --git a/services/workbench2/src/components/multiselect-toolbar/MultiselectToolbar.tsx b/services/workbench2/src/components/multiselect-toolbar/MultiselectToolbar.tsx
index 6500593166..2d255f163d 100644
--- a/services/workbench2/src/components/multiselect-toolbar/MultiselectToolbar.tsx
+++ b/services/workbench2/src/components/multiselect-toolbar/MultiselectToolbar.tsx
@@ -126,7 +126,7 @@ export const MultiselectToolbar = connect(
                     data-cy='multiselect-toolbar'
                     >
                     {actions.length ? (
-                        <IntersectionObserverWrapper menuLength={actions.length}>
+                        <IntersectionObserverWrapper menuLength={actions.length} isTransitioning>
                             {actions.map((action, i) =>{
                                 const { hasAlts, useAlts, name, altName, icon, altIcon } = action;
                             return hasAlts ? (
diff --git a/services/workbench2/src/components/multiselect-toolbar/ms-toolbar-overflow-wrapper.tsx b/services/workbench2/src/components/multiselect-toolbar/ms-toolbar-overflow-wrapper.tsx
index 1f683b6013..47b019e0ff 100644
--- a/services/workbench2/src/components/multiselect-toolbar/ms-toolbar-overflow-wrapper.tsx
+++ b/services/workbench2/src/components/multiselect-toolbar/ms-toolbar-overflow-wrapper.tsx
@@ -38,6 +38,7 @@ const styles: StyleRulesCallback<CssRules> = (theme: ArvadosTheme) => ({
 type WrapperProps = {
     children: OverflowChild[];
     menuLength: number;
+    isTransitioning: boolean;
 };
 
 export const IntersectionObserverWrapper = withStyles(styles)((props: WrapperProps & WithStyles<CssRules>) => {
@@ -63,7 +64,7 @@ export const IntersectionObserverWrapper = withStyles(styles)((props: WrapperPro
         setVisibilityMap((prev) => ({
             ...prev,
             ...updatedEntries,
-            [lastEntryId]: Object.keys(updatedEntries)[0] === lastEntryId,
+            [lastEntryId]: !props.isTransitioning && Object.keys(updatedEntries)[0] === lastEntryId,
         }));
     };
 

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list