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

Added ctrl+u to clear buffer. #275

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Added ctrl+u to clear buffer. #275

wants to merge 1 commit into from

Conversation

weabot
Copy link

@weabot weabot commented Mar 5, 2017

The client was unusable for me without the ctrl+u hotkey which clears the buffer from GNU Readline (I believe). So I changed that.

It's a very simple change that should make the line editing more intuitive to users used to GNU Readline.

@rikai
Copy link

rikai commented Mar 5, 2017

Firstly, this is a wonderful update.

However, I think may cause some confusion for many users as CTRL+U is also often used for applying text formatting (usually underline, which Quassel has support for, though it is also currently missing the ability to be binded to a key) in input fields in many text input fields in GUIs. Running it by several other users, also almost unanimously revealed the same expectation from them.

So, to rectify this, my proposal for a change you could make would be to instead add a keybind for this functionality, allowing it to be user-assignable to whatever a user prefers with a default of no binding assigned.

Do you foresee any issues with such a solution or is that good for you as well? :)

@weabot
Copy link
Author

weabot commented Mar 5, 2017

That also sounds great. Maybe there could be a way (which could be keybound to prevent the user from always having to go in their settings) to easily switch between readline-like line editing and word processor (libreoffice)-like text editing. The current mode would be saved in the config file so that the user can have what they expect when starting up the client again. That would basically tell bufferwidget which set of hotkeys to respond to.

For now I'll try to make this a qt keybind as you said.

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