Remove popups and hints in text area. #556

mafflin · 2016-10-28T18:03:01Z

Please add option to disable annoying popups and alerts.

bjnsn · 2016-11-01T00:03:10Z

If you're referring to this (which shows up when pausing typing), then I (politely) second the need for control.

I thought this was the same as #544 but show_errors_inline doesn't seem to impact it.

ghost · 2016-11-01T08:39:39Z

Absolutely agree. I had to manually edit the script to get rid of the annoying popup.
I also noticed that after recent changes my color scheme is broken now

bypotatoes · 2016-11-01T08:55:28Z

+1

lorem--ipsum · 2016-11-02T07:53:19Z

Who thought this could be a good idea ? If I wanted annoying popups with bright colors, I'd be using Webstorm.

xaralis · 2016-11-02T09:45:06Z

Please please, make this optional, the worst thing about it is that it makes your code lines jump back and forth and makes coding pretty creepy.

roblav96 · 2016-11-02T11:12:45Z

#thisfeaturesucks lol

plwalters · 2016-11-06T15:21:04Z

@ledz3pp can you share what you did to get rid of this?

bjnsn · 2016-11-07T19:22:38Z

@PWKad Reverting to version 0.1.16 is one short-term option (worked for me).

plwalters · 2016-11-08T15:48:27Z

@bjnsn nice, I'll do that now as this is causing me to have a bunch of typos that I've never seen in Sublime before as the box shows and hides itself on in multi-line text mode.

cdeutsch · 2016-11-11T21:11:21Z

This makes sublime unusable. Revert

KirillGrishin · 2016-11-13T13:10:07Z

This is really annoying. I am pretty sure those who thought of it had best intentions in mind, but this is so annoying and distracting. Please please please, turn it off, or make optional.

p-mcgowan · 2016-11-14T19:39:46Z

To anyone else looking for the solution: comment lines 26 - 29 (inclusive) in .../Packges/TypeScript/typescript/commands/error_info.py and restart sublime

...
if len(error_text) > 0:
  # if PHANTOM_SUPPORT:
  #       template = '<body><style>div.error {{ background-color: brown; padding: 5px; color: white }}</style><div class="error">{0}</div></body>'
  #       display_text = template.format(error_text)
  #       self.view.add_phantom("typescript_error", self.view.sel()[0], display_text, sublime.LAYOUT_BLOCK)
  self.view.set_status("typescript_error", error_text)

caiogondim · 2016-11-23T13:00:02Z

What happens when the package is updated?
Will we have to change the code after every update?

Constructive critique here:
This popup is def not "Sublime-like"
One thing that make Sublime users use it on the first place is it's focus on content
Please, consider putting this feature behind a flag

p-mcgowan · 2016-11-23T18:12:39Z

Yes, you will likely have to make the changes again after the update. PR-562 looks like it will solve this problem.

drdanmorris · 2016-11-23T21:35:30Z

Yes, please remove the annoying red inline error boxes. The TypeScript plugin is generally awesome, but this was an awful idea. I'll be doing the manual script hack for now (thanks p-mcgowan) until sanity has prevailed.

speedy250 · 2016-11-30T19:25:00Z

I've been looking for this solution in all the wrong places; anywhere but the Microsoft plugin. Bizarre addition to the plugin, and thanks p-mcgowan for the hack.

distinctdan · 2016-12-03T23:07:44Z

+1 for removing this, although my main issue is that it's popping up for errors that aren't really errors. For example, I have a line that reads: public activeContainer: JQuery;, which causes the plugin to error with "Cannot find name JQuery". However, my project compiles fine (I'm using grunt to do my compilation). I've included JQuery's .d.ts. Also, I'm not using the /// comment at the top of my files because the modern ts compiler doesn't need it, not sure if that matters.

plwalters · 2016-12-03T23:47:14Z

Dan that's a completely separate issue sublime isn't picking up your typings or perhaps even the tsconfig but your build is

abenhamdine · 2016-12-12T09:07:59Z

It's done, you can clone the current repo into your Packages folder.

distinctdan · 2016-12-18T02:56:58Z

Yup, thanks @PWKad, at first I didn't realize I needed a tsconfig since I wasn't using the build system and I didn't want to mess up my grunt process, but adding one did the trick.

Now about the popup, I do like that it's showing useful information in a relevant context. I think the main issue is the presentation. It feels weird that it shifts the text, and it's kind of annoying that it's big and red and not closable if I press escape.

I think it may work better as a tooltip that hovers under the offending line instead of an inline thing. Someone else mentioned a delay, which could be good, and it would also be nice if the tooltip line-wrapped, so if it's a really long error, you don't have to scroll horizontally.

distinctdan · 2017-01-21T19:49:53Z

Hey, just wanted to say thanks for fixing this. The new error tooltips are great.

madneon · 2017-06-17T12:39:25Z

What if I can't find TypeScript package and I have this annoying phantom popups? How to disable them?

DanielRosenwasser · 2017-10-07T04:36:09Z

@madneon Just ensure you upgrade using package control. You shouldn't get this behavior anymore.

mafflin changed the title ~~Remove stupid hints in text area!!!!~~ Remove popups and hints in text area. Nov 1, 2016

lorem--ipsum mentioned this issue Nov 2, 2016

[Microsoft/inlineError] Extremely annoying, option to disable? #552

Closed

p-mcgowan mentioned this issue Nov 14, 2016

Annoying error bars after build in code #544

Closed

DanielRosenwasser closed this as completed Oct 7, 2017

DanielRosenwasser added the Fixed label Oct 7, 2017

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Remove popups and hints in text area. #556

Remove popups and hints in text area. #556

mafflin commented Oct 28, 2016 •

edited

bjnsn commented Nov 1, 2016 •

edited

ghost commented Nov 1, 2016

bypotatoes commented Nov 1, 2016

lorem--ipsum commented Nov 2, 2016

xaralis commented Nov 2, 2016

roblav96 commented Nov 2, 2016

plwalters commented Nov 6, 2016

bjnsn commented Nov 7, 2016

plwalters commented Nov 8, 2016

cdeutsch commented Nov 11, 2016

KirillGrishin commented Nov 13, 2016

p-mcgowan commented Nov 14, 2016 •

edited

caiogondim commented Nov 23, 2016

p-mcgowan commented Nov 23, 2016

drdanmorris commented Nov 23, 2016

speedy250 commented Nov 30, 2016

distinctdan commented Dec 3, 2016

plwalters commented Dec 3, 2016

abenhamdine commented Dec 12, 2016

distinctdan commented Dec 18, 2016

distinctdan commented Jan 21, 2017

madneon commented Jun 17, 2017

DanielRosenwasser commented Oct 7, 2017

Remove popups and hints in text area. #556

Remove popups and hints in text area. #556

Comments

mafflin commented Oct 28, 2016 • edited

bjnsn commented Nov 1, 2016 • edited

ghost commented Nov 1, 2016

bypotatoes commented Nov 1, 2016

lorem--ipsum commented Nov 2, 2016

xaralis commented Nov 2, 2016

roblav96 commented Nov 2, 2016

plwalters commented Nov 6, 2016

bjnsn commented Nov 7, 2016

plwalters commented Nov 8, 2016

cdeutsch commented Nov 11, 2016

KirillGrishin commented Nov 13, 2016

p-mcgowan commented Nov 14, 2016 • edited

caiogondim commented Nov 23, 2016

p-mcgowan commented Nov 23, 2016

drdanmorris commented Nov 23, 2016

speedy250 commented Nov 30, 2016

distinctdan commented Dec 3, 2016

plwalters commented Dec 3, 2016

abenhamdine commented Dec 12, 2016

distinctdan commented Dec 18, 2016

distinctdan commented Jan 21, 2017

madneon commented Jun 17, 2017

DanielRosenwasser commented Oct 7, 2017

mafflin commented Oct 28, 2016 •

edited

bjnsn commented Nov 1, 2016 •

edited

p-mcgowan commented Nov 14, 2016 •

edited