Skip to content
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

Feature request: Go to line #67

Open
vogelsgesang opened this issue Oct 31, 2017 · 5 comments
Open

Feature request: Go to line #67

vogelsgesang opened this issue Oct 31, 2017 · 5 comments

Comments

@vogelsgesang
Copy link
Contributor

From time to time, I am using simple ad-hoc scripts to search for more complicated patterns in log files. E.g., recently I had to write a small script which gave me a list of all threads which did acquire locks but did not release them again. I keep those scripts very simple, and they usually output only line numbers of interesting events.

It would be perfect if I could enter the obtained line number in a Go to line dialog.
Usually, I would just go ahead and implement that feature. However, Go to line is usually associated with the keyboard combination Ctrl+G - and this one is already used for the highlight dialog.

Hence, I wanted to start a discussion first: Do you consider Go to line useful? Do you think we should reassign Ctrl+G to Go to line and find some other shortcut for Highlight?

@terencesoh
Copy link
Contributor

I would find it useful. In fact, I added that feature in the original C++ TLV and assigned it to Ctrl-G as well.
However the original Ctrl-H for highlight does not translate well to Mac when we moved to Qt, as Cmd-H is used on macOS system-wide to hide windows, so it got the Ctrl/Cmd-G. We may reassign Ctrl/Cmd-T or other keys to Highlight, though that may trip some users who are now used to current keys.

@luiseeo
Copy link
Contributor

luiseeo commented Nov 1, 2017

Terence, does that mean that there's an implementation for Go to line already?
I was never totally happy about using Ctrl+G for the highlight dialog and sometimes I still press Ctrl+H as that one made more sense.
The highlight dialog is probably the one the I use the most, what if we assign it to a function key? F8 or F9 maybe. Not sure if these have a convention attached to them (like with F5 that is normally refresh)...

@terencesoh
Copy link
Contributor

Correction - I added Go to line in the original C# TLV.
We could assign both a Ctrl shortcut key and a function key to the highlight dialog. Just need to make sure they work well on both Windows and Mac. Maybe we can have an internal vote on the preferred keys.

@alv53
Copy link
Contributor

alv53 commented Nov 9, 2017

Also, what do you guys think should happen if someone does the following:

1.) Turns on highlight only mode, which hides line x from view
2.) User attempts to go to line x.

Should we give an error and not go to the line, go to the line but have it hidden, or maybe go to the nearest line?

@vogelsgesang
Copy link
Contributor Author

I think going to the closest line is the most intuitive thing we can do in that case

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants