-
Notifications
You must be signed in to change notification settings - Fork 578
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
Search in terminal #1533
Search in terminal #1533
Conversation
Great ! I agree the search box is not ideal, subpanel a-la iterm2 or Firefox is much better ! Can’t wait to test it ! Thanks for your effort, I really appreciate :) |
User can search their terminal by search box. Search is based on vte. Default accel-key is CTRL+SHIFT-F, it will then show up a search box at right bottom of the page. User can search by hitting Return to search next, or hitting Return with Shift to search previous, or using next/prev button to search. ESC or anyother focus-out of search entry will hide the search box. NOTE: it will only search on last focus terminal if the terminal has been split out.
If we don't block notebook button-press-event, it will affect search box, when double click on search box button or search entry, it will then add a new page with focus, which is what we don't want. So, block this event when search box is visible, and unblock it when search box is show off.
853d38c
to
f8de098
Compare
@gsemet I think it is good for merge now, please take a test on it :) |
yep! i'm testing it now! |
@gsemet Hi, I'm able to see the search box with Ctrl+shift+F, but it doesn't seem to actually work for me. It doesn't highlight or jump to any text. I tried a few different themes in case the color was hard to see. |
Same as @austinmw for me on Ubuntu 20.04, Gnome desktop.
|
Same for me, I have the search box but nothing gets searched ^^" |
Any opened issue or pull request regarding the unusable search box? One can type inside but nothing gets highlighted or done once they type enter, or "next" or "previous" arrow result. |
Hello. Feel free to work on it and provide this pull request ! |
For #116, currently we have two ways to implement it, one is the search dialog (we have search.glade), another is like gedit or iterm using a small search box to search the terminal.
This PR trying to implement the latter, this is how it work to float the search box on the page:
add
into overlayadd_overlay
into overlayIn this way, we can then implement the search function base on
vte.Terminal
.Some problem:
VTE problem:
TODO:
Feel free to test it, you can test it with
CTRL+SHIFT+F
to search