microsoft / microsoft/TypeScript-TmLanguage
Theming: Distinguish between `import` types
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- TypeScript
- Estrellas
- 471
- Forks
- 149
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
In Atom, we can theme imports based on the type of thing being imported. As you can see below, import aliases/names are styled differently than the class Knex being imported. This has always been immensely handy for at a glance differentiation and validation of imports as well as for organizing imports.
Below is the same theme applied to VS Code, but without the ability to differentiate.
I believe this comes down to a lack of TextMate scope. Here are the available scopes for makeContext:
variable.other.readwrite.alias.ts
meta.import.ts
source.ts
And the available scopes for Knex:
variable.other.readwrite.alias.ts
meta.block.ts
meta.import.ts
source.ts
My ignorant suggestion: it would probably be enough to catch capital-cased aliases for adding an additional scope.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Empieza por las reglas gramaticales de TypeScript implicadas en makeContext y compara los scopes que se muestran para el alias de importación y Knex. Determina cómo se clasifican actualmente los nombres importados y define el scope distinto necesario para el theming. Se considera terminado cuando los alias de importación y las clases importadas puedan estilizarse de forma diferente, con los scopes resultantes verificados frente al ejemplo.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- typescript
- Área
- tooling
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 38/100