Give pagination bar the same z-index as topbar (#2457)

* give pagination bar the same z-index as topbar * fix e2e test
oxidecomputer · Sep 19, 2024 · fd82458 · fd82458
1 parent 7daaa33
commit fd82458
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/app/layouts/helpers.tsx b/app/layouts/helpers.tsx
@@ -27,7 +27,7 @@ export function ContentPane() {
           <Outlet />
         </main>
       </div>
-      <div className="sticky bottom-0 shrink-0 justify-between overflow-hidden border-t bg-default border-secondary empty:border-t-0">
+      <div className="sticky bottom-0 z-topBar shrink-0 justify-between overflow-hidden border-t bg-default border-secondary empty:border-t-0">
         <Pagination.Target />
         <PageActionsTarget />
       </div>

diff --git a/test/e2e/snapshots.e2e.ts b/test/e2e/snapshots.e2e.ts
@@ -30,6 +30,10 @@ test('Confirm delete snapshot', async ({ page }) => {
 
   const row = page.getByRole('row', { name: 'disk-1-snapshot-6' })
 
+  // scroll a little so the dropdown menu isn't behind the pagination bar
+  await page.getByRole('table').click() // focus the content pane
+  await page.mouse.wheel(0, 200)
+
   async function clickDelete() {
     await row.getByRole('button', { name: 'Row actions' }).click()
     await page.getByRole('menuitem', { name: 'Delete' }).click()