Scott.Idem/OSIT-AE-App-Svelte
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: clean up Agreed/Not Agreed button icon classes

Browse Source 
Icons now use class="mr-1" consistent with all other Lucide icons in the
codebase. Removed hardcoded text-green-500 on Check and bg-red-500/text-white
on X — the button's ae_btn_success_filled/ae_btn_warning_filled classes already
handle theming in both light and dark mode via Skeleton presets.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Scott Idem
2026-06-15 17:09:41 -04:00
parent affbd0dc20
commit 06f52bae34
1 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/routes/events/[event_id]/(pres_mgmt)/presenter/[presenter_id]/presenter_page_menu.svelte
View File

@@ -124,7 +124,7 @@ async function on_delete(method: 'delete' | 'disable') {
}} }}
class="btn btn-sm ae_btn_success_filled mx-1 font-bold" class="btn btn-sm ae_btn_success_filled mx-1 font-bold"
title="Agreed to terms and conditions"> title="Agreed to terms and conditions">
<Check size="1em" class="px-1 text-green-500" /> <Check size="1em" class="mr-1" />
Agreed Agreed
</button> </button>
{:else} {:else}
@@ -141,9 +141,7 @@ async function on_delete(method: 'delete' | 'disable') {
}} }}
class="btn btn-sm ae_btn_warning_filled mx-1 font-bold" class="btn btn-sm ae_btn_warning_filled mx-1 font-bold"
title="View terms and conditions"> title="View terms and conditions">
<X <X size="1em" class="mr-1" />
size="1em"
class="mx-1 bg-red-500 px-1 text-white" />
Not yet agreed Not yet agreed
</button> </button>
{/if} {/if}