search

MiguelCastillo / Brackets-InteractiveLinter

Interactive linting for Brackets
Other
119 stars 26 forks source link

Interactive Linter says All is Ok in status bar when it isn't #61

Closed JeffryBooher closed 10 years ago

JeffryBooher commented 10 years ago

Open a javascript file add some code introduce some lint ==> status bar says "OK" yet there are lightbulbs all of the place saying it's not alright.

If you are working on a really big file and there is 1 or 2 pieces of lint then you may overlook them plus the false-positive is a bad experience since it leads the user to believe their code is lint-free when it isn't until they open the console and see an RTE due to lint.

Mark-Simulacrum commented 10 years ago

Unfortunately, Brackets does not currently expose CodeInspection#updatePanelTitleAndStatusBar, so this is impossible (might be possible through DOM edits) to implement in InteractiveLinter.

Mark-Simulacrum commented 10 years ago

One thing Interactive Linter could do is add it's own indicator, though. Or replace the default indicator with our own. What do you think?

JeffryBooher commented 10 years ago

we have hide / showIndicator. Since the built in linter doesn't run it won't try to show the default indicator so showing your own and hiding the built in indicator is the way to go

Mark-Simulacrum commented 10 years ago

I've implemented a lint indicator for this issue in #62.

Mark-Simulacrum commented 10 years ago

Fixed by #62.