krasa / GrepConsole

IntelliJ plugin - https://plugins.jetbrains.com/plugin/7125
Apache License 2.0
440 stars 57 forks source link

Option to Not Grep Files (Console Only) #81

Closed Mochnant closed 7 years ago

Mochnant commented 7 years ago

It doesn't seem like there is a way to disable Grep Console for open files in the editor, while allowing it to modify the console. Is this possible, or might such a flag be added? For those of us with no need to grep files it would seem like a performance improvement.

krasa commented 7 years ago

Why would you need to disable it for the editor, when it is not doing anything to the editor until you click on "Grep Highlight Editor" action.

Mochnant commented 7 years ago

Is there any way to edit the options and search patterns without using "Grep Highlight Editor"? It is also not clear that it is doing nothing at all. I'll take your word if you say it is not impacting performance in any way, of course, but from a user perspective it is non-obvious.

EDIT: To add a bit more background on settings, it appears to change them, you need to use "Grep Highlight Editor", make your changes, say OK which saves the changes AND causes your editor window to be highlighted, then you need to use "Clear Grep Highlights in Editor" to return it to "normal". Perhaps the fix is to make a separate settings menu?

krasa commented 7 years ago

What are you trying to do? Change settings for the console?

Then you have 2 ways:

Mochnant commented 7 years ago

Okay, I see. So the Tools | Grep Highlight Editor windows that opens is the same settings windows that the icon next to the console accesses (the changes are reflected in both) but the OK button on Grep Highlight Editor also processes the editor window.

Thanks for the screenshot. I had not noticed that option. As for what I was trying to do, I was just trying to be sure that Grep Console wasn't scanning editor windows constantly that I had no interest in grepping.

I think this is cleared up now, thank you for the explanations.

Mochnant commented 7 years ago

Closing this issue as resolved.