VSinerva/freesewing
1
0
Fork 0
Code Activity

[react] fix: Button state for boolean core options. Fixes #229

Browse source
This commit is contained in:
joostdecock 2025-04-28 07:16:39 +02:00
parent 663528fa36
commit 5cfa9df287
1 changed files with 12 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

16
packages/react/components/Editor/components/HeaderMenu.mjs
View file

@ -312,19 +312,27 @@ export const HeaderMenuDraftViewIcons = (props) => {
{ux >= levels.complete ? (
<Button
lgOnly
updateHandler={() => update.settings('complete', settings.complete ? 0 : 1)}
updateHandler={() =>
update.settings('complete', [false, 0, '0'].includes(settings.complete) ? 1 : 0)
}
tooltip="Turns Details on or off (see Core Settings)"
>
<DetailIcon className={`${size} ${!settings.complete ? 'tw:text-secondary' : muted}`} />
<DetailIcon
className={`${size} ${[false, 0, '0'].includes(settings.complete) ? 'tw:text-secondary' : muted}`}
/>
</Button>
) : null}
{ux >= levels.expand ? (
<Button
lgOnly
updateHandler={() => update.settings('expand', settings.expand ? 0 : 1)}
updateHandler={() =>
update.settings('expand', [false, 0, '0'].includes(settings.expand) ? 1 : 0)
}
tooltip="Turns Expand on or off (see Core Settings)"
>
<ExpandIcon className={`${size} ${settings.expand ? 'tw:text-secondary' : muted}`} />
<ExpandIcon
className={`${size} ${[false, 0, '0'].includes(settings.expand) ? 'tw:text-secondary' : muted}`}
/>
</Button>
) : null}
<HeaderMenuIconSpacer />