diff --git a/docs/AUDIT_REDESIGN.md b/docs/AUDIT_REDESIGN.md index 1d92993..75fba45 100644 --- a/docs/AUDIT_REDESIGN.md +++ b/docs/AUDIT_REDESIGN.md @@ -56,7 +56,7 @@ Status: ⬜ open · 🟦 in PR · ✅ fixed (commit hash on the right). | D1 | News, ResearchCards, Screener show plain "Loading…" text. Should be skeleton placeholders to prevent layout jank. | 🟡 | ✅ | 0fb1d63 | | D2 | Light theme broke contrast in legacy tabs (`SettingsTab`, `ConfigTab`, etc.) which were styled for dark. | 🟡 | ✅ | 5ca8183 | | D3 | `TickerHeader` exchange is hard-coded "NASDAQ". | 🟢 | ✅ | 1e502b9 | -| D4 | `ScreenerView` "More sectors…" `` doesn't visually highlight the chosen sector pill once selected. | 🟡 | ✅ | 3bf1ca7 | | D5 | Layout is fixed-width (72 px sidebar + 308 px right panel). Breaks below ~1024 px viewport. No mobile-web responsive breakpoints. | 🟠 | ✅ | a9c33b1 | | D6 | `VisualRuleBuilder` and `CodeStrategyEditor` use `setTimeout` without cleanup — leak warning if component unmounts mid-timer. | 🟢 | ⬜ | | | D7 | No keyboard shortcuts (⌘K search, ⌘S save in code editor, ⌘Enter run backtest). | 🟢 | ⬜ | | diff --git a/docs/COMPLETION_CHECKLIST.md b/docs/COMPLETION_CHECKLIST.md index 59e899d..d83a6e2 100644 --- a/docs/COMPLETION_CHECKLIST.md +++ b/docs/COMPLETION_CHECKLIST.md @@ -414,7 +414,7 @@ Expected platform result: Acceptance: exchange comes from profile/market metadata when available and falls back to a neutral placeholder, not always `NASDAQ`. -- [ ] `D4` Highlight selected Screener sector from "More sectors". +- [x] `D4` Highlight selected Screener sector from "More sectors". Implementation: `3bf1ca7`. Acceptance: the selected sector is visually reflected in pills/select state, keyboard and screen-reader state remains clear, and sorting/filtering remains unchanged.