Microsoft/monaco-editor

F# highlighting issues with block comments tokenized by strings

Open

#1.240 aberto em 18 de dez. de 2018

Ver no GitHub
 (0 comments) (0 reactions) (1 assignee)JavaScript (1.283 forks)batch import
buggrammarshelp wantedopen for PR

Métricas do repositório

Stars
 (14.836 stars)
Métricas de merge de PR
 (Mesclagem média 6h 26m) (15 fundiu PRs em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
O problema é um bug na realce de sintaxe de F# onde strings dentro de comentários de bloco são tokenizadas incorretamente. A correção envolve modificar a gramática TextMate para F# no código fonte do editor Monaco (provavelmente 'src/fsharp.ts' ou similar). Primeiro, examine os padrões de gramática existentes para entender como comentários de bloco e strings são definidos. Em seguida, ajuste os padrões para garantir que strings não sejam tokenizadas quando aparecerem dentro de comentários de bloco, conforme a especificação F#. Teste com os exemplos fornecidos: '(* Here's a code snippet: let s = "*)" *) let i = 1' e '(* """ *) let j = 2'. Verifique se o problema foi resolvido em versões posteriores ou se há uma pull request existente.
Pilha de tecnologia
javascripttypescript
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
F# syntax highlightingMonaco editor tokenizer
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
55