[arvados] updated: 2.7.0-5933-g17647c7f12

git repository hosting git at public.arvados.org
Wed Jan 31 20:01:06 UTC 2024


Summary of changes:
 services/workbench2/cypress/integration/collection.spec.js         | 1 +
 services/workbench2/cypress/integration/process.spec.js            | 1 +
 services/workbench2/cypress/integration/project.spec.js            | 1 +
 .../src/components/multiselect-toolbar/MultiselectToolbar.tsx      | 7 ++++---
 4 files changed, 7 insertions(+), 3 deletions(-)

       via  17647c7f12eae3557dfefbe8432c785360630d3d (commit)
      from  9905892f3c53dd6f88a14a1c00d4963bf733ad09 (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 17647c7f12eae3557dfefbe8432c785360630d3d
Author: Lisa Knox <lisaknox83 at gmail.com>
Date:   Wed Jan 31 15:00:58 2024 -0500

    21427: changed spec to allow for delay in toolbar transition Arvados-DCO-1.1-Signed-off-by: Lisa Knox <lisa.knox at curii.com>

diff --git a/services/workbench2/cypress/integration/collection.spec.js b/services/workbench2/cypress/integration/collection.spec.js
index 54c570f7c4..32349ab884 100644
--- a/services/workbench2/cypress/integration/collection.spec.js
+++ b/services/workbench2/cypress/integration/collection.spec.js
@@ -63,6 +63,7 @@ describe("Collection panel tests", function () {
             cy.get("[data-cy=side-panel-tree]").contains("Home Projects").click();
             cy.waitForDom()
             cy.get('[data-cy=data-table-row]').contains(name).should('exist').parent().parent().parent().parent().click()
+            cy.waitForDom()
             cy.get('[data-cy=multiselect-button]').should('have.length', msButtonTooltips.length)
             for (let i = 0; i < msButtonTooltips.length; i++) {
                 cy.get('[data-cy=multiselect-button]').eq(i).trigger('mouseover');
diff --git a/services/workbench2/cypress/integration/process.spec.js b/services/workbench2/cypress/integration/process.spec.js
index f7925457d2..133cabf1f9 100644
--- a/services/workbench2/cypress/integration/process.spec.js
+++ b/services/workbench2/cypress/integration/process.spec.js
@@ -123,6 +123,7 @@ describe("Process tests", function () {
                 cy.get("[data-cy=side-panel-tree]").contains("Home Projects").click();
                 cy.waitForDom()
                 cy.get('[data-cy=data-table-row]').contains(containerRequest.name).should('exist').parent().parent().parent().parent().click()
+                cy.waitForDom()
                 cy.get('[data-cy=multiselect-button]').should('have.length', msButtonTooltips.length)
                 for (let i = 0; i < msButtonTooltips.length; i++) {
                     cy.get('[data-cy=multiselect-button]').eq(i).trigger('mouseover');
diff --git a/services/workbench2/cypress/integration/project.spec.js b/services/workbench2/cypress/integration/project.spec.js
index e6185c108e..47eed5ffe8 100644
--- a/services/workbench2/cypress/integration/project.spec.js
+++ b/services/workbench2/cypress/integration/project.spec.js
@@ -213,6 +213,7 @@ describe("Project tests", function () {
         cy.go('back')
 
         cy.get('[data-cy=data-table-row]').contains(projName).should('exist').parent().parent().parent().click()
+        cy.waitForDom()
         cy.get('[data-cy=multiselect-button]').should('have.length', msButtonTooltips.length)
         for (let i = 0; i < msButtonTooltips.length; i++) {
             cy.get('[data-cy=multiselect-button]').eq(i).trigger('mouseover');
diff --git a/services/workbench2/src/components/multiselect-toolbar/MultiselectToolbar.tsx b/services/workbench2/src/components/multiselect-toolbar/MultiselectToolbar.tsx
index 0746ceafa4..6500593166 100644
--- a/services/workbench2/src/components/multiselect-toolbar/MultiselectToolbar.tsx
+++ b/services/workbench2/src/components/multiselect-toolbar/MultiselectToolbar.tsx
@@ -47,7 +47,7 @@ const styles: StyleRulesCallback<CssRules> = (theme: ArvadosTheme) => ({
         width: 0,
         height: '2.7rem',
         padding: 0,
-        margin: "1rem auto auto 0.5rem",
+        margin: "1rem auto auto 0.3rem",
         transition: `width ${WIDTH_TRANSITION}ms`,
         overflow: 'hidden',
     },
@@ -56,7 +56,7 @@ const styles: StyleRulesCallback<CssRules> = (theme: ArvadosTheme) => ({
         flexDirection: "row",
         height: '2.7rem',
         padding: 0,
-        margin: "1rem auto auto 0.5rem",
+        margin: "1rem auto auto 0.3rem",
         overflow: 'hidden',
         transition: `width ${WIDTH_TRANSITION}ms`,
     },
@@ -64,6 +64,7 @@ const styles: StyleRulesCallback<CssRules> = (theme: ArvadosTheme) => ({
         width: "2.5rem",
         height: "2.5rem ",
         paddingLeft: 0,
+        border: "1px solid transparent",
     },
     iconContainer: {
         height: '100%',
@@ -121,7 +122,7 @@ export const MultiselectToolbar = connect(
             <React.Fragment>
                 <Toolbar
                     className={isTransitioning ? classes.transition: classes.root}
-                    style={{ width: `${(actions.length * 2.5) + 4.5}rem`}}
+                    style={{ width: `${(actions.length * 2.5) + 6}rem`}}
                     data-cy='multiselect-toolbar'
                     >
                     {actions.length ? (

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list