Theia: false positive error tabbar icon

Created on 5 Sep 2019  路  4Comments  路  Source: eclipse-theia/theia

Description

image

Reproduction Steps

  • open a ts file
  • type some error code
  • fix error code
  • save file

OS and Theia version:
Diagnostics:

bug help wanted markers shell

Most helpful comment

@vince-fugnitto I believe you that it was by my changes, but it used to hang the browser too :) anyway we should a fix now, could you have a look? i can do it after migrating Monaco.

@akosyakov not blaming you at all :) I know the original implementation was problematic and worse if it hangs the browser. I'll see if I can debug as to why when I get the chance.

All 4 comments

@502647092 you can disable problem decorators with preferences if it is annoying.

It used to work correctly, I suspect that the error is from the changes in https://github.com/theia-ide/theia/pull/6044/commits/67a04cba04454b3c77399a5483f2544e204b38af

There are a few problems, for instance:

  • tabbars now display information severity decorators which seem noisy.
  • tabbars display information severity decorators at a higher priority than error (when an editor has both errors and infos, the infos are displayed in the decorator)
  • tabbar decorations are not refreshed when the editor is updated/saved.

@vince-fugnitto I believe you that it was by my changes, but it used to hang the browser too :) anyway we should a fix now, could you have a look? i can do it after migrating Monaco.

@vince-fugnitto I believe you that it was by my changes, but it used to hang the browser too :) anyway we should a fix now, could you have a look? i can do it after migrating Monaco.

@akosyakov not blaming you at all :) I know the original implementation was problematic and worse if it hangs the browser. I'll see if I can debug as to why when I get the chance.

Was this page helpful?
0 / 5 - 0 ratings