From 834ae3ceb57644e68f4fee1d2dde33631ae31d6a Mon Sep 17 00:00:00 2001 From: Guillim Date: Tue, 17 Jun 2025 15:45:00 +0200 Subject: [PATCH] Adjutment to the padding (#12619) it is an unusual 2.75 here. I tried to set it to 3 however it requires a big re-work on all cells of the table. The main concern being the new border left of 1px for the "focus mode" Note: to be fair, we are not 100% aligned with the Figma on the paddings and alignment contruction. It is not related to this issue, but makes this fix not ideal since not aligned with our CSS standards. Fixes https://github.com/twentyhq/twenty/issues/12607 --------- Co-authored-by: Charles Bochet --- .../components/RecordTableCellCheckbox.tsx | 1 + .../record-table-header/components/RecordTableHeader.tsx | 2 +- .../components/RecordTableHeaderCheckboxColumn.tsx | 6 ++++-- 3 files changed, 6 insertions(+), 3 deletions(-) diff --git a/packages/twenty-front/src/modules/object-record/record-table/record-table-cell/components/RecordTableCellCheckbox.tsx b/packages/twenty-front/src/modules/object-record/record-table/record-table-cell/components/RecordTableCellCheckbox.tsx index 532c8fe9b..8e83205ba 100644 --- a/packages/twenty-front/src/modules/object-record/record-table/record-table-cell/components/RecordTableCellCheckbox.tsx +++ b/packages/twenty-front/src/modules/object-record/record-table/record-table-cell/components/RecordTableCellCheckbox.tsx @@ -16,6 +16,7 @@ const StyledContainer = styled.div` height: 32px; justify-content: center; min-width: ${TABLE_CELL_CHECKBOX_MIN_WIDTH}; + padding-right: ${({ theme }) => theme.spacing(1)}; `; const StyledRecordTableTd = styled(RecordTableTd)` diff --git a/packages/twenty-front/src/modules/object-record/record-table/record-table-header/components/RecordTableHeader.tsx b/packages/twenty-front/src/modules/object-record/record-table/record-table-header/components/RecordTableHeader.tsx index 5a6211b54..e9b0e74ee 100644 --- a/packages/twenty-front/src/modules/object-record/record-table/record-table-header/components/RecordTableHeader.tsx +++ b/packages/twenty-front/src/modules/object-record/record-table/record-table-header/components/RecordTableHeader.tsx @@ -8,7 +8,7 @@ import { RecordTableHeaderDragDropColumn } from '@/object-record/record-table/re import { RecordTableHeaderLastColumn } from '@/object-record/record-table/record-table-header/components/RecordTableHeaderLastColumn'; import { MOBILE_VIEWPORT } from 'twenty-ui/theme'; -export const FIRST_TH_WIDTH = '9px'; +export const FIRST_TH_WIDTH = '10px'; const StyledTableHead = styled.thead` cursor: pointer; diff --git a/packages/twenty-front/src/modules/object-record/record-table/record-table-header/components/RecordTableHeaderCheckboxColumn.tsx b/packages/twenty-front/src/modules/object-record/record-table/record-table-header/components/RecordTableHeaderCheckboxColumn.tsx index bb1ecc2e0..e997ed7ef 100644 --- a/packages/twenty-front/src/modules/object-record/record-table/record-table-header/components/RecordTableHeaderCheckboxColumn.tsx +++ b/packages/twenty-front/src/modules/object-record/record-table/record-table-header/components/RecordTableHeaderCheckboxColumn.tsx @@ -16,6 +16,8 @@ const StyledContainer = styled.div` display: flex; height: 32px; justify-content: center; + width: 24px; + padding-right: ${({ theme }) => theme.spacing(1)}; background-color: ${({ theme }) => theme.background.primary}; `; @@ -27,8 +29,8 @@ const StyledColumnHeaderCell = styled.th<{ isFirstRowActiveOrFocused ? 'none' : `1px solid ${theme.border.color.light}`}; - border-right: transparent; - width: 30px; + width: 28px; + box-sizing: border-box; `; export const RecordTableHeaderCheckboxColumn = () => {