No <alt>[1-9] key bindings for tab switching (as per gnome-terminal)

Bug #272906 reported by Richard Khoury
12
Affects Status Importance Assigned to Milestone
Terminator
Fix Released
Wishlist
Richard Khoury

Bug Description

Sorry guys for my laziness, but I couldn't figure out how to branch and push my own changes using bzr (spent over an hour trying and kept getting strange errors). I figured if I submit the patches here you could deal with them as required.

In any case, these patches will allow you to switch tabs via the <alt>[1-9] key combinations in the same way gnome-terminal and firefox works. This is something I really missed having moved from gnome-terminal to terminator.

Two patches to follow...

Related branches

Revision history for this message
Richard Khoury (richiek) wrote :
Revision history for this message
Richard Khoury (richiek) wrote :

Second patch...

Revision history for this message
Chris Jones (cmsj) wrote :

Thanks for the patch, although I rather think that this should be implemented via the existing keybindings support we have.
I am in two minds about what the defaults should be though - I have always found the alt-1 default of gnome-terminal to be hugely irritating because it clashes with irssi.

Revision history for this message
Richard Khoury (richiek) wrote :

If I changed this to be <alt>/<ctr>(/<win-key>?) selectable via the config file (or even switched off altogether) would that be considered more useful?

Revision history for this message
Richard Khoury (richiek) wrote :

Check it the fix attached and update this bug-report if any problems exist.

Changed in terminator:
assignee: nobody → richiek
status: New → Fix Committed
Revision history for this message
Chris Jones (cmsj) wrote :

I'm unsetting the Fix Committed status on this, just because we use that to mean committed to trunk. I'll review the merge proposal.

Changed in terminator:
importance: Undecided → Wishlist
status: Fix Committed → Confirmed
Revision history for this message
Richard Khoury (richiek) wrote :

Cool. Be sure to lay any criticisms on me as thick as necessary. I'm only fresh to this project (and haven't done any python programming before) so I'm not used to the protocols/procedures you have in place.

Revision history for this message
Jamu Kakar (jkakar) wrote :

I happened to implement the fix for this before noticing this bug
report or Richard's prior work. I've attached the branch with my
changes in case there's something useful there that can be merged in
with what's already in progress.

Also, for what it's worth, this is a must-have feature for me.
While it isn't available, I'll maintain my own branch to play with
Terminator (I implemented this feature after using it for two
minutes and realized I couldn't use it without Alt-Num). I would
expect a lot of people rely on Alt-Num and would be turned off to
find Terminator lacking this behaviour by default. It's a common
idiom used by Firefox, GNOME Terminal, Gwibber and X-Chat GNOME.

Revision history for this message
Thomas Hurst (tom.hurst) wrote :

I've merged Jamu's branch, with minor changes to the keybinding names (to match gnome-terminal). Since it's bit of a bikeshed issue, they're unbound by default; if we can work out a way to turn <Alt><Shift>1 into <Alt>! or whatever, I'd be open to having those as default.

Changed in terminator:
status: Confirmed → Fix Committed
Revision history for this message
Jamu Kakar (jkakar) wrote :

I've merged trunk, with Thomas's changes, and included more changes
that make it possible to set the accelerator key directly in the
Preference dialog.

Revision history for this message
Richard Khoury (richiek) wrote :

I'm really glad to see this feature finally getting merged into trunk.
Many thanks for all those who helped!

Chris Jones (cmsj)
Changed in terminator:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.