fix(web): remove discard option from event target selector

Unorchestrated events are discarded by default, so an explicit discard
target is redundant. Drop the discard CommandGroup (and the now-unused
CommandSeparator import); the currentLabel() discard branch is kept so
any pre-existing discard bindings still render correctly.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
Junyan Qin
2026-06-26 19:09:35 +08:00
parent af459c1a72
commit 40e7481032
@@ -30,7 +30,6 @@ import {
CommandInput,
CommandItem,
CommandList,
CommandSeparator,
} from '@/components/ui/command';
import {
Select,
@@ -293,20 +292,6 @@ function TargetCombobox({
))}
</CommandGroup>
)}
<CommandSeparator />
<CommandGroup>
<CommandItem
value="__discard__"
className="text-destructive"
onSelect={() => select('discard')}
>
<Ban className="mr-2 size-3.5 shrink-0" />
{t('bots.targetDiscard')}
{current === 'discard' && (
<Check className="ml-auto size-3.5 shrink-0" />
)}
</CommandItem>
</CommandGroup>
</CommandList>
</Command>
</PopoverContent>