From 91fe855434c5006f436ebdb62e1d547a29fb916f Mon Sep 17 00:00:00 2001 From: Christian Vidal Wolf Date: Fri, 24 Apr 2026 17:20:47 +0200 Subject: [PATCH] feat: allow applying filters with Enter key --- src/components/ColumnFilterPopover.tsx | 10 ++++++---- src/components/PricingView.tsx | 1 + 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/src/components/ColumnFilterPopover.tsx b/src/components/ColumnFilterPopover.tsx index 28f169c..25ed587 100644 --- a/src/components/ColumnFilterPopover.tsx +++ b/src/components/ColumnFilterPopover.tsx @@ -245,9 +245,11 @@ export function ColumnFilterPopover({ value={search} onChange={e => setSearch(e.target.value)} onKeyDown={(e) => { - if (e.key === 'Enter' && search) { + if (e.key === 'Enter') { e.preventDefault(); - onSelectAll(filteredValues); + if (search) { + onSelectAll(filteredValues); + } onClose(); } }} @@ -371,7 +373,7 @@ export function ColumnFilterPopover({ value={condition.value} onChange={e => setCondition({ ...condition, value: e.target.value })} onKeyDown={(e) => { - if (e.key === 'Enter' && condition.value) { + if (e.key === 'Enter') { e.preventDefault(); applyCondition(); } @@ -389,7 +391,7 @@ export function ColumnFilterPopover({ value={condition.value2 || ''} onChange={e => setCondition({ ...condition, value2: e.target.value })} onKeyDown={(e) => { - if (e.key === 'Enter' && condition.value && condition.value2) { + if (e.key === 'Enter') { e.preventDefault(); applyCondition(); } diff --git a/src/components/PricingView.tsx b/src/components/PricingView.tsx index f1a25e8..af363d9 100644 --- a/src/components/PricingView.tsx +++ b/src/components/PricingView.tsx @@ -2086,6 +2086,7 @@ function TextFilterPopover({ value, onChange, onClose, triggerId }: { placeholder={`Term ${idx + 1}...`} value={term} onChange={e => updateTerm(idx, e.target.value)} + onKeyDown={e => e.key === 'Enter' && onClose()} autoFocus={idx === value.terms.length - 1} className="w-full bg-slate-900 border border-slate-700 rounded-md p-2 pl-8 pr-8 text-xs text-white focus:outline-none focus:border-blue-500 transition-all" />