Microsoft/TypeScript

Inlay hints look awkward for callback parameters with a single parameter

Open

#45.389 geöffnet am 10. Aug. 2021

Auf GitHub ansehen
 (4 Kommentare) (5 Reaktionen) (0 zugewiesene Personen)TypeScript (6.726 Forks)batch import
Domain: LS: Inlay HintsExperience EnhancementFix AvailableHelp WantedRescheduledSuggestion

Repository-Metriken

Stars
 (48.455 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 6T 17h) (9 gemergte PRs in 30 T)

Beschreibung

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) => ({
```

Contributor Guide

Research-Richtung
Das Problem meldet, dass Inlay Hinweise für Callbacks mit einem einzelnen Parameter ungeschickt aussehen, wenn die Klammern weggelassen werden. Die Korrektur sollte anpassen, wie der Hinweistext in diesem Fall angezeigt wird. Der relevante Code befindet sich wahrscheinlich im Inlay Hinweise Modul des TypeScript Compiler Dienstes (z. B. src/services/inlayHints.ts oder ähnlich). Der Mitwirkende sollte untersuchen, wie das Hinweis Label für Callback Parameter konstruiert wird, und die Formatierung ändern, um den Parametertyp ohne zusätzliche Leerzeichen oder Zeichen anzuzeigen.
Tech Stack
typescript
Domain
developer experience
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
TypeScriptGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
45