From 1ab78fc5c191a64de85b2f722dbd027015786269 Mon Sep 17 00:00:00 2001 From: Sage Abdullah Date: Fri, 19 Jan 2024 16:23:06 +0000 Subject: [PATCH] Fix first column padding in page listings when custom ordering is active --- client/scss/components/_listing.scss | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/client/scss/components/_listing.scss b/client/scss/components/_listing.scss index e12b92c71d..3395652d13 100644 --- a/client/scss/components/_listing.scss +++ b/client/scss/components/_listing.scss @@ -623,7 +623,12 @@ table.listing { // so the checkbox align with the breadcrumbs toggle. padding-inline-start: theme('spacing.5'); } + } + // Remove left padding from the second column if either bulk action or + // custom ordering is active, as the padding to match the breadcrumbs is + // already handled by the first column. + &:has(.bulk-actions-filter-checkbox, .ord) { th:nth-child(2), td:nth-child(2) { padding-inline-start: 0; @@ -632,7 +637,9 @@ table.listing { // If no bulk actions are present, and nice padding is not applied, // apply the same 80px padding via the first column's left padding. - &:not(.nice-padding &, .report &):not(:has(.bulk-actions-filter-checkbox)) { + &:not(.nice-padding &, .report &):not( + :has(.bulk-actions-filter-checkbox, .ord) + ) { th:first-child, td:first-child { padding-inline-start: theme('spacing.20');