Hi Boris,

Why this change was needed? People can add these shortcuts easily if they want, 
no?

For your information, it was breaking my vim shortcuts because shift left/right 
are already used there to switch between tabs ... of vim.

Daniel

On Tue, 12 Apr 2016 13:48:47 -0700
Boris Faure <bill...@gmail.com> wrote:

> billiob pushed a commit to branch master.
> 
> http://git.enlightenment.org/apps/terminology.git/commit/?id=15fa7afaa5b6f33acb5889dd4136bf1b6c0faf27
> 
> commit 15fa7afaa5b6f33acb5889dd4136bf1b6c0faf27
> Author: Boris Faure <bill...@gmail.com>
> Date:   Tue Apr 12 22:47:49 2016 +0200
> 
>     key bindings: add shift+left/right to switch tabs
> ---
>  README            |  2 ++
>  man/terminology.1 | 18 +++++++++++++++---
>  src/bin/config.c  | 10 ++++++++--
>  3 files changed, 25 insertions(+), 5 deletions(-)
> 
> diff --git a/README b/README
> index 3dcd486..f4fc574 100644
> --- a/README
> +++ b/README
> @@ -40,6 +40,8 @@ Shift+PgUp = Scroll 1 page up
>  Shift+PgDn = Scroll 1 page down
>  Shift+Up = Scroll 1 line up
>  Shift+Down = Scroll 1 line down
> +Shift+Left = switch focus to previous terminal inside a window
> +Shift+Right = switch focus to next terminal inside a window
>  Shift+Insert = Paste Clipboard (ctrl+v/c) selection
>  Shift+Ctrl+Insert = Paste Primary (highlight) selection
>  Shift+Keypad-Plus = Font size up 1
> diff --git a/man/terminology.1 b/man/terminology.1
> index 89933f9..5dffb29 100644
> --- a/man/terminology.1
> +++ b/man/terminology.1
> @@ -1,5 +1,5 @@
>  .\" Manpage for Terminology
> -.TH man 1 "12 March 2016" "0.9.1" "Terminology man page"
> +.TH man 1 "12 April 2016" "0.9.1" "Terminology man page"
>  .SH NAME
>  Terminology \- Terminal Emulator written with EFL (Enlightenment
> Foundation Libraries). .SH SYNOPSIS
> @@ -196,6 +196,16 @@ Scroll terminology one line up
>  Scroll terminology one line down
>  .
>  .TP
> +.B Shift+Left
> +Switch focus to previous terminal inside a window when using splits,
> or the +previous tab.
> +.
> +.TP
> +.B Shift+Right
> +Switch focus to next terminal inside a window when using splits, or
> the next +tab.
> +.
> +.TP
>  .B Shift+Insert
>  Paste Clipboard (ctrl+v/c) selection
>  .
> @@ -221,11 +231,13 @@ Copy highlight to Clipboard (same as ctrl+c in
> gui apps). .
>  .TP
>  .B Ctrl+PgUp
> -Switch focus to previous terminal inside a window (when using
> splits). +Switch focus to previous terminal inside a window when
> using splits, or the +previous tab.
>  .
>  .TP
>  .B Ctrl+PgDn
> -Switch focus to next terminal inside a window (when using splits).
> +Switch focus to next terminal inside a window when using splits, or
> the next +tab.
>  .
>  .TP
>  .B Ctrl+Shift+h
> diff --git a/src/bin/config.c b/src/bin/config.c
> index 1385751..d2c3894 100644
> --- a/src/bin/config.c
> +++ b/src/bin/config.c
> @@ -7,7 +7,7 @@
>  #include "col.h"
>  #include "utils.h"
>  
> -#define CONF_VER 11
> +#define CONF_VER 12
>  
>  #define LIM(v, min, max) {if (v >= max) v = max; else if (v <= min)
> v = min;} 
> @@ -362,6 +362,8 @@ _add_default_keys(Config *config)
>     ADD_KB("KP_Subtract", 0, 0, 1, 0, "decrease_font_size");
>     ADD_KB("KP_Multiply", 0, 0, 1, 0, "reset_font_size");
>     ADD_KB("KP_Divide", 0, 0, 1, 0, "copy_clipboard");
> +   ADD_KB("Left", 0, 0, 1, 0, "term_prev");
> +   ADD_KB("Right", 0, 0, 1, 0, "term_next");
>  }
>  
>  void
> @@ -597,7 +599,11 @@ config_load(const char *key)
>                  case 10:
>                    config->font.bolditalic = EINA_TRUE;
>                    /*pass through*/
> -                case CONF_VER: /* 11 */
> +                case 11:
> +                  _add_key(config, "Left", 0, 0, 1, 0, "term_prev");
> +                  _add_key(config, "Right", 0, 0, 1, 0, "term_next");
> +                  /*pass through*/
> +                case CONF_VER: /* 12 */
>                    config->version = CONF_VER;
>                    break;
>                  default:
> 


------------------------------------------------------------------------------
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to