Skip to content
This repository was archived by the owner on Apr 18, 2024. It is now read-only.

Commit 7b5decf

Browse files
authored
fix: LSDV-5131: Fix broken styles for controls (#1386)
`mod` is used inside Button and overrides everything if set directly. So for new UI this way is ok for now, but old code should not have it at all.
1 parent 159f1a2 commit 7b5decf

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

src/components/SidePanels/OutlinerPanel/ViewControls.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -189,10 +189,12 @@ const Grouping = <T extends string>({
189189
);
190190
}, [value, optionsList, readableValue, direction]);
191191

192+
// mods are already set in the button from type, so use it only in new UI
193+
const extraStyles = isFF(FF_DEV_3873) ? { mod: { newUI: true } } : undefined;
192194

193195
return (
194196
<Dropdown.Trigger content={dropdownContent} style={{ width: 200 }}>
195-
<Button type="text" mod={{ newUI: isFF(FF_DEV_3873) }} icon={readableValue.icon} style={{
197+
<Button type="text" {...extraStyles} icon={readableValue.icon} style={{
196198
padding: isFF(FF_DEV_3873) ? '0 6px 0 2px': 0,
197199
whiteSpace: 'nowrap',
198200
}} extra={(

0 commit comments

Comments
 (0)