The trailing newline character in line comments is not properly scoped
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 35/100
- Tipo de issue
- Bug
- Clareza
- Razoavelmente clara
- Status de atividade
- Estagnada
- Stack de tecnologia
- typescript
- Domínio
- tooling
Direção de pesquisa
Comece localizando a regra da gramática TextMate do TypeScript que define o scope dos comentários de linha e, em seguida, compare o tratamento da quebra de linha terminadora com o comportamento relatado do Sublime Text. Considera-se concluído quando a quebra de linha estiver no scope como parte do comentário de linha, preservando o suporte aos editores mencionados na descrição do repositório; nenhum arquivo de teste é identificado na issue.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
TypeScript Sublime Package version: 3.8.2
As the syntax currently stands, the newline character that terminates a line comment is not scoped as being a part of the comment body itself. This causes various issues in Sublime Text in particular, although not being familiar with the other editors that are also using this syntax I'm not sure if there's a reason why this is the case or not.
For example, in Sublime the auto_complete_selector setting is used to control in what circumstances the autocompletion popup will automatically trigger. The general practice is to include source - comment in this setting to stop autocompletion from happening inside of a comment, since that is rarely useful. Since the newline isn't scoped, the autocomplete popup triggers constantly while entering comments.
In addition, various plugins and commands attempt to continue comments onto the next line when pressing enter, which generally requires that the end of the current comment be scoped as a comment so as to be syntax agnostic.
In order to support this for TypeScript, extra key bindings or plugin logic to try and detect that the current position is the end of a line comment are required, which arguably is information that the syntax should be providing.
As indicated above, since I'm purely a Sublime user I'm not sure if the other editors targeted by this syntax don't exhibit such issues or if there's some other reason why the newline is not currently being scoped as a comment.
- Linguagem predominante
- TypeScript
- Estrelas
- 471
- Forks
- 149
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Guia de contribuição
Nenhum guia de contribuição indexado para este repositório
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de microsoft/TypeScript-TmLanguage
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
microsoft/TypeScript-TmLanguage#1071 ·
-
triste Aberta
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 5/100
microsoft/TypeScript-TmLanguage#1078 ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 68/100
microsoft/TypeScript-TmLanguage#1076 · 6 comentários ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 55/100
microsoft/TypeScript-TmLanguage#1074 · 2 comentários ·
-
typescript-basics: fix highlighting for triple-slash reference directives with preserve="true" Aberta
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 58/100
microsoft/TypeScript-TmLanguage#1066 · 1 comentário ·
Todas as issues de microsoft/TypeScript-TmLanguage
Issues semelhantes
-
bug(cli): hapi doctor inline-media prints a fabricated B:\ helper-script path in packaged installs Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
-
Crush Aberta
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 85/100
catppuccin/catppuccin#3125 ·
-
Add a SECURITY.md Aberta
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 90/100
ElementsProject/cln-application#167 · 1 comentário · 1 reação ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
Quantco/pnpm-licenses#17 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100