-
Notifications
You must be signed in to change notification settings - Fork 4
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
Context around line #42
Comments
Hello @gerrywd, thanks for your comment! I've never thought about a feature like that. Let me see if I can find the time to add something like it. |
🤞 Thank you for considering this. 😄 My current use case in on |
Thanks for sharing this! I've never used Can you share an example of the command you use to filter through |
So for instance You could do that by searching the man page or using grep, but it doesn't have the fuzzy find and instant feedback that scout has which makes finding things much quicker. |
The same goes for git. You can |
Maybe if hitting [tab] once you are on a line that could then show the context for the selected line? Perhaps that would be the best way? |
That could be a good approach. I was thinking that too much context lines could clutter the results, but at the same time you want to quickly see what you are looking for. I'll need to think about this. Thanks for all the feedback and suggestions, I appreciate it. |
No worries at all. I agree regarding your point about the clutter, which is why maybe the key to activate might be better. An alternative that I've seen is to have a separate pane to the right, but I'm not sure as that might be a lot of effort. So whatever is the easiest for you would be my personal preference. Anyway, thanks for even considering the idea and making Scout... (edit: and Github 😁 ) |
Yes, the right pane would be great, but printing to the I hope I can find time soon to work on this. |
I know the feeling 😁 |
So, I've been working a bit on this and I realized that the UI part is more complex than I want to admit. I think I'm going to do a rework on it so hopefully displaying more information, like the context, becomes easier. Just wanted to share an update. |
Oh yikes! Thank you for your efforts! I'm sorry to hear it has been tricky. UI in the terminal is very difficult unless you are using some sort of framework and even then I'm not sure. I remember writing my own simple UI many many years ago using ncurses in C and it was brutal. Just in case you haven't seen this one: Although I'm not sure how nicely this would play with your existing code. Perhaps that's some of the issues you are running into? |
Yeah, I know |
Sorry to hear that. If its not worth it please don't feel in any way pressured. I'm just as happy for you to close the issue. I was only suggesting this if it wasn't going to be a lot of work. |
@gerrywd I'm very sorry about the radio silence. I was trying to put up together a new version with this feature, but I couldn't find the time. I'll keep working on it but it probably will take long. I'm going to close this issue but I'll ping you if I manage to release the new version. Thanks again for the feature request! 🙇🏻 |
@jhbabon no worries at all. It was only a suggestion and I can understand not having the time. Only keep working on it if it's something YOU really want. Live is too short to waste on random feature requests. Thank you for the honest response and thank you for the efforts you have put into this project. 👍 Stay awesome. 😃 |
This is a fantastic tool. It would be amazing to be able to specify a number of context lines to be displayed surrounding a match, such as with the -A -B and -C flags in grep.
The text was updated successfully, but these errors were encountered: