change back button to link (#6374)

uktrade · Dec 20, 2023 · a8680b3 · a8680b3
1 parent fe19c98
commit a8680b3
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 9 deletions.
diff --git a/src/client/modules/Tasks/TaskDetails/TaskButtons.jsx b/src/client/modules/Tasks/TaskDetails/TaskButtons.jsx
@@ -12,9 +12,13 @@ import { TASK_ARCHIVE_TASK, buttonState2props } from './state'
 import { GREY_3, TEXT_COLOUR } from '../../../utils/colours'
 
 const ButtonWrapper = styled.div`
+  min-height: 71px;
   * {
     margin-left: ${SPACING.SCALE_4};
   }
+  * {
+    vertical-align: baseline;
+  }
 `
 
 export const TaskButtons = ({ task, returnUrl }) => (
@@ -49,15 +53,12 @@ export const TaskButtons = ({ task, returnUrl }) => (
             Edit
           </Button>
         )}
-        <Button
-          buttonColour={GREY_3}
-          buttonTextColour={TEXT_COLOUR}
-          as={Link}
+        <Link
+          data-test="task-back-link"
           href={returnUrl ?? urls.dashboard.myTasks()}
-          data-test="back-button"
         >
           Back
-        </Button>
+        </Link>
       </ButtonWrapper>
     </GridRow>
   </>

diff --git a/test/component/cypress/specs/Tasks/TaskDetails/TaskButtons.cy.jsx b/test/component/cypress/specs/Tasks/TaskDetails/TaskButtons.cy.jsx
@@ -32,7 +32,7 @@ describe('Task buttons', () => {
     })
 
     it('should show the Back link to dashboard when no return url exists', () => {
-      assertLink('back-button', urls.dashboard.myTasks())
+      assertLink('task-back-link', urls.dashboard.myTasks())
     })
   })
 
@@ -52,7 +52,7 @@ describe('Task buttons', () => {
     })
 
     it('should show the Back link to dashboard when no return url exists', () => {
-      assertLink('back-button', urls.dashboard.myTasks())
+      assertLink('task-back-link', urls.dashboard.myTasks())
     })
   })
 
@@ -64,7 +64,7 @@ describe('Task buttons', () => {
     })
 
     it('should the back button with the returnUrl as the href', () => {
-      assertLink('back-button', 'a/b/c')
+      assertLink('task-back-link', 'a/b/c')
     })
   })
 })