{"thread":{"id":"12973","subject":"[REGRESSION] git-gui","startedAt":"2008-04-03T13:43:08Z","lastAt":"2008-04-08T02:18:07Z","messageCount":14,"participants":["André Goddard Rosa","Jonathan del Strother","Michele Ballabio","Shawn O. Pearce","Marco Roeland","David Kågedal","Teemu Likonen"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"73600","messageId":"007901c89590$a827f7c0$93a7c10a@LGE.NET","threadId":"12973","inReplyTo":null,"subject":"[REGRESSION] git-gui","fromName":"André Goddard Rosa","fromEmail":"andre_rosa@lge.com","sentAt":"2008-04-03T13:43:08Z","receivedAt":"2008-04-03T13:43:08Z","isPatch":false,"sender":{"key":"andre_rosa@lge.com","avatar":null},"body":"Hi, all!\n \n    After updating to git version 1.5.5.rc3 I always get the error below\nwhen starting \"git gui\".\n    Linux distributions used are opensuse 10 and opensuse Factory.\n \n# git gui\nError in startup script: bad event type or keysym \"[\"\n    while executing\n\"bind $ui_comm <$M1B-Key-\\[> {show_less_context;break}\"\n    file \"/usr/bin/git-gui\" linux 2718)\n\n    I think it's related to git-gui 0.10 recently added into the tree.\n \nThanks a lot,\nAndré\n"},{"id":"73603","messageId":"57518fd10804030700r6de977f5p6f0418e6eaea2583@mail.gmail.com","threadId":"12973","inReplyTo":"007901c89590$a827f7c0$93a7c10a@LGE.NET","subject":"Re: [REGRESSION] git-gui","fromName":"Jonathan del Strother","fromEmail":"maillist@steelskies.com","sentAt":"2008-04-03T14:00:47Z","receivedAt":"2008-04-03T14:00:47Z","isPatch":false,"sender":{"key":"jon.delstrother@bestbefore.tv","avatar":"https://gravatar.com/avatar/754e21ab701c00e2d21fc261187254c34b2a1c0b959d9ee5be1a295990be3081?d=mp&s=160"},"body":"On Thu, Apr 3, 2008 at 2:43 PM, André Goddard Rosa <andre_rosa@lge.com> wrote:\n> Hi, all!\n>\n>     After updating to git version 1.5.5.rc3 I always get the error below\n>  when starting \"git gui\".\n>     Linux distributions used are opensuse 10 and opensuse Factory.\n>\n>  # git gui\n>  Error in startup script: bad event type or keysym \"[\"\n>     while executing\n>  \"bind $ui_comm <$M1B-Key-\\[> {show_less_context;break}\"\n>     file \"/usr/bin/git-gui\" linux 2718)\n>\n>     I think it's related to git-gui 0.10 recently added into the tree.\n\nDoh, I broke git :(\n\nThe problem is in c91ee2bd61 (Add shortcut keys for Show More/Less Context).\nWorks fine on my OS X 10.5 installation with tcl 8.4.7 - can anyone\nsuggest an alternative way of binding the left/right brackets?  I'm\nnot familiar with Tcl, and didn't manage to find a list of key names.\n"},{"id":"73607","messageId":"007d01c89597$c6d7f060$93a7c10a@LGE.NET","threadId":"12973","inReplyTo":"57518fd10804030700r6de977f5p6f0418e6eaea2583@mail.gmail.com","subject":"RES: [REGRESSION] git-gui","fromName":"André Goddard Rosa","fromEmail":"andre_rosa@lge.com","sentAt":"2008-04-03T14:34:05Z","receivedAt":"2008-04-03T14:34:05Z","isPatch":false,"sender":{"key":"andre_rosa@lge.com","avatar":null},"body":"Hi,\n\n>     I think it's related to git-gui 0.10 recently added into the tree.\n> Doh, I broke git :(\n> The problem is in c91ee2bd61 (Add shortcut keys for Show More/Less\nContext).\n> Works fine on my OS X 10.5 installation with tcl 8.4.7 - can anyone\nsuggest an alternative way of binding the left/right brackets?  I'm not\nfamiliar with Tcl, and > \n> didn't manage to find a list of key names.\n\nYes, reverting this commit fix the issue.\n\nThank you,\nAndré\n"},{"id":"73628","messageId":"200804032304.17054.barra_cuda@katamail.com","threadId":"12973","inReplyTo":"57518fd10804030700r6de977f5p6f0418e6eaea2583@mail.gmail.com","subject":"Re: [REGRESSION] git-gui","fromName":"Michele Ballabio","fromEmail":"barra_cuda@katamail.com","sentAt":"2008-04-03T21:04:16Z","receivedAt":"2008-04-03T21:04:16Z","isPatch":false,"sender":{"key":"barra_cuda@katamail.com","avatar":"https://avatars.githubusercontent.com/u/16371673?v=4"},"body":"On Thursday 03 April 2008, Jonathan del Strother wrote:\n> On Thu, Apr 3, 2008 at 2:43 PM, André Goddard Rosa <andre_rosa@lge.com> wrote:\n> > Hi, all!\n> >\n> >     After updating to git version 1.5.5.rc3 I always get the error below\n> >  when starting \"git gui\".\n> >     Linux distributions used are opensuse 10 and opensuse Factory.\n> >\n> >  # git gui\n> >  Error in startup script: bad event type or keysym \"[\"\n> >     while executing\n> >  \"bind $ui_comm <$M1B-Key-\\[> {show_less_context;break}\"\n> >     file \"/usr/bin/git-gui\" linux 2718)\n> >\n> >     I think it's related to git-gui 0.10 recently added into the tree.\n> \n> Doh, I broke git :(\n> \n> The problem is in c91ee2bd61 (Add shortcut keys for Show More/Less Context).\n> Works fine on my OS X 10.5 installation with tcl 8.4.7 - can anyone\n> suggest an alternative way of binding the left/right brackets?  I'm\n> not familiar with Tcl, and didn't manage to find a list of key names.\n\nThese changes should help (haven't tried):\n\t\\[ -> bracketleft\n\t\\] -> bracketright\nbut some European keyboards do not have easy access to brackets, so\nother keys would be preferable (comma and period, for example, or 1 and 2).\n\nSee\n\tman keysyms\nfor a list of key names.\n"},{"id":"73635","messageId":"57518fd10804031615m5c299df6hf31de3c689e16521@mail.gmail.com","threadId":"12973","inReplyTo":"200804032304.17054.barra_cuda@katamail.com","subject":"Re: [REGRESSION] git-gui","fromName":"Jonathan del Strother","fromEmail":"maillist@steelskies.com","sentAt":"2008-04-03T23:15:39Z","receivedAt":"2008-04-03T23:15:39Z","isPatch":false,"sender":{"key":"jon.delstrother@bestbefore.tv","avatar":"https://gravatar.com/avatar/754e21ab701c00e2d21fc261187254c34b2a1c0b959d9ee5be1a295990be3081?d=mp&s=160"},"body":"On Thu, Apr 3, 2008 at 10:04 PM, Michele Ballabio\n<barra_cuda@katamail.com> wrote:\n>\n> On Thursday 03 April 2008, Jonathan del Strother wrote:\n>  > On Thu, Apr 3, 2008 at 2:43 PM, André Goddard Rosa <andre_rosa@lge.com> wrote:\n>  > > Hi, all!\n>  > >\n>  > >     After updating to git version 1.5.5.rc3 I always get the error below\n>  > >  when starting \"git gui\".\n>  > >     Linux distributions used are opensuse 10 and opensuse Factory.\n>  > >\n>  > >  # git gui\n>  > >  Error in startup script: bad event type or keysym \"[\"\n>  > >     while executing\n>  > >  \"bind $ui_comm <$M1B-Key-\\[> {show_less_context;break}\"\n>  > >     file \"/usr/bin/git-gui\" linux 2718)\n>  > >\n>  > >     I think it's related to git-gui 0.10 recently added into the tree.\n>  >\n>  > Doh, I broke git :(\n>  >\n>  > The problem is in c91ee2bd61 (Add shortcut keys for Show More/Less Context).\n>  > Works fine on my OS X 10.5 installation with tcl 8.4.7 - can anyone\n>  > suggest an alternative way of binding the left/right brackets?  I'm\n>  > not familiar with Tcl, and didn't manage to find a list of key names.\n>\n>  These changes should help (haven't tried):\n>         \\[ -> bracketleft\n>         \\] -> bracketright\n>  but some European keyboards do not have easy access to brackets, so\n>  other keys would be preferable (comma and period, for example, or 1 and 2).\n>\n>  See\n>         man keysyms\n>  for a list of key names.\n>\n\nAnyone else got any opinions on an appropriate shortcut?  How about\n'=' and '-' (+ and - without the shift), or are those not necessarily\ntogether either?  (Are there really layouts where '[' and ']' aren't\nnext to each other?)\n"},{"id":"73639","messageId":"20080404012723.GL10274@spearce.org","threadId":"12973","inReplyTo":"57518fd10804031615m5c299df6hf31de3c689e16521@mail.gmail.com","subject":"Re: [REGRESSION] git-gui","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2008-04-04T01:27:23Z","receivedAt":"2008-04-04T01:27:23Z","isPatch":false,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"Jonathan del Strother <maillist@steelskies.com> wrote:\n> On Thu, Apr 3, 2008 at 10:04 PM, Michele Ballabio\n> <barra_cuda@katamail.com> wrote:\n> > On Thursday 03 April 2008, Jonathan del Strother wrote:\n> >  > On Thu, Apr 3, 2008 at 2:43 PM, André Goddard Rosa <andre_rosa@lge.com> wrote:\n> >  > >\n> >  > >  # git gui\n> >  > >  Error in startup script: bad event type or keysym \"[\"\n> >  > >     while executing\n> >  > >  \"bind $ui_comm <$M1B-Key-\\[> {show_less_context;break}\"\n> >  >\n> >  > Doh, I broke git :(\n\nYou and me both Jonathan.  I tested it on both Mac OS X and Win32,\nand on Tcl/Tk 8.4.1, 8.4.10, 8.4.15, and 8.5.0.  Never ran into\nthis failure.  So the original poster must be running 8.4.0, and\n8.4.0 must not support this binding.  Added in 8.4.1?  *sigh*\n\n> >  These changes should help (haven't tried):\n> >         \\[ -> bracketleft\n> >         \\] -> bracketright\n> >  but some European keyboards do not have easy access to brackets, so\n> >  other keys would be preferable (comma and period, for example, or 1 and 2).\n> \n> Anyone else got any opinions on an appropriate shortcut?  How about\n> '=' and '-' (+ and - without the shift), or are those not necessarily\n> together either?  (Are there really layouts where '[' and ']' aren't\n> next to each other?)\n\nI don't know, [ and ] seem reasonable to me.  Git is mostly a source\ncode control system.  A lot of languages use [ and ] as part of\ntheir syntax.  If you are typing on a keyboard that is difficult to\naccess these keys on, you are probably already used to contorting\nyour fingers.  :-|\n\nI wrote up a patch today based on Michele's suggested change.\nIt works everywhere I can test, but I don't have an 8.4.0\ninstallation like the original poster.\n\nUnless someone posts a patch to change the keys away from [ and ]\nI say leave them as-is.  But I'm willing to entertain a change if\nsomeone who cares writes a patch for it.\n\n-- \nShawn.\n"},{"id":"73640","messageId":"20080404013356.GM10274@spearce.org","threadId":"12973","inReplyTo":"20080404012723.GL10274@spearce.org","subject":"Re: [REGRESSION] git-gui","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2008-04-04T01:33:56Z","receivedAt":"2008-04-04T01:33:56Z","isPatch":false,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"\"Shawn O. Pearce\" <spearce@spearce.org> wrote:\n> Jonathan del Strother <maillist@steelskies.com> wrote:\n> > >  > >  # git gui\n> > >  > >  Error in startup script: bad event type or keysym \"[\"\n> > >  > >     while executing\n> > >  > >  \"bind $ui_comm <$M1B-Key-\\[> {show_less_context;break}\"\n> \n> > >  These changes should help (haven't tried):\n> > >         \\[ -> bracketleft\n> > >         \\] -> bracketright\n> > >  but some European keyboards do not have easy access to brackets, so\n> > >  other keys would be preferable (comma and period, for example, or 1 and 2).\n> \n> I wrote up a patch today based on Michele's suggested change.\n\nHmm, maybe I should offer the patch.  :-)\n\n--8>--\nSubject: [PATCH] git-gui: Switch keybindings for [ and ] to bracketleft and bracketright\n\nThanks to Michele Ballabio for the quick fix.\nThis resolves the error introduced by c91ee2bd61.\n\nSigned-off-by: Shawn O. Pearce <spearce@spearce.org>\n---\n git-gui.sh |    8 ++++----\n 1 files changed, 4 insertions(+), 4 deletions(-)\n\ndiff --git a/git-gui.sh b/git-gui.sh\nindex d4e4c17..1a67105 100755\n--- a/git-gui.sh\n+++ b/git-gui.sh\n@@ -2739,8 +2739,8 @@ bind $ui_comm <$M1B-Key-v> {tk_textPaste %W; %W see insert; break}\n bind $ui_comm <$M1B-Key-V> {tk_textPaste %W; %W see insert; break}\n bind $ui_comm <$M1B-Key-a> {%W tag add sel 0.0 end;break}\n bind $ui_comm <$M1B-Key-A> {%W tag add sel 0.0 end;break}\n-bind $ui_comm <$M1B-Key-\\[> {show_less_context;break}\n-bind $ui_comm <$M1B-Key-\\]> {show_more_context;break}\n+bind $ui_comm <$M1B-Key-bracketleft> {show_less_context;break}\n+bind $ui_comm <$M1B-Key-bracketright> {show_more_context;break}\n \n bind $ui_diff <$M1B-Key-x> {tk_textCopy %W;break}\n bind $ui_diff <$M1B-Key-X> {tk_textCopy %W;break}\n@@ -2784,8 +2784,8 @@ bind .   <$M1B-Key-t> do_add_selection\n bind .   <$M1B-Key-T> do_add_selection\n bind .   <$M1B-Key-i> do_add_all\n bind .   <$M1B-Key-I> do_add_all\n-bind .   <$M1B-Key-\\[> {show_less_context;break}\n-bind .   <$M1B-Key-\\]> {show_more_context;break}\n+bind .   <$M1B-Key-bracketleft> {show_less_context;break}\n+bind .   <$M1B-Key-bracketright> {show_more_context;break}\n bind .   <$M1B-Key-Return> do_commit\n foreach i [list $ui_index $ui_workdir] {\n \tbind $i <Button-1>       \"toggle_or_diff         $i %x %y; break\"\n-- \n1.5.5.rc3.148.gb77a7\n\n\n-- \nShawn.\n"},{"id":"73649","messageId":"20080404110201.GA8298@fiberbit.xs4all.nl","threadId":"12973","inReplyTo":"20080404013356.GM10274@spearce.org","subject":"Re: [REGRESSION] git-gui","fromName":"Marco Roeland","fromEmail":"marco.roeland@xs4all.nl","sentAt":"2008-04-04T11:02:01Z","receivedAt":"2008-04-04T11:02:01Z","isPatch":false,"sender":{"key":"marco.roeland@xs4all.nl","avatar":null},"body":"On Thursday April 3rd 2008 at 21:33 Shawn O. Pearce wrote:\n\n> Hmm, maybe I should offer the patch.  :-)\n> \n> --8>--\n> Subject: [PATCH] git-gui: Switch keybindings for [ and ] to bracketleft and bracketright\n> \n> Thanks to Michele Ballabio for the quick fix.\n> This resolves the error introduced by c91ee2bd61.\n> \n> Signed-off-by: Shawn O. Pearce <spearce@spearce.org>\n\nThis solves the problem on Debian GNU/Linux 'sid', with Tcl/Tk version\n8.5.1. So somehow the Linux implementation seems to be the problem, not\nthe Tcl/Tk version as such. Man keysyms here does list \"bracketleft\" and\n\"bracketright\" but no direct symbols for '[' or ']'. Thanks!\n-- \nMarco Roeland\n"},{"id":"73650","messageId":"00e801c89645$ee77fd50$93a7c10a@LGE.NET","threadId":"12973","inReplyTo":"20080404110201.GA8298@fiberbit.xs4all.nl","subject":"RES: [REGRESSION] git-gui","fromName":"André Goddard Rosa","fromEmail":"andre_rosa@lge.com","sentAt":"2008-04-04T11:20:43Z","receivedAt":"2008-04-04T11:20:43Z","isPatch":false,"sender":{"key":"andre_rosa@lge.com","avatar":null},"body":"> Hmm, maybe I should offer the patch.  :-)\n> \n> --8>--\n> Subject: [PATCH] git-gui: Switch keybindings for [ and ] to \n> bracketleft and bracketright\n> \n> Thanks to Michele Ballabio for the quick fix.\n> This resolves the error introduced by c91ee2bd61.\n> \n> Signed-off-by: Shawn O. Pearce <spearce@spearce.org>\n\n> This solves the problem on Debian GNU/Linux 'sid', with Tcl/Tk version\n8.5.1. So somehow the Linux implementation seems to be the problem, not the\nTcl/Tk version as > such. Man keysyms here does list \"bracketleft\" and\n\"bracketright\" but no direct symbols for '[' or ']'. Thanks!\n\nIt solves the issue indeed. Please apply.\n\nThank you all,\nAndré\n"},{"id":"73668","messageId":"200804042304.42794.barra_cuda@katamail.com","threadId":"12973","inReplyTo":"20080404012723.GL10274@spearce.org","subject":"Re: [REGRESSION] git-gui","fromName":"Michele Ballabio","fromEmail":"barra_cuda@katamail.com","sentAt":"2008-04-04T21:04:42Z","receivedAt":"2008-04-04T21:04:42Z","isPatch":false,"sender":{"key":"barra_cuda@katamail.com","avatar":"https://avatars.githubusercontent.com/u/16371673?v=4"},"body":"On Friday 04 April 2008, Shawn O. Pearce wrote:\n> > Anyone else got any opinions on an appropriate shortcut?  How about\n> > '=' and '-' (+ and - without the shift), or are those not necessarily\n> > together either?  (Are there really layouts where '[' and ']' aren't\n> > next to each other?)\n> \n> I don't know, [ and ] seem reasonable to me.  Git is mostly a source\n> code control system.  A lot of languages use [ and ] as part of\n> their syntax.  If you are typing on a keyboard that is difficult to\n> access these keys on, you are probably already used to contorting\n> your fingers.  :-|\n\nTo clarify, [ and ] are quite easy to type for me: the first problem\nis that to type [ I have to press AltGr-è (or AltGr-8). With Ctrl, it\nmakes three keypresses: I can handle this, but - second problem - it\nseems Tcl/Tk cannot (unless I'm wrong, which can also be very likely):\n\nkeys.tcl:\n-->8--\n#!/usr/bin/wish\nbind . <Key> {\n\tputs \"            %%K - %%k - %%N - %%T - %%D - %%A - %%d\"\n\tputs \"You pressed %K - %k - %N - %T - %D - %A - %d\"\n}\n--8<--\n\nTrying Ctrl+[:\n$ ./keys.tcl \n            %K - %k - %N - %T - %D - %A - %d \nYou pressed Control_R - 109 - 65508 - 2 - 109 - {} - ?? \n            %K - %k - %N - %T - %D - %A - %d \nYou pressed ISO_Level3_Shift - 113 - 65027 - 2 - 113 - {} - ?? \n            %K - %k - %N - %T - %D - %A - %d \nYou pressed ?? - 34 - 0 - 2 - 34 - ?\n\nTrying [:\n$ ./keys.tcl \n            %K - %k - %N - %T - %D - %A - %d \nYou pressed ISO_Level3_Shift - 113 - 65027 - 2 - 113 - {} - ?? \n            %K - %k - %N - %T - %D - %A - %d \nYou pressed ?? - 34 - 0 - 2 - 34 - [ - ?? \n\nSo, it doesn't recognize [ at all at the keysym (%K) level.\nNo idea why. This is Tcl/Tk 8.4.15 on Linux.\n\n> I wrote up a patch today based on Michele's suggested change.\n> It works everywhere I can test, but I don't have an 8.4.0\n> installation like the original poster.\n> \n> Unless someone posts a patch to change the keys away from [ and ]\n> I say leave them as-is.  But I'm willing to entertain a change if\n> someone who cares writes a patch for it.\n\nHere it is. Of course if I'm the only one that has problems with ]/[,\nforget it. But it seems reasonable to me to do:\n\nctrl-+ and ctrl-=\t-> more context\nctrl--\t\t\t-> less context\n\nSince these keybindings are used in gitk too, they should be safe. If\nsomeone doesn't like this change, just say so. If someone thinks my\nsetup is strange and knows what to do to fix it up, let me know.\n\n-->8--\nFrom 4334c7690d80f7c05ec40f2efa4306ef88933469 Mon Sep 17 00:00:00 2001\nFrom: Michele Ballabio <barra_cuda@katamail.com>\nDate: Fri, 4 Apr 2008 18:29:48 +0200\nSubject: [PATCH] git-gui: use +/- instead of ]/[ to show more/less context in diff\n\nOn some systems, brackets cannot be used as event details\n(they don't have a keysym), so use +/- instead (both on\nkeyboard and keypad) and add ctrl-= as a synonym of ctrl-+\nfor convenience.\n\nSigned-off-by: Michele Ballabio <barra_cuda@katamail.com>\n---\n git-gui.sh |   18 ++++++++++++------\n 1 files changed, 12 insertions(+), 6 deletions(-)\n\ndiff --git a/git-gui.sh b/git-gui.sh\nindex 748efcc..744b51c 100755\n--- a/git-gui.sh\n+++ b/git-gui.sh\n@@ -2064,11 +2064,11 @@ if {[is_enabled multicommit] || [is_enabled singlecommit]} {\n \n \t.mbar.commit add command -label [mc \"Show Less Context\"] \\\n \t\t-command show_less_context \\\n-\t\t-accelerator $M1T-\\[\n+\t\t-accelerator $M1T-\\-\n \n \t.mbar.commit add command -label [mc \"Show More Context\"] \\\n \t\t-command show_more_context \\\n-\t\t-accelerator $M1T-\\]\n+\t\t-accelerator \"$M1T-+ $M1T-=\"\n \n \t.mbar.commit add separator\n \n@@ -2715,8 +2715,11 @@ bind $ui_comm <$M1B-Key-v> {tk_textPaste %W; %W see insert; break}\n bind $ui_comm <$M1B-Key-V> {tk_textPaste %W; %W see insert; break}\n bind $ui_comm <$M1B-Key-a> {%W tag add sel 0.0 end;break}\n bind $ui_comm <$M1B-Key-A> {%W tag add sel 0.0 end;break}\n-bind $ui_comm <$M1B-Key-bracketleft> {show_less_context;break}\n-bind $ui_comm <$M1B-Key-bracketright> {show_more_context;break}\n+bind $ui_comm <$M1B-Key-minus> {show_less_context;break}\n+bind $ui_comm <$M1B-Key-KP_Subtract> {show_less_context;break}\n+bind $ui_comm <$M1B-Key-equal> {show_more_context;break}\n+bind $ui_comm <$M1B-Key-plus> {show_more_context;break}\n+bind $ui_comm <$M1B-Key-KP_Add> {show_more_context;break}\n \n bind $ui_diff <$M1B-Key-x> {tk_textCopy %W;break}\n bind $ui_diff <$M1B-Key-X> {tk_textCopy %W;break}\n@@ -2760,8 +2763,11 @@ bind .   <$M1B-Key-t> do_add_selection\n bind .   <$M1B-Key-T> do_add_selection\n bind .   <$M1B-Key-i> do_add_all\n bind .   <$M1B-Key-I> do_add_all\n-bind .   <$M1B-Key-bracketleft> {show_less_context;break}\n-bind .   <$M1B-Key-bracketright> {show_more_context;break}\n+bind .   <$M1B-Key-minus> {show_less_context;break}\n+bind .   <$M1B-Key-KP_Subtract> {show_less_context;break}\n+bind .   <$M1B-Key-equal> {show_more_context;break}\n+bind .   <$M1B-Key-plus> {show_more_context;break}\n+bind .   <$M1B-Key-KP_Add> {show_more_context;break}\n bind .   <$M1B-Key-Return> do_commit\n foreach i [list $ui_index $ui_workdir] {\n \tbind $i <Button-1>       \"toggle_or_diff         $i %x %y; break\"\n-- \n1.5.4.5\n"},{"id":"73686","messageId":"20080405040106.GW10274@spearce.org","threadId":"12973","inReplyTo":"200804042304.42794.barra_cuda@katamail.com","subject":"Re: [REGRESSION] git-gui","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2008-04-05T04:01:06Z","receivedAt":"2008-04-05T04:01:06Z","isPatch":false,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"Michele Ballabio <barra_cuda@katamail.com> wrote:\n> \n> So, it doesn't recognize [ at all at the keysym (%K) level.\n> No idea why. This is Tcl/Tk 8.4.15 on Linux.\n...\n> ctrl-+ and ctrl-=\t-> more context\n> ctrl--\t\t\t-> less context\n\nThis is reasonable.  I like.\n \n>  \t.mbar.commit add command -label [mc \"Show Less Context\"] \\\n>  \t\t-command show_less_context \\\n> -\t\t-accelerator $M1T-\\[\n> +\t\t-accelerator $M1T-\\-\n>  \n>  \t.mbar.commit add command -label [mc \"Show More Context\"] \\\n>  \t\t-command show_more_context \\\n> -\t\t-accelerator $M1T-\\]\n> +\t\t-accelerator \"$M1T-+ $M1T-=\"\n\nSadly Mac OS X does not like this accelerator in the menu.\nIt drops the accelerator entirely and just shows nothing.\nI changed it to show \"$M1T-=\", but kept the bindings as\nyou had them below, so $M1T-+ is an alias.\n  \n> @@ -2715,8 +2715,11 @@ bind $ui_comm <$M1B-Key-v> {tk_textPaste %W; %W see insert; break}\n>  bind $ui_comm <$M1B-Key-V> {tk_textPaste %W; %W see insert; break}\n>  bind $ui_comm <$M1B-Key-a> {%W tag add sel 0.0 end;break}\n>  bind $ui_comm <$M1B-Key-A> {%W tag add sel 0.0 end;break}\n> -bind $ui_comm <$M1B-Key-bracketleft> {show_less_context;break}\n> -bind $ui_comm <$M1B-Key-bracketright> {show_more_context;break}\n> +bind $ui_comm <$M1B-Key-minus> {show_less_context;break}\n> +bind $ui_comm <$M1B-Key-KP_Subtract> {show_less_context;break}\n> +bind $ui_comm <$M1B-Key-equal> {show_more_context;break}\n> +bind $ui_comm <$M1B-Key-plus> {show_more_context;break}\n> +bind $ui_comm <$M1B-Key-KP_Add> {show_more_context;break}\n>  \n>  bind $ui_diff <$M1B-Key-x> {tk_textCopy %W;break}\n>  bind $ui_diff <$M1B-Key-X> {tk_textCopy %W;break}\n> @@ -2760,8 +2763,11 @@ bind .   <$M1B-Key-t> do_add_selection\n>  bind .   <$M1B-Key-T> do_add_selection\n>  bind .   <$M1B-Key-i> do_add_all\n>  bind .   <$M1B-Key-I> do_add_all\n> -bind .   <$M1B-Key-bracketleft> {show_less_context;break}\n> -bind .   <$M1B-Key-bracketright> {show_more_context;break}\n> +bind .   <$M1B-Key-minus> {show_less_context;break}\n> +bind .   <$M1B-Key-KP_Subtract> {show_less_context;break}\n> +bind .   <$M1B-Key-equal> {show_more_context;break}\n> +bind .   <$M1B-Key-plus> {show_more_context;break}\n> +bind .   <$M1B-Key-KP_Add> {show_more_context;break}\n>  bind .   <$M1B-Key-Return> do_commit\n>  foreach i [list $ui_index $ui_workdir] {\n>  \tbind $i <Button-1>       \"toggle_or_diff         $i %x %y; break\"\n> -- \n> 1.5.4.5\n> \n\n-- \nShawn.\n"},{"id":"73802","messageId":"871w5it8r7.fsf@lysator.liu.se","threadId":"12973","inReplyTo":"57518fd10804031615m5c299df6hf31de3c689e16521@mail.gmail.com","subject":"Re: [REGRESSION] git-gui","fromName":"David Kågedal","fromEmail":"davidk@lysator.liu.se","sentAt":"2008-04-07T07:38:20Z","receivedAt":"2008-04-07T07:38:20Z","isPatch":false,"sender":{"key":"davidk@lysator.liu.se","avatar":"https://avatars.githubusercontent.com/u/60530?v=4"},"body":"\"Jonathan del Strother\" <maillist@steelskies.com> writes:\n\n> Anyone else got any opinions on an appropriate shortcut?  How about\n> '=' and '-' (+ and - without the shift), or are those not necessarily\n> together either?  (Are there really layouts where '[' and ']' aren't\n> next to each other?)\n\nA Swedish keyboard has [] on alt-gr (Modeshift) 8 and 9.  Plus and\nminus are unshifted and = is on shift-0.\n\nSo with a Swedish layout, it is annoying when zoom in is on = rather\nthan +, just because someone assumed that it would be harder to press\nthe more logical plus key.\n\n-- \nDavid Kågedal\n"},{"id":"73809","messageId":"20080407081052.GA5413@mithlond.arda.local","threadId":"12973","inReplyTo":"871w5it8r7.fsf@lysator.liu.se","subject":"Re: [REGRESSION] git-gui","fromName":"Teemu Likonen","fromEmail":"tlikonen@iki.fi","sentAt":"2008-04-07T08:10:52Z","receivedAt":"2008-04-07T08:10:52Z","isPatch":false,"sender":{"key":"tlikonen@iki.fi","avatar":null},"body":"David Kågedal kirjoitti (7.4.2008 klo 9.38):\n\n> \"Jonathan del Strother\" <maillist@steelskies.com> writes:\n> \n> > Anyone else got any opinions on an appropriate shortcut?  How about\n> > '=' and '-' (+ and - without the shift), or are those not\n> > necessarily together either?  (Are there really layouts where '['\n> > and ']' aren't next to each other?)\n> \n> A Swedish keyboard has [] on alt-gr (Modeshift) 8 and 9.  Plus and\n> minus are unshifted and = is on shift-0.\n> \n> So with a Swedish layout, it is annoying when zoom in is on = rather\n> than +, just because someone assumed that it would be harder to press\n> the more logical plus key.\n\nThe above also applies to Finnish and Norwegian keyboards, probably to\nmany more. The key pair '=' and '-' would be a poor UI decision. In\nprinciple one should never assume that a key is in certain place in the\nkeyboard; it's much better idea to be logical with _characters_. So if\na key pair much be defined for different aspects of the same\nfunctionality, let's choose logical pairs from common characters: +- []\n{} () <> zZ aA bB ... (To me all these are OK.)\n"},{"id":"73841","messageId":"20080408021807.GX10274@spearce.org","threadId":"12973","inReplyTo":"20080407081052.GA5413@mithlond.arda.local","subject":"Re: [REGRESSION] git-gui","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2008-04-08T02:18:07Z","receivedAt":"2008-04-08T02:18:07Z","isPatch":false,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"Teemu Likonen <tlikonen@iki.fi> wrote:\n> David Kågedal kirjoitti (7.4.2008 klo 9.38):\n> > \"Jonathan del Strother\" <maillist@steelskies.com> writes:\n> > \n> > > Anyone else got any opinions on an appropriate shortcut?  How about\n> > > '=' and '-' (+ and - without the shift), or are those not\n> > > necessarily together either?  (Are there really layouts where '['\n> > > and ']' aren't next to each other?)\n> > \n> > A Swedish keyboard has [] on alt-gr (Modeshift) 8 and 9.  Plus and\n> > minus are unshifted and = is on shift-0.\n> > \n> > So with a Swedish layout, it is annoying when zoom in is on = rather\n> > than +, just because someone assumed that it would be harder to press\n> > the more logical plus key.\n> \n> The above also applies to Finnish and Norwegian keyboards, probably to\n> many more. The key pair '=' and '-' would be a poor UI decision. In\n> principle one should never assume that a key is in certain place in the\n> keyboard; it's much better idea to be logical with _characters_. So if\n> a key pair much be defined for different aspects of the same\n> functionality, let's choose logical pairs from common characters: +- []\n> {} () <> zZ aA bB ... (To me all these are OK.)\n\nWell, Ctrl-+ and Ctrl-= are bound to the same action (increase\ncontext), and Ctrl-- is bound to the opposite action (decrease\ncontext).  So Swedish folks should be able to use Ctrl-+ and\nbe happy.\n\nExcept it doesn't show in the menubar as being a possible keyboard\ndriven action.  That was in the original patch, but I took it out\nto enable the accelator to draw on Mac OS X.\n\n-- \nShawn.\n"}]}