docs(D4): mark screener sector highlight complete

This commit is contained in:
Saravana Achu Mac 2026-05-04 17:47:52 -07:00
parent 3bf1ca7d51
commit d36c2bae5e
2 changed files with 2 additions and 2 deletions

View File

@ -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…" `<select>` doesn't visually highlight the chosen sector pill once selected. | 🟡 | ⬜ | |
| D4 | `ScreenerView` "More sectors…" `<select>` 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). | 🟢 | ⬜ | |

View File

@ -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.