search

molsonkiko / JsonToolsNppPlugin

A Notepad++ plugin providing tools for JSON like linting, querying, a tree view, and CSV conversion.
Apache License 2.0
93 stars 10 forks source link

[Check JSON syntax now] clear (Syntax errors in JSON) pane upon no errors found #80

Open emanueol opened 2 months ago

emanueol commented 2 months ago

Example:

  1. have some json error(s) on the editor
  2. execute option [Check JSON syntax now]
  3. ok: bottom popup "Syntax errors in JSON" shows errors.
  4. fix errors on editor
  5. execute again option [Check JSON syntax now]
  6. nok: bottom popup "Syntax errors in JSON" still showing previous errors.

Would be nice to at least clear "Syntax errors in JSON" pane. Check JSON syntax now clear (Syntax errors in JSON) pane upon no errors found

and/or add success No errors found (in beautiful green color) :)

molsonkiko commented 1 month ago

Thanks for reporting this issue, @emanueol !

This issue should be addressed by this commit. You can follow steps 5-8 of my instructions for downloading unreleased versions to test the commit. If that commit fixes your issue, you can keep using the new version of JsonTools.dll.

molsonkiko commented 2 weeks ago

JsonTools version 8.2, incorporating a fix for this issue, is now live. I will soon submit a PR to include v8.2 in the plugins manager.

emanueol commented 2 weeks ago

JsonTools version 8.2, incorporating a fix for this issue, is now live. I will soon submit a PR to include v8.2 in the plugins manager.

  1. Notepad++ > Plugins Admin > showed me a version 8.1 of JsonTools (so seems you 8.2 still not available automatically).
  2. So then I wanted to check status of Plugins Admin, googled for "notepad++ plugin manager" but found a repo that seems discontinued [https://github.com/bruderstein/nppPluginManager] "(..)(ATTENTION: This plugin is not actively supported since 2019. The webservice to host the plugin list was stopped. Please use the built-in solution of N++.(..)". image

So I suppose, when you say "(..) I will soon submit a PR to include v8.2 in the plugins manager.(..)" what does that mean ? What will bring your 8.2 available automatically to me, is this the right sequence:

JsonTools repo: https://github.com/molsonkiko/JsonToolsNppPlugin plugins manager repo: ???? Notepad++ repo: I guess its this one ? https://github.com/notepad-plus-plus/notepad-plus-plus ?

Thanks

molsonkiko commented 2 weeks ago

@emanueol

Sorry if my previous post wasn't clear. In the image you showed above, the Plugins Admin for Notepad++ v8.7 lists JsonTools v8.1 as the most recent version because that was the last version I had told the nppPluginList GitHub repo about when Notepad++ v8.7 was released. The Plugins Admin has no way to check what is the most recent version of a plugin; it lists a static snapshot of the most recent versions that the plugin maintainer told them about.

I just told the nppPluginList about JsonTools v8.2, but Notepad++ v8.7.1 has already released, which means that you won't be able to download JsonTools v8.2 via the Plugins Admin until Notepad++ v8.7.2 at the earliest.

However, it is relatively easy to update JsonTools to v8.2 by following these steps:

  1. Download the zip archive of JsonTools v8.2 from the release page
  2. Extract the folder in that archive and rename it to JsonTools.
  3. Replace the contents of the JsonTools plugin folder for your Notepad++ installation with the contents of the downloaded folder.

By following those steps, you're just doing what Notepad++ does when you use the Plugins Admin, except manually rather than through a GUI.

EDIT: If you're still confused, read the user manual section on installing plugins.