Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Settings robustness and dialog polish

Aperta
  • Qualcosa non va?
#53 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
38/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
react, rust, tauri, typescript
Ambito
database, desktop, testing

Direzione di ricerca

Start with docs/notes/plan-05-followups.md#deferred-minors and the listed files: src-tauri/src/settings.rs, src/stores/settings.ts, src/main.tsx, src/App.tsx, and the three dialog and window-title files. Run pnpm test to map existing coverage, then address each listed concern and add a test or record a one-line ruling for it. Done when pnpm test && pnpm lint pass, plus pnpm test:rust if Rust files change.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

area:backend area:frontend size:M status:ready type:task
Context

Deferred minors of Plan 5 Tasks 1, 3 and 4.
Source: docs/notes/plan-05-followups.md (Settings, Safe mode, UI shell).

To do
  • Settings file: parent directory not fsynced after the atomic rename; a numeric overflow or negative value resets ALL fields (clamp that field instead).
  • The theme is applied after an async IPC (one frame of the system theme at startup); main.tsx load() untested; load() not sequenced against an in-flight update(), a failed stale update's error is dropped.
  • ⚙ glyph hard-coded in App.tsx (move to strings.ts); no tests for the dialog reverting after a failed settings_set.
  • Font names: the grammar is ASCII only (CJK font names cannot be set).
  • ConfirmRunDialog and SettingsDialog: the Tab trap works only while focus is inside the overlay; useWindowTitle swallows windowSetTitle errors; double border-b between the top bar and WorkspaceSwitcher.
Acceptance criteria
  • Each item fixed with a test, or closed with a one-line ruling in the PR
  • pnpm test && pnpm lint green (and pnpm test:rust when src-tauri/ changes)
Files likely touched
  • src-tauri/src/settings.rs, src/stores/settings.ts, src/main.tsx, src/App.tsx
  • src/features/settings/SettingsDialog.tsx, src/features/safety/ConfirmRunDialog.tsx, src/features/workspace/useWindowTitle.ts
Related plan / spec

docs/notes/plan-05-followups.md#deferred-minors

Lingua principale
TypeScript
Stelle
0
Fork
0
Merge medio
1h 2m
PR unite (30g)
41

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di Cemoche/tablemax

Tutte le issue di Cemoche/tablemax

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.