sourcegraph/sourcegraph-public-snapshot

Fix selection after running a settings quick action

Offen

#2.756 geöffnet am 15.03.2019

Auf GitHub ansehenContributor Guide
 (8 Kommentare) (0 Reaktionen) (2 zugewiesene Personen)Go (1.374 Forks)auto 404
bugfrontend-platformgood first issuewebapp

Repository-Metriken

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

Beschreibung

The settings quick actions (the buttons on the global/org/user settings JSON editor pages) yield the wrong selection after clicking them in some cases. The selection should be the newly added line, but it instead is the line above.

This only occurs for actions (such as setSearchContextLines) that DO NOT specify a selection text.

This is because the selection is computed as:

                    selection = monaco.Selection.fromPositions(
                        monacoEdits[0].range.getStartPosition(),
                        monacoEdits[monacoEdits.length - 1].range.getEndPosition()
                    )

and this breaks when the edit needs to add a trailing comma to the preceding line.

Contributor Guide

Research-Richtung
Untersuchen Sie die Berechnung der Auswahl in den Schnellaktionen der Einstellungen. Das Problem ist, dass wenn eine Bearbeitung ein nachgestelltes Komma zur vorherigen Zeile hinzufügt, der Auswahlbereich falsch berechnet wird. Korrigieren Sie die Auswahllogik, um die neu hinzugefügte Zeile korrekt auszuwählen.
Tech Stack
typescript
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
TypeScriptGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70