migrate on:click to onclick for svelte 5

This commit is contained in:
Justin Edmund 2025-12-21 23:51:57 -08:00
parent 103d0c652d
commit 0429bb62bd
6 changed files with 12 additions and 12 deletions

View file

@ -22,7 +22,7 @@
<button <button
class="job-item" class="job-item"
class:selected class:selected
on:click={onClick} onclick={onClick}
aria-pressed={selected} aria-pressed={selected}
aria-label="{job.name.en} - {selected ? 'Currently selected' : 'Click to select'}" aria-label="{job.name.en} - {selected ? 'Currently selected' : 'Click to select'}"
> >

View file

@ -69,7 +69,7 @@
{/if} {/if}
{#if canEdit && job} {#if canEdit && job}
<button class="change-job-button" on:click={onSelectJob} aria-label="Change job"> <button class="change-job-button" onclick={onSelectJob} aria-label="Change job">
<Icon name="arrow-left" size={16} /> <Icon name="arrow-left" size={16} />
</button> </button>
{/if} {/if}
@ -80,7 +80,7 @@
{#if job} {#if job}
<div class="job-header"> <div class="job-header">
{#if canEdit} {#if canEdit}
<button class="job-name clickable" on:click={onSelectJob}> <button class="job-name clickable" onclick={onSelectJob}>
<div class="job-name-row"> <div class="job-name-row">
<img src={jobIconUrl} alt="{job.name.en} icon" class="job-icon" /> <img src={jobIconUrl} alt="{job.name.en} icon" class="job-icon" />
<h3>{job.name.en}</h3> <h3>{job.name.en}</h3>
@ -140,8 +140,8 @@
class:editable={canEdit} class:editable={canEdit}
role={canEdit ? 'button' : undefined} role={canEdit ? 'button' : undefined}
tabindex={canEdit ? 0 : undefined} tabindex={canEdit ? 0 : undefined}
on:click={() => canEdit && onSelectAccessory?.()} onclick={() => canEdit && onSelectAccessory?.()}
on:keydown={(e) => { onkeydown={(e) => {
if (canEdit && onSelectAccessory && (e.key === 'Enter' || e.key === ' ')) { if (canEdit && onSelectAccessory && (e.key === 'Enter' || e.key === ' ')) {
e.preventDefault() e.preventDefault()
onSelectAccessory() onSelectAccessory()

View file

@ -29,7 +29,7 @@
<button <button
class="skill-item {getSkillColorClass(skill)}" class="skill-item {getSkillColorClass(skill)}"
class:current={variant === 'current'} class:current={variant === 'current'}
on:click={onClick} onclick={onClick}
{disabled} {disabled}
aria-label="{skill.name.en} - {getSkillCategoryName(skill)} skill" aria-label="{skill.name.en} - {getSkillCategoryName(skill)} skill"
> >
@ -43,7 +43,7 @@
</div> </div>
{#if variant === 'current' && onRemove} {#if variant === 'current' && onRemove}
<button class="remove-button" on:click|stopPropagation={onRemove} aria-label="Remove skill"> <button class="remove-button" onclick={(e) => { e.stopPropagation(); onRemove?.(); }} aria-label="Remove skill">
Remove Remove
</button> </button>
{/if} {/if}

View file

@ -54,7 +54,7 @@
class="skill-slot editable" class="skill-slot editable"
class:empty={!isFilled} class:empty={!isFilled}
style:--category-color={categoryColor} style:--category-color={categoryColor}
on:click={handleClick} onclick={handleClick}
type="button" type="button"
> >
{@render SlotBody({ locked: false })} {@render SlotBody({ locked: false })}

View file

@ -30,7 +30,7 @@
<button <button
class="tier-button" class="tier-button"
class:selected={selectedTiers.has(tier.value)} class:selected={selectedTiers.has(tier.value)}
on:click={() => handleTierClick(tier.value)} onclick={() => handleTierClick(tier.value)}
type="button" type="button"
> >
{tier.shortLabel} {tier.shortLabel}
@ -42,7 +42,7 @@
<button <button
class="tier-button" class="tier-button"
class:selected={selectedTiers.has(tier.value)} class:selected={selectedTiers.has(tier.value)}
on:click={() => handleTierClick(tier.value)} onclick={() => handleTierClick(tier.value)}
type="button" type="button"
> >
{tier.shortLabel} {tier.shortLabel}

View file

@ -224,10 +224,10 @@
<img <img
src={imageUrl} src={imageUrl}
alt="Test image" alt="Test image"
on:error={(e) => { onerror={(e) => {
e.currentTarget.classList.add('error') e.currentTarget.classList.add('error')
}} }}
on:load={(e) => { onload={(e) => {
e.currentTarget.classList.remove('error') e.currentTarget.classList.remove('error')
}} }}
/> />