eclipse-theia/theia

[editor] Group unsaved editors in one dialog when prompting user to save them

Offen

#4.849 geöffnet am 10.04.2019

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)TypeScript (2.478 Forks)batch import
editorhelp wantedui/ux

Repository-Metriken

Stars
 (18.676 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

In VS Code, when I close multiple dirty editors, the dirty editors are grouped in one dialog: Screen Shot 2019-04-10 at 09 56 10

Theia asks for each editor: screencast 2019-04-10 09-57-07

Group them. In theory, this can be configurable, but no need to overcomplicate it. Offer the user to save all or nothing. If she is unsure, she can press cancel and manage the dirty editors manually.

Contributor Guide

Research-Richtung
Untersuchen Sie, wie der Editor Schließhandler funktioniert, wahrscheinlich in 'packages/editor/src/browser/editor manager.ts'. Suchen Sie nach der 'saveAll' oder 'closeAll' Logik. Studieren Sie die vorhandene Dialog API in '@theia/core/lib/browser/dialogs', um einen einzigen Bestätigungsdialog zu erstellen, der alle ungespeicherten Editoren auflistet. Das Ziel ist es, mehrere aufeinanderfolgende Abfragen durch einen zusammengefassten Dialog zu ersetzen, der die Optionen Alle speichern, Alle verwerfen oder Abbrechen bietet.
Tech Stack
typescript
Domain
frontend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
TypeScriptGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60