microsoft / microsoft/TypeScript-Sublime-Plugin
Diagnostics for an empty region does not show up
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:

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:

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:

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:


Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Ö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