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

major updates to syntax checking: multiple errors supported #83

Conversation

jasonwilliams
Copy link
Member

@jasonwilliams jasonwilliams commented Jun 5, 2016

  • A new filename so its not so ambigous
  • Fixed errors on textCommand (now the plugin checks before trying to display a popup)
  • Now using the new sublime popup API (https://forum.sublimetext.com/t/dev-build-3070/14538)
  • The error code is now a link to the correct page on the rust website
  • Now supports multiple errors!
  • Now highlights the erroneous text causing the error, as well as displaying a dot in the gutter
  • tidied up code
  • A check which requires sublime >= 3113 to prevent crashes on Sublime Text 2

As you can see getting the tooltip to appear is a bit tricky, im trying to see if Sublime has a better option
Would be nice to have some beta testers for this

@jasonwilliams
Copy link
Member Author

testingpr4

@VelocityRa
Copy link

VelocityRa commented Jul 27, 2016

Nice! Should make a new project, the maintainer of this one doesn't seem to merge an PRs...
Guess people can just use "Package Control: Add Repository" to use your fork.

@jasonwilliams
Copy link
Member Author

I will close this PR, as the improved work was done in:
#87

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

Successfully merging this pull request may close these issues.

2 participants