[arvados] updated: 2.7.0-5937-gc2a865cb5c

git repository hosting git at public.arvados.org
Fri Feb 2 15:15:55 UTC 2024


Summary of changes:
 .../components/multiselect-toolbar/ms-toolbar-overflow-wrapper.tsx    | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

       via  c2a865cb5c270bc1d4ce021d9d0cf3dd6ea1871c (commit)
      from  7d355f5ebdeffa4dbdc9cb3544c99390060ee927 (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 c2a865cb5c270bc1d4ce021d9d0cf3dd6ea1871c
Author: Lisa Knox <lisaknox83 at gmail.com>
Date:   Fri Feb 2 10:15:50 2024 -0500

    21427: modified visibility map check Arvados-DCO-1.1-Signed-off-by: Lisa Knox <lisa.knox at curii.com>

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..32f977e1a4 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
@@ -53,6 +53,7 @@ export const IntersectionObserverWrapper = withStyles(styles)((props: WrapperPro
         const updatedEntries: Record<string, boolean> = {};
         entries.forEach((entry) => {
             const targetid = entry.target.dataset.targetid as string;
+            //if true, the element is visible
             if (entry.isIntersecting) {
                 updatedEntries[targetid] = true;
             } else {
@@ -67,8 +68,9 @@ export const IntersectionObserverWrapper = withStyles(styles)((props: WrapperPro
         }));
     };
 
+    //ensures that the last element is always visible if the second to last is visible
     useEffect(() => {
-        if ((prevNumHidden.current === 2 || prevNumHidden.current === 0) && numHidden === 1) {
+        if ((prevNumHidden.current > 1 || prevNumHidden.current === 0) && numHidden === 1) {
             setVisibilityMap((prev) => ({
                 ...prev,
                 [lastEntryId]: true,

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list