Skip to content

Fix aware in select option#2469

Merged
ganyicz merged 1 commit intomainfrom
filip/select-option-aware
Mar 5, 2026
Merged

Fix aware in select option#2469
ganyicz merged 1 commit intomainfrom
filip/select-option-aware

Conversation

@ganyicz
Copy link
Collaborator

@ganyicz ganyicz commented Mar 5, 2026

The scenario

Using indicator="checkbox" on a <flux:select> listbox doesn't have any effect with Blaze:

<flux:select variant="listbox" indicator="checkbox" multiple>
    <flux:select.option value="1">Option 1</flux:select.option>
    <flux:select.option value="2">Option 2</flux:select.option>
</flux:select>

The problem

The @aware directive in select/option/index.blade.php didn't include indicator, so Blaze didn't propagate it through the delegate component:

flux:select (indicator="checkbox")
  └ flux:select.option — @aware(['variant']) ← indicator not included
      └ flux:delegate-component
          └ flux:select.option.variants.custom — @aware(['indicator'])

The solution

Add indicator to the @aware directive in select/option/index.blade.php

@aware([ 'variant', 'indicator' ])

Fixes livewire/blaze#115

@ganyicz ganyicz merged commit 497b35c into main Mar 5, 2026
@ganyicz ganyicz deleted the filip/select-option-aware branch March 5, 2026 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

@aware does not propagate through fold+delegate boundaries

1 participant