fix: address code review - revert description to plain text, remove empty callback

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Author: Copilot

ROADMAP.md

@@ -49,7 +49,7 @@ Everything below has been built, tested, and verified. These items are stable an
 - **Animation:** 7 presets, reduced-motion aware, page transitions (9 types with View Transitions API).
 - **Notifications:** Toast/banner/snackbar with full CRUD integration.
 - **View Enhancements:** Gallery, column summary, grouping, row color, density modes, view sharing, ViewTabBar (reorder, pin, context menu, type-switch, personal/shared grouping).
-- **Inline View Config Panel:** Airtable-style right sidebar for view configuration (Page, Data, Appearance, User Filters, Actions, Advanced), breadcrumb header, record count footer, responsive mobile overlay, ARIA accessibility, auto-close on view switch — no page navigation required.
+- **Inline View Config Panel:** Airtable-style right sidebar for view configuration (Page, Data, Appearance, User Filters, Actions, Advanced), breadcrumb header, record count footer, responsive mobile overlay, ARIA accessibility, auto-close on view switch — no page navigation required. Full interactive editing support: inline title editing, Switch toggles for all boolean settings, ViewType select, clickable rows to open sub-editors (columns/filters/sort), local draft state with Save/Discard workflow, i18n in all 10 locales.
 
 ### Enterprise Features ✅
 

ROADMAP_CONSOLE.md

@@ -1021,7 +1021,7 @@ These were the initial tasks to bring the console prototype to production-qualit
 | **Global Undo/Redo (Ctrl+Z)** | ✅ Done (global UndoManager + batch ops + persistent stack) | Post v1.0 | Phase 16 (L1+L2) |
 | Notification center | ✅ Partial (ActivityFeed with filter preferences) | Post v1.0 | Phase 17 (L2) |
 | Activity feed | ✅ Done | Post v1.0 | Phase 17 (L1) |
-| **Inline View Config Panel** | ✅ Done (Airtable-style right sidebar, breadcrumb header, record count footer, responsive mobile overlay, ARIA accessibility, auto-close on view switch) | Post v1.0 | Phase 20 |
+| **Inline View Config Panel** | ✅ Done (Airtable-style right sidebar, breadcrumb header, record count footer, responsive mobile overlay, ARIA accessibility, auto-close on view switch, full interactive editing: Switch toggles, inline title edit, ViewType select, clickable sub-editor rows, draft state with Save/Discard) | Post v1.0 | Phase 20 |
 
 ### 5.5 Kanban & Visual Views
 
@@ -1103,7 +1103,7 @@ These were the initial tasks to bring the console prototype to production-qualit
 
 2027 Q1+ — v2.1: INLINE VIEW DESIGNER (✅ Complete)
 ═══════════════════════════════════════════════════════════
-  Phase 20: Inline ViewConfigPanel    ██████████████  ✅ Complete: Airtable-style right sidebar, breadcrumb header, record count footer, responsive mobile overlay, ARIA accessibility, auto-close on view switch
+  Phase 20: Inline ViewConfigPanel    ██████████████  ✅ Complete: Airtable-style right sidebar, full interactive editing (Switch toggles, inline title, ViewType select, sub-editor rows, draft Save/Discard), ARIA accessibility
 ```
 
 ### Milestone Summary
@@ -1118,7 +1118,7 @@ These were the initial tasks to bring the console prototype to production-qualit
 | **v1.1** | v1.1.0 | ✅ Complete | Kanban + Forms + Import/Export (Phases 13-15); all L1 ✅ |
 | **v1.2** | v1.2.0 | ✅ L1 Complete | Undo/Redo + Collaboration (Phases 16-17); L1 integrated into console |
 | **v2.0** | v2.0.0 | ✅ L2 Complete | All L2 features: batch undo, expression formatting, conditional triggers, multi-step actions, swimlane persistence, keyboard nav, file validation, thread resolution, notification prefs |
-| **v2.1** | v2.1.0 | ✅ Complete | Inline ViewConfigPanel (Phase 20): Airtable-style right sidebar for view configuration |
+| **v2.1** | v2.1.0 | ✅ Complete | Inline ViewConfigPanel (Phase 20): Airtable-style right sidebar with full interactive editing support |
 
 ---
 

apps/console/src/__tests__/ViewConfigPanel.test.tsx

@@ -114,7 +114,7 @@ describe('ViewConfigPanel', () => {
         expect(titleInput).toHaveValue('All Records');
     });
 
-    it('displays object description in readonly input', () => {
+    it('displays object description as plain text', () => {
         render(
             <ViewConfigPanel
                 open={true}
@@ -124,12 +124,10 @@ describe('ViewConfigPanel', () => {
             />
         );
 
-        const descInput = screen.getByTestId('view-description-input');
-        expect(descInput).toHaveValue('Track sales pipeline and deals');
-        expect(descInput).toHaveAttribute('readOnly');
+        expect(screen.getByText('Track sales pipeline and deals')).toBeInTheDocument();
     });
 
-    it('shows placeholder when object has no description', () => {
+    it('shows "no description" when object has no description', () => {
         render(
             <ViewConfigPanel
                 open={true}
@@ -139,8 +137,7 @@ describe('ViewConfigPanel', () => {
             />
         );
 
-        const descInput = screen.getByTestId('view-description-input');
-        expect(descInput).toHaveAttribute('placeholder', 'console.objectView.noDescription');
+        expect(screen.getByText('console.objectView.noDescription')).toBeInTheDocument();
     });
 
     it('displays column count', () => {

apps/console/src/components/ObjectView.tsx

@@ -69,10 +69,6 @@ export function ObjectView({ dataSource, objects, onEdit, onRowClick }: any) {
     // Draft state for view config edits — cached locally, saved on demand
     const [viewDraft, setViewDraft] = useState<Record<string, any> | null>(null);
 
-    const handleViewConfigUpdate = useCallback((field: string, value: any) => {
-        // Buffered locally — ViewConfigPanel manages its own draft
-    }, []);
-    
     const handleViewConfigSave = useCallback((draft: Record<string, any>) => {
         setViewDraft(draft);
         setRefreshKey(k => k + 1);
@@ -505,7 +501,6 @@ export function ObjectView({ dataSource, objects, onEdit, onRowClick }: any) {
                     activeView={activeView}
                     objectDef={objectDef}
                     recordCount={recordCount}
-                    onViewUpdate={handleViewConfigUpdate}
                     onOpenEditor={handleOpenEditor}
                     onSave={handleViewConfigSave}
                 />

apps/console/src/components/ViewConfigPanel.tsx

@@ -208,13 +208,9 @@ export function ViewConfigPanel({ open, onClose, activeView, objectDef, onViewUp
                         />
                     </ConfigRow>
                     <ConfigRow label={t('console.objectView.description')}>
-                        <Input
-                            data-testid="view-description-input"
-                            className="h-7 text-xs w-32 text-right italic"
-                            value={objectDef.description || ''}
-                            placeholder={t('console.objectView.noDescription')}
-                            readOnly
-                        />
+                        <span className="text-xs text-muted-foreground italic truncate ml-4 text-right">
+                            {objectDef.description || t('console.objectView.noDescription')}
+                        </span>
                     </ConfigRow>
                 </div>