Skip to content

Commit 8dcf863

Browse files
authored
chore(compass-components): only put the border on selected themed tabs COMPASS-8025 (#5953)
only put the border on selected themed tabs
1 parent 370988a commit 8dcf863

File tree

1 file changed

+7
-2
lines changed
  • packages/compass-components/src/components/workspace-tabs

1 file changed

+7
-2
lines changed

packages/compass-components/src/components/workspace-tabs/tab.tsx

+7-2
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ const tabStyles = css({
3737
overflow: 'hidden',
3838

3939
// leave space so the active and other tabs line up
40-
borderTop: '4px solid transparent',
40+
paddingTop: spacing[100],
4141

4242
backgroundColor: 'var(--workspace-tab-background-color)',
4343
color: 'var(--workspace-tab-color)',
@@ -110,14 +110,18 @@ const tabDarkThemeStyles = css({
110110
const selectedTabStyles = css({
111111
color: 'var(--workspace-tab-selected-color)',
112112
backgroundColor: 'var(--workspace-tab-selected-background-color)',
113-
borderTopColor: 'var(--workspace-tab-selected-border-color)',
114113
boxShadow: 'inset -1px 0 0 0 var(--workspace-tab-border-color)',
115114

116115
'&:hover': {
117116
cursor: 'default',
118117
},
119118
});
120119

120+
const selectedThemedTabStyles = css({
121+
borderTop: `${spacing[100]}px solid var(--workspace-tab-selected-border-color)`,
122+
paddingTop: 0,
123+
});
124+
121125
const draggingTabStyles = css({
122126
cursor: 'grabbing !important',
123127
});
@@ -240,6 +244,7 @@ function Tab({
240244
tabStyles,
241245
themeClass,
242246
isSelected && selectedTabStyles,
247+
isSelected && tabTheme && selectedThemedTabStyles,
243248
isDragging && draggingTabStyles,
244249
subtitle && animatedSubtitleStyles
245250
)}

0 commit comments

Comments
 (0)