Microsoft/monaco-editor

F# highlighting issues with block comments tokenized by strings

Open

#1240 aperta il 18 dic 2018

Vedi su GitHub
 (0 commenti) (0 reazioni) (1 assegnatario)JavaScript (1283 fork)batch import
buggrammarshelp wantedopen for PR

Metriche repository

Star
 (14.836 star)
Metriche merge PR
 (Merge medio 6h 26m) (15 PR mergiate in 30 g)

Descrizione

monaco-editor version: 0.15.6 Browser: Edge OS: Windows NT x64 10.0.17763 Note: Issue also affects VS Code

Steps reproducing the issue: F# tokenizes strings in block comments. Basically, Monaco is breaking the highlighting of code that uses strings within block comments. Here are some images:

This image is from The F# 4.1 Language Specification manual (section 3.2, pg. 23). Imgur

This is how Visual Studio IDE correctly interprets the styling: Imgur

This is how Monaco interpret the styling: Imgur

Comments used to test the highlighter:

(* Here's a code snippet: let s = "*)" *) let i = 1
(* """ *) let j = 2

Guida contributor

Direzione di ricerca
Il problema è un bug nell'evidenziazione della sintassi F# in cui le stringhe all'interno dei commenti a blocco vengono tokenizzate in modo errato. La correzione comporta la modifica della grammatica TextMate per F# nel sorgente dell'editor Monaco (probabilmente 'src/fsharp.ts' o simile). Innanzitutto, esamina i pattern grammaticali esistenti per capire come sono definiti i commenti a blocco e le stringhe. Quindi, modifica i pattern per garantire che le stringhe non vengano tokenizzate quando compaiono all'interno di commenti a blocco, come da specifica F#. Testa con gli esempi forniti: '(* Here's a code snippet: let s = "*)" *) let i = 1' e '(* """ *) let j = 2'. Verifica se il problema è stato risolto nelle versioni successive o se esiste già una pull request.
Tech stack
javascripttypescript
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
F# syntax highlightingMonaco editor tokenizer
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
55