From bd141e35857f9aefd3e35fca6131bb72334990e4 Mon Sep 17 00:00:00 2001
From: "vamsi.balaga" <vamsi.balaga@getzino.com>
Date: Tue, 3 Nov 2026 11:19:07 +0530
Subject: [PATCH] feat: show the search box on every record view list

The backend searches its columns even though it marks every field
is_search: false, so the box was never shown. Render it always and keep
the field-name placeholder for views that do flag searchable fields.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 src/components/activity/RecordView.jsx | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/src/components/activity/RecordView.jsx b/src/components/activity/RecordView.jsx
index 25fa9f6..8727fa0 100644
--- a/src/components/activity/RecordView.jsx
+++ b/src/components/activity/RecordView.jsx
@@ -46,6 +46,8 @@ export default function RecordView({
     return () => clearTimeout(id)
   }, [term, query.search, setSearch])
 
+  // Names the placeholder only: the backend searches its own columns whatever
+  // is_search says (it returns false on every field), so the box always shows.
   // A rich column's fields stay searchable.
   const searchable = fields
     .flatMap((f) => (f.data_type === 'rich' ? [...f.title, ...f.subtitle] : [f]))
@@ -66,13 +68,15 @@ export default function RecordView({
   return (
     <>
       <Toolbar count={total} countLabel="records">
-        {searchable.length > 0 && (
-          <SearchInput
-            value={term}
-            onChange={setTerm}
-            placeholder={`Search by ${searchable.map((f) => displayLabel(f.output_label)).join(', ')}…`}
-          />
-        )}
+        <SearchInput
+          value={term}
+          onChange={setTerm}
+          placeholder={
+            searchable.length > 0
+              ? `Search by ${searchable.map((f) => displayLabel(f.output_label)).join(', ')}…`
+              : 'Search…'
+          }
+        />
         {toolbar}
         {loading && <span className="toolbar__count">Loading…</span>}
       </Toolbar>