sourcegraph/sourcegraph-public-snapshot

Fix selection after running a settings quick action

Aperta

#2756 aperta il 15 mar 2019

Vedi su GitHubGuida contributor
 (8 commenti) (0 reazioni) (2 assegnatari)Go (1374 fork)auto 404
bugfrontend-platformgood first issuewebapp

Metriche repository

Star
 (10.290 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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.

Guida contributor

Direzione di ricerca
Indaga il calcolo della selezione nelle azioni rapide delle impostazioni. Il problema è che quando una modifica aggiunge una virgola finale alla riga precedente, l'intervallo di selezione viene calcolato erroneamente. Correggi la logica di selezione per selezionare correttamente la nuova riga aggiunta.
Tech stack
typescript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
TypeScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70