Microsoft/monaco-editor

F# highlighting issues with block comments tokenized by strings

Open

#1.240 geöffnet am 18. Dez. 2018

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (1 zugewiesene Person)JavaScript (1.283 Forks)batch import
buggrammarshelp wantedopen for PR

Repository-Metriken

Stars
 (14.836 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 6h 26m) (15 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Das Problem ist ein Fehler in der F# Syntaxhervorhebung, bei dem Zeichenketten innerhalb von Blockkommentaren falsch tokenisiert werden. Die Lösung beinhaltet die Änderung der TextMate Grammatik für F# im Monaco Editor Quellcode (wahrscheinlich 'src/fsharp.ts' oder ähnlich). Untersuchen Sie zunächst die vorhandenen Grammatikmuster, um zu verstehen, wie Blockkommentare und Zeichenketten definiert sind. Passen Sie dann die Muster an, um sicherzustellen, dass Zeichenketten nicht tokenisiert werden, wenn sie innerhalb von Blockkommentaren vorkommen, gemäß der F# Spezifikation. Testen Sie mit den bereitgestellten Beispielen: '(* Here's a code snippet: let s = "*)" *) let i = 1' und '(* """ *) let j = 2'. Prüfen Sie, ob das Problem in späteren Versionen behoben wurde oder ob es bereits einen Pull Request gibt.
Tech Stack
javascripttypescript
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
F# syntax highlightingMonaco editor tokenizer
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
55