yhirose / vscode-filtertext

Filter Text extension for VS Code
MIT License
30 stars 9 forks source link

Fix used document filter flag when false #17

Closed karomine closed 5 years ago

yhirose commented 5 years ago

@karomine, could you explain more about your change? What is your intention with it? Thanks.

karomine commented 5 years ago

If you run filter text in place with nothing selected, just to pipe the output into buffer, and have "Use Document If Empty Selection" set to false, nothing happens because the range ends up being undefined. This change sets the range to the current cursor position for that case.

yhirose commented 5 years ago

@karomine, thanks for explanation. I now fully understand what the PR does. The version 0.0.14 includes the fix. Thanks for your contribution!