question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Faulty popup help for comma inside an object in an array (again)

See original GitHub issue

TS Template added by @mjbvz

TypeScript Version: 4.2.0-dev.20210112

Search Terms

  • signatureHelp
  • parameter hints

  • VSCode Version: 1.52.1
  • OS version: Darwin x64 19.6.0

I can’t seem to re-open my last issue (#106738) so I created this one. I just want to know if you can reproduce the issue (being “push”-popup-help appearing after pressing comma) in the playground link provided.

Playground Link

If you can reproduce it there but playground is not good enough for this kind of an issue I will settle for rewriting the code affected. It’s not a huge deal, but the popup is mildly annoying when it’s irrelevant.

Does this issue occur when all extensions are disabled?: Yes

Issue Analytics

  • State:open
  • Created 3 years ago
  • Comments:10 (5 by maintainers)

github_iconTop GitHub Comments

1reaction
MrAndersen1commented, Jan 29, 2021

@DanielRosenwasser I can reproduce it without fail in our project (which I cannot share unfortunately) but not in a clean project. Since I could also reproduce it in the playground, and it has limited resources available, it led me to believe it was a timing issue where everyone with a faster computer than me couldn’t reproduce it. But that was just a guess based on those observations along with the fact the cpu has to work a lot more in a big project vs a clean one.

You guys are way more likely to find the issue than me guessing 😃 Might be monaco then? Anyways, if I can help in any way let me know 😃 Though I’m not familiar with server traces and signature help.

Edit: It happens in vscode too, that’s where I originally ran into it, just so you know. If it uses monaco too though then it checks out.

0reactions
MrAndersen1commented, Mar 8, 2021

It doesn’t reproduce when I use TypeScript › Tsserver: Trace = “messages” and look at the output, which makes it look more and more like a timing issue to me, could be the reason it’s so difficult for everyone else to reproduce.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Faulty popup help for comma inside an object in an array · Issue ...
Put the cursor after "id": 4 (still inside the object) and press comma. We then get a popup about Push when we shouldn't....
Read more >
Trailing commas - JavaScript - MDN Web Docs
JavaScript allows trailing commas wherever a comma-separated list of values is accepted and more values may be expected after the last item.
Read more >
Finding the comma, where is it coming from? Javascript Arrays ...
The problem is HCP365Object.nonHCP365Pixels.map((x) => x) . map() returns an array, and calling toString() will list all items separated by ...
Read more >
Windows PowerShell Cheat Sheet
Change is persistent, a change (see below) within PowerShell is volatile. ... The array object $a is piped to the alias sort of...
Read more >
Best practices for using trailing commas in JavaScript
In practice, however, you rarely need to work with a sparse array. And if you do, your existing code will most likely handle...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found