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.

Sign up for GitHub

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

Jump to bottom

Show error information when hovering over line #862

Open
Kyu opened this issue May 25, 2020 · 0 comments · May be fixed by #864
Open

Show error information when hovering over line #862

Kyu opened this issue May 25, 2020 · 0 comments · May be fixed by #864

Comments

@Kyu
Copy link

Kyu commented May 25, 2020 •
edited
Loading

Wanted Behaviour

An option to allow tooltips to display error information on hover. e.g like it does for function definitions

image

Also referenced in #688 #752

Actual Behaviour

I either have to enable phantoms, or click the offending line and check the status bar for error info.

Steps to Reproduce

Enter syntax incorrect python code into the window with "anaconda_linter_phantoms" set to true. (side note, phantoms don't disappear when you set them to false, I have to restart sublime)

image

ST3, Anaconda and OS versions

3.2.2 Build 3211. Anaconda 2.1.36, Windows 10 64-bit

ST3 Console Logs

Anaconda's JsonServer Logs

Note: Anaconda's JsonServer logs can be found in:

  • Linux: ~/.local/share/anaconda/logs
  • OS X: ~/Library/Logs/anaconda
  • Windows: %APPDATA\Anaconda\Logs

Making this issue because the Readme said to make an issue before making a PR. I'd also like to work on a PR for this issue.
Kyu added a commit to Kyu/anaconda that referenced this issue May 27, 2020
@Kyu
Adds linting messages on line hover to anaconda. Resolves DamnWidget#862
1f28d53 


	* Added on_hover listener to show a popup if a line contains a linting messages
	* Added "anaconda_linter_hover_message" to Anaconda.sublime-settings. Defaults to false.
@Kyu Kyu linked a pull request May 27, 2020 that will close this issue
Open
Kyu added a commit to Kyu/anaconda that referenced this issue May 28, 2020
@Kyu
Uses css to display linting messages on hover for DamnWidget#862
d4f717e 
    * Created new files: templates/tooltops/error_warning.tpl and templates/tooltips/error_warning_helper.tpl
    * Created new Class TooltipHelper in anaconda_lib/tooltips.py
    * Created new function get_specific_lineo_msgs in anaconda_lib/linting/sublime.py
    * Modified css/popup.css to add linting message color codes.
    * Cleaned up listeners/linting.py, added css viewing using tooltips.py
Kyu added a commit to Kyu/anaconda that referenced this issue May 28, 2020
@Kyu
Uses css to display linting messages on hover for DamnWidget#862
67ae14d 
    * Created new files: templates/tooltops/error_warning.tpl and templates/tooltips/error_warning_helper.tpl
    * Created new Class TooltipHelper in anaconda_lib/tooltips.py
    * Created new function get_specific_lineo_msgs in anaconda_lib/linting/sublime.py
    * Modified css/popup.css to add linting message color codes.
    * Cleaned up listeners/linting.py, added css viewing using tooltips.py
Kyu added a commit to Kyu/anaconda that referenced this issue May 28, 2020
@Kyu
Uses css to display linting messages on hover for DamnWidget#862
e31b2fa 
    * Created new files: templates/tooltops/error_warning.tpl and templates/tooltips/error_warning_helper.tpl
    * Created new Class TooltipHelper in anaconda_lib/tooltips.py
    * Created new function get_specific_lineo_msgs in anaconda_lib/linting/sublime.py
    * Modified css/popup.css to add linting message color codes.
    * Modified Tooltip.show_tooltip in anaconda_lib/tooltips.py to add optional **kwargs for popup
    * Cleaned up listeners/linting.py, added css viewing using tooltips.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Adds linting messages on line hover to anaconda. Resolves #862 Kyu/anaconda
1 participant
@Kyu