Microsoft/TypeScript

Inlay hints look awkward for callback parameters with a single parameter

Open

#45 389 ouverte le 10 août 2021

Voir sur GitHub
 (4 commentaires) (5 réactions) (0 assignés)TypeScript (6 726 forks)batch import
Domain: LS: Inlay HintsExperience EnhancementFix AvailableHelp WantedRescheduledSuggestion

Métriques du dépôt

Stars
 (48 455 stars)
Métriques de merge PR
 (Merge moyen 6j 17h) (9 PRs mergées en 30 j)

Description

When you have a callback whose parameter has no parentheses, inlay hints starts to look a little bit weird.

For example, in this code

            return hints.map(hint => ({
                ...hint,
                position: scriptInfo.positionToLineOffset(hint.position),
            }));

you get the following:

image

Ideally, you'd see that line rewritten as

callbackfn: (hint: InlayHint) => ({
```

Guide contributeur

Direction de recherche
Le problème signale que les indications intégrées pour les callbacks avec un seul paramètre semblent maladroites lorsque les parenthèses sont omises. La correction doit ajuster la façon dont le texte de l'indication est affiché dans ce cas. Le code pertinent se trouve probablement dans le module des indications intégrées du service du compilateur TypeScript (par exemple, src/services/inlayHints.ts ou similaire). Le contributeur doit examiner comment l'étiquette de l'indication est construite pour les paramètres de callback et modifier le formatage pour afficher le type du paramètre sans espacement ou caractères supplémentaires.
Stack technique
typescript
Domaine
developer experience
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
TypeScriptGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
45