Skip to content

Commit 7608822

Browse files
committed
fix(configurator): hide essentials card in Advanced mode
In Advanced mode the full token catalogue is already visible and responds to the Modified-only and Internal filters. Showing a separate unfiltered Essentials card alongside it broke both filters from the user's perspective: the curated tokens stayed on screen regardless of filter state, making it look like the checkboxes had no effect on most rows. Fix: gate the Essentials card behind `!advanced` — the same guard used for the Basic curated forms. The Advanced catalogue contains every essential token anyway, so nothing is lost; filters now work exactly as expected. https://claude.ai/code/session_01MAgtQ7JY16X2TqZZyGfkuu
1 parent 121577f commit 7608822

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

configurator/src/components/DomainPanel.svelte

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -228,7 +228,7 @@
228228
/>
229229
{/if}
230230
231-
<!-- Essentials (always shown when present) -->
231+
<!-- Essentials (Basic mode only) -->
232232
{#if domain.brandColors}
233233
<!-- Brand colors domain: light/dark pair rows for every brand color -->
234234
<section class="cfg-card panel__card">
@@ -255,12 +255,15 @@
255255
</div>
256256
</section>
257257
{/each}
258-
{:else if essentials.length}
258+
{:else if !advanced && essentials.length}
259+
<!-- In Advanced mode the full filtered catalogue covers everything;
260+
showing essentials here would bypass the Modified-only / Internal
261+
filters and confuse the user with duplicated unfiltered rows. -->
259262
<section class="cfg-card panel__card">
260263
{@render cardHead(
261264
'Essentials',
262265
essentials.length,
263-
advanced ? '' : `Curated for most projects · switch to Advanced (A) for the full ${domainTokens.length}-token catalogue.`
266+
`Curated for most projects · switch to Advanced (A) for the full ${domainTokens.length}-token catalogue.`
264267
)}
265268
<div class="panel__card-rows">
266269
{#each essentials as token (token.name)}

0 commit comments

Comments
 (0)