search result item labels (#456)
## Summary - show element and proficiency labels below item name in search results - remove element text from right side of row - update hover to use background color instead of border - clean up unused filter CSS ## Type changes - update proficiency type to `number | number[]` to handle both weapons (single) and characters (array)
This commit is contained in:
parent
5edb225d2d
commit
a6d77a4463
4 changed files with 39 additions and 70 deletions
|
|
@ -53,8 +53,8 @@ export interface SearchResult {
|
|||
element?: number
|
||||
/** Rarity level */
|
||||
rarity?: number
|
||||
/** Weapon/Character proficiency */
|
||||
proficiency?: number
|
||||
/** Proficiency - number for weapons, array for characters */
|
||||
proficiency?: number | number[]
|
||||
/** Series - object for weapons (WeaponSeriesRef), number array for characters */
|
||||
series?: { id: string; slug: string; name: { en: string; ja: string } } | number[]
|
||||
/** URL for entity image */
|
||||
|
|
|
|||
|
|
@ -41,7 +41,8 @@ export interface SearchPageResult {
|
|||
name: { en?: string; ja?: string }
|
||||
element?: number
|
||||
rarity?: number
|
||||
proficiency?: number
|
||||
/** Proficiency - number for weapons, array for characters */
|
||||
proficiency?: number | number[]
|
||||
/** Series - object for weapons, number array for characters */
|
||||
series?: { id: string; slug: string; name: { en: string; ja: string } } | number[]
|
||||
imageUrl?: string
|
||||
|
|
|
|||
|
|
@ -12,6 +12,8 @@
|
|||
import Icon from '../Icon.svelte'
|
||||
import Input from '../ui/Input.svelte'
|
||||
import CharacterTags from '$lib/components/tags/CharacterTags.svelte'
|
||||
import ElementLabel from '$lib/components/labels/ElementLabel.svelte'
|
||||
import ProficiencyLabel from '$lib/components/labels/ProficiencyLabel.svelte'
|
||||
import ElementPicker from '../ui/element-picker/ElementPicker.svelte'
|
||||
import RarityPicker from '../ui/rarity-picker/RarityPicker.svelte'
|
||||
import ProficiencyPicker from '../ui/proficiency-picker/ProficiencyPicker.svelte'
|
||||
|
|
@ -55,18 +57,6 @@
|
|||
// Refs
|
||||
let sentinelEl = $state<HTMLElement>()
|
||||
|
||||
// Constants
|
||||
const elements = [
|
||||
{ value: 0, label: 'Null', color: 'var(--grey-50)' },
|
||||
{ value: 1, label: 'Wind', color: 'var(--wind-bg)' },
|
||||
{ value: 2, label: 'Fire', color: 'var(--fire-bg)' },
|
||||
{ value: 3, label: 'Water', color: 'var(--water-bg)' },
|
||||
{ value: 4, label: 'Earth', color: 'var(--earth-bg)' },
|
||||
{ value: 5, label: 'Dark', color: 'var(--dark-bg)' },
|
||||
{ value: 6, label: 'Light', color: 'var(--light-bg)' }
|
||||
]
|
||||
|
||||
|
||||
// Debounce search query changes
|
||||
$effect(() => {
|
||||
const query = searchQuery
|
||||
|
|
@ -467,7 +457,21 @@
|
|||
class="result-image"
|
||||
loading="lazy"
|
||||
/>
|
||||
<span class="result-name">{getItemName(item)}</span>
|
||||
<div class="result-info">
|
||||
<span class="result-name">{getItemName(item)}</span>
|
||||
<div class="result-labels">
|
||||
{#if item.element !== undefined}
|
||||
<ElementLabel element={item.element} size="small" />
|
||||
{/if}
|
||||
{#if Array.isArray(item.proficiency)}
|
||||
{#each item.proficiency as prof}
|
||||
<ProficiencyLabel proficiency={prof} size="small" />
|
||||
{/each}
|
||||
{:else if item.proficiency !== undefined}
|
||||
<ProficiencyLabel proficiency={item.proficiency} size="small" />
|
||||
{/if}
|
||||
</div>
|
||||
</div>
|
||||
{#if type === 'character'}
|
||||
<CharacterTags character={item} />
|
||||
{/if}
|
||||
|
|
@ -476,14 +480,6 @@
|
|||
{:else if owned}
|
||||
<Icon name="check" size={14} class="owned-indicator" />
|
||||
{/if}
|
||||
{#if item.element !== undefined}
|
||||
<span
|
||||
class="result-element"
|
||||
style:color={elements.find(e => e.value === item.element)?.color}
|
||||
>
|
||||
{elements.find(e => e.value === item.element)?.label}
|
||||
</span>
|
||||
{/if}
|
||||
</button>
|
||||
</li>
|
||||
{/each}
|
||||
|
|
@ -617,40 +613,6 @@
|
|||
color: var(--text-primary);
|
||||
}
|
||||
}
|
||||
|
||||
.filter-buttons {
|
||||
display: flex;
|
||||
flex-wrap: wrap;
|
||||
gap: $unit-half;
|
||||
}
|
||||
|
||||
.proficiency-grid {
|
||||
display: grid;
|
||||
grid-template-columns: repeat(3, 1fr);
|
||||
gap: $unit-half;
|
||||
}
|
||||
|
||||
.filter-btn {
|
||||
padding: $unit-half $unit;
|
||||
border: 1px solid var(--border-primary);
|
||||
background: var(--bg-secondary);
|
||||
border-radius: $input-corner;
|
||||
font-size: $font-small;
|
||||
cursor: pointer;
|
||||
transition: all 0.2s;
|
||||
color: var(--text-primary);
|
||||
|
||||
&:hover {
|
||||
background: var(--bg-tertiary);
|
||||
border-color: var(--border-secondary);
|
||||
}
|
||||
|
||||
&.active {
|
||||
background: var(--accent-blue);
|
||||
color: white;
|
||||
border-color: var(--accent-blue);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
.results-section {
|
||||
|
|
@ -682,16 +644,15 @@
|
|||
align-items: center;
|
||||
gap: $unit;
|
||||
padding: $unit;
|
||||
border: 1px solid transparent;
|
||||
border: none;
|
||||
border-radius: $input-corner;
|
||||
background: var(--bg-secondary);
|
||||
background: transparent;
|
||||
cursor: pointer;
|
||||
transition: all 0.2s;
|
||||
transition: background-color 0.15s ease-out;
|
||||
text-align: left;
|
||||
|
||||
&:hover {
|
||||
background: var(--bg-tertiary);
|
||||
border-color: var(--accent-blue);
|
||||
}
|
||||
|
||||
&:active:not(:disabled) {
|
||||
|
|
@ -702,11 +663,9 @@
|
|||
&:disabled {
|
||||
opacity: 0.5;
|
||||
cursor: not-allowed;
|
||||
background: var(--bg-disabled);
|
||||
|
||||
&:hover {
|
||||
background: var(--bg-disabled);
|
||||
border-color: transparent;
|
||||
background: transparent;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
@ -720,16 +679,23 @@
|
|||
flex-shrink: 0;
|
||||
}
|
||||
|
||||
.result-name {
|
||||
.result-info {
|
||||
flex: 1;
|
||||
display: flex;
|
||||
flex-direction: column;
|
||||
gap: $unit-half;
|
||||
min-width: 0;
|
||||
}
|
||||
|
||||
.result-name {
|
||||
font-size: $font-regular;
|
||||
color: var(--text-primary);
|
||||
}
|
||||
|
||||
.result-element {
|
||||
font-size: $font-small;
|
||||
font-weight: $bold;
|
||||
flex-shrink: 0;
|
||||
.result-labels {
|
||||
display: flex;
|
||||
align-items: center;
|
||||
gap: $unit-half;
|
||||
}
|
||||
|
||||
:global(.collection-indicator) {
|
||||
|
|
|
|||
|
|
@ -28,6 +28,8 @@ export interface AddItemResult {
|
|||
element?: number
|
||||
/** Rarity level */
|
||||
rarity?: number
|
||||
/** Proficiency - number for weapons, array for characters */
|
||||
proficiency?: number | number[]
|
||||
/** Collection ID if the item was selected from user's collection */
|
||||
collectionId?: string
|
||||
/** Type of entity */
|
||||
|
|
|
|||
Loading…
Reference in a new issue