microsoft / microsoft/TypeScript-Sublime-Plugin

Diagnostics for an empty region does not show up

Offen
#637 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
JavaScript
Sterne
1.7k
Forks
236
Ø Merge
21 T. 13 Std.
Gemergte PRs (30 T.)
1

Beschreibung

I'm using tslint-language-service, and it works fine except for rules like semicolon or trailing-comma. The diagnostics produced by these rules correctly end up in Sublime, but they don't get rendered because they appear inside an empty region at the end of the line and Sublime refuses to render them:

image

I've added a quick workaround, like this:

diff --git typescript/listeners/idle.py typescript/listeners/idle.py
index f48df56..30e6a8f 100644
--- typescript/listeners/idle.py
+++ typescript/listeners/idle.py
@@ -146,6 +146,9 @@ class IdleListener:
                     # character instead so user can still see the highlight.
                     if start == view.size() and start == end:
                         region = last_visible_character_region(view)
+                    elif start == view.line(start).end() and start == end:
+                        end = view.text_point(start_line + 1, 0)
+                        region = sublime.Region(start, end)
                     else:
                         region = sublime.Region(start, end)

which seems to at least put a small red squiggle at the end of the line:

image

but hovering over it to get the message doesn't work. It also seems that the issue is more general, in that any region that only contains whitespace does not show up, e.g. when I add some trailing spaces:

image

so while this is a conservative change that solves this specific issue with semicolon and trailing-comma, I'm not sure this is the right solution in the general case. SublimeLinter adds a gutter mark for empty regions, so maybe that's a viable solution?

For reference, this behaves properly in VSCode:

image
image

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne in typescript/listeners/idle.py bei der Regionenkonstruktion von IdleListener und reproduziere das Problem mit Diagnosen zu Semikolons oder nachgestellten Kommas an Zeilenenden sowie mit Regionen, die nur aus Leerraum bestehen. Vergleiche die bestehende Umgehungslösung mit dem Gutter-Verhalten von SublimeLinter; als erledigt gilt die Aufgabe, wenn leere oder nur aus Leerraum bestehende Diagnosen sichtbar sind und ihre Meldungen in Sublime geöffnet werden können.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python, typescript
Bereich
developer-experience, tooling
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.