Skip to content

Commit ea176e1

Browse files
fix: add disable style on "Flux Sync" label (#5585)
* fix: add disable style on "Flux Sync" label * chore: add comment
1 parent eb22bcc commit ea176e1

File tree

2 files changed

+17
-6
lines changed

2 files changed

+17
-6
lines changed

src/dataExplorer/components/Schema.scss

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,10 @@
3535
.flux-sync--label {
3636
padding-left: $cf-space-2xs;
3737

38+
.disabled {
39+
opacity: 50%;
40+
}
41+
3842
.selector-title {
3943
padding-bottom: 0;
4044
}

src/dataExplorer/components/SchemaBrowserHeading.tsx

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,9 @@ const SchemaBrowserHeading: FC = () => {
2525
const {fluxSync, toggleFluxSync} = useContext(FluxQueryBuilderContext)
2626
const {selection} = useContext(PersistanceContext)
2727

28+
// Disable means diverged, used to not allow turning on or off the toggle
2829
const disableToggle: boolean = selection.composition?.diverged
30+
const disableTooltipText = disableToggle ? FLUX_SYNC_DISABLE_TEXT : ''
2931

3032
const handleFluxSyncToggle = () => {
3133
toggleFluxSync(!fluxSync)
@@ -45,14 +47,19 @@ const SchemaBrowserHeading: FC = () => {
4547
onChange={handleFluxSyncToggle}
4648
testID="flux-sync--toggle"
4749
disabled={disableToggle}
48-
tooltipText={disableToggle ? FLUX_SYNC_DISABLE_TEXT : ''}
50+
tooltipText={disableTooltipText}
4951
/>
5052
<InputLabel className="flux-sync--label">
51-
<SelectorTitle
52-
title="Flux Sync"
53-
info={FLUX_SYNC_TOOLTIP}
54-
icon={IconFont.Sync}
55-
/>
53+
<div
54+
className={`${disableToggle ? 'disabled' : ''}`}
55+
title={disableTooltipText}
56+
>
57+
<SelectorTitle
58+
title="Flux Sync"
59+
info={FLUX_SYNC_TOOLTIP}
60+
icon={IconFont.Sync}
61+
/>
62+
</div>
5663
</InputLabel>
5764
</FlexBox>
5865
</FlexBox>

0 commit comments

Comments
 (0)