Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Falsely indicating 2 issues in parentheses though none exist #9

Closed
brettz9 opened this issue Mar 24, 2016 · 9 comments
Closed

Falsely indicating 2 issues in parentheses though none exist #9

brettz9 opened this issue Mar 24, 2016 · 9 comments

Comments

@brettz9
Copy link

brettz9 commented Mar 24, 2016

Though I have no current linting errors, bottom dock shows up in red with "2" listed in parentheses and when clicked, shows no issues within itself either.

@benjaminRomano
Copy link
Owner

Hey brettz9 could you post a screenshot of the bug. Also which linters were you using

@brettz9
Copy link
Author

brettz9 commented Mar 24, 2016

bd1
bd2

As per the screenshots, the linters are todo-manager and linter-eslint. linter and linter-manager are also installed as are linter-csslint, linter-htmlhint, linter-jsonlint, linter-markdown, linter-package-json-validator, linter-xmllint.

@benjaminRomano
Copy link
Owner

The 2 in the bottom right indicates that you currently have two panes open in bottom-dock (TODO and linter) and the red eye indicates bottom dock isn't currently visible

@brettz9
Copy link
Author

brettz9 commented Mar 24, 2016

Hmm, oops. :) Ok, well, could I request then that the color be changed from red because that is a color I very much notice and sets off alarm bells.

Also, even the parentheses trigger the idea of problems for me, and I'm not sure what benefit indicating them has (unless it is to be a reminder that the number available may change with each document type).

@benjaminRomano
Copy link
Owner

Yeah that's understandable. I'll change the color scheme and remove the parentheses (not really sure why I included them in the first place 😄)

@brettz9
Copy link
Author

brettz9 commented Mar 28, 2016

Thanks! As far as why you included them, I can see one scenario where having a number in parentheses (and color) there would be useful, and that is if one has disabled "Display Linter Info in the Status Bar" and/or "Display Linter Status Info in the Status Bar" within the base "linter", one might wish to opt to show the error status and number of errors there instead. Not a pressing need for me at least though.

Btw, speaking of colors, is it possible to opt for them within the expanded Bottom Dock table, e.g., how "Error" shows up in red in the default linter display?

@brettz9
Copy link
Author

brettz9 commented Mar 28, 2016

Btw, I updated to your latest version and the color is gone, but the parentheses (and the option for them) are still present.

One weird thing too I just noticed is that when I click on the "Report issue" or "CHANGELOG", etc. with the settings page for your package, it repeats the action two times, e.g., opens two report issue tabs in the browser. This didn't happen with the settings for another package I tested which strikes me as weird, as I'd think this would be controlled by Atom and not sure why it'd happen for your package specifically.

@benjaminRomano
Copy link
Owner

So the parentheses are still showing in the status bar? If they are you might want to re-install the bottom-dock package since they shouldn't be there anymore.

Can you file a new issue for the color highlighting of errors. I'll try to get around to that when I have a bit more time.

Lastly, I think the two report issue tabs being opened is probably just some weird Atom bug.

@brettz9
Copy link
Author

brettz9 commented Mar 29, 2016

No, the parentheses now aren't showing anymore. Ok, I can file a new issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants