Theming: Distinguish between `import` types
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 38/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- typescript
- Domaine
- tooling
Piste de recherche
Commencez par les règles de grammaire TypeScript impliquées dans makeContext et comparez les scopes affichés pour l’alias d’importation et Knex. Déterminez comment les noms importés sont actuellement classifiés et définissez le scope distinct nécessaire au theming. C’est terminé lorsque les alias d’importation et les classes importées peuvent être stylisés différemment, avec les scopes obtenus vérifiés par rapport à l’exemple.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- TypeScript
- Étoiles
- 471
- Forks
- 149
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de microsoft/TypeScript-TmLanguage
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
microsoft/TypeScript-TmLanguage#1071 ·
-
triste Ouverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 5/100
microsoft/TypeScript-TmLanguage#1078 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 68/100
microsoft/TypeScript-TmLanguage#1076 · 6 commentaires ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 55/100
microsoft/TypeScript-TmLanguage#1074 · 2 commentaires ·
-
typescript-basics: fix highlighting for triple-slash reference directives with preserve="true" Ouverte
Difficulté 1/5 1-3 heures Accessibilité débutants 58/100
microsoft/TypeScript-TmLanguage#1066 · 1 commentaire ·
Toutes les issues de microsoft/TypeScript-TmLanguage
Issues similaires
-
bug(cli): hapi doctor inline-media prints a fabricated B:\ helper-script path in packaged installs Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
-
Crush Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 85/100
catppuccin/catppuccin#3125 ·
-
Add a SECURITY.md Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
ElementsProject/cln-application#167 · 1 commentaire · 1 réaction ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Quantco/pnpm-licenses#17 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100