refactor: split SettingsDialog into per-panel sub-components
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 52/100
- Tipo di issue
- Refactoring
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Tranquilla
- Stack tecnologico
- react, typescript
Direzione di ricerca
Start with src/components/settings/SettingsDialog.tsx and trace how the tab navigation selects each panel and how settings are read and saved. Extract the panels into src/components/settings/panels/, keeping the parent responsible for routing and passing settings through props or the store hook. Done means each panel is under 200 lines, settings persist after an app reload, and TypeScript, tests, and the build pass.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
src/components/settings/SettingsDialog.tsx is 1,321 lines. Each settings panel (General, Editor, Explorer, AI, Keymaps, Templates, Updates) is its own world but they all share one giant file.
What to extract
One PR per panel is fine. Order suggestion (lightest first):
<AISettingsPanel>— the AI provider config (OpenAI / Anthropic / Google / Ollama selector, API key, endpoint, model).<KeymapsPanel>— preset selector + custom-action overrides.<TemplatesPanel>— live-template CRUD.<UpdatesPanel>— the in-app updater UI.<ExplorerSettingsPanel>— the show/hide toggles.<GeneralSettingsPanel>— everything else.
Suggested approach
- Components live in
src/components/settings/panels/. - The parent
SettingsDialogkeeps the tab nav + active-panel routing. - Each panel takes
settings+setSetting(or its store hook) as props — no panel shoulduseSettings()independently if the parent already has it.
Done criteria
SettingsDialog.tsxbecomes a router; each panel < 200 lines.- All settings still persist correctly across app reloads (manual test).
- TypeScript / tests / build green.
Out of scope
- Redesigning the visual layout.
- Adding new settings.
Audit reference
K4 in the pre-launch audit.
- Lingua principale
- TypeScript
- Stelle
- 6
- Fork
- 1
- Merge medio
- 17m
- PR unite (30g)
- 2
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di openidle-dev/queryden
-
Multi Query ClusterAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
openidle-dev/queryden#240 ·
-
bug enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
openidle-dev/queryden#187 ·
-
Refresh database listAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 58/100
openidle-dev/queryden#237 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
openidle-dev/queryden#192 ·
-
dependencies
Difficoltà 2/5 1-3 ore Idoneità per principianti 58/100
openidle-dev/queryden#186 ·
Tutte le issue di openidle-dev/queryden
Issue simili
-
Mend: dependency security vulnerability
Difficoltà 1/5 1-3 ore Idoneità per principianti 65/100
fdm-monster/fdm-monster#5563 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
be-BOP-io-SA/be-BOP#2849 ·
-
1️⃣ phase/under-investigation flaky/confirmed flaky/needs-decision type/bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
growilabs/growi#12246 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
🐛 The Rulekeeper - An index run that gives up on the lock ends SucceededForse già presa @github-actions l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
SSWConsulting/SSW.Rules#2809 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
automated-triage type/bug type/ci
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno