git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] gitk: Allow toggling of checkboxes by clicking on the label
@ 2008-05-07 22:43 Jean-Luc Herren
  0 siblings, 0 replies; only message in thread
From: Jean-Luc Herren @ 2008-05-07 22:43 UTC (permalink / raw)
  To: git; +Cc: Paul Mackerras


Signed-off-by: Jean-Luc Herren <jlh@gmx.ch>
---
I hope this makes any sense, since I do not know any tcl/tk at all.  I
think being able to click on the label of checkboxes is good manner,
since otherwise you have to click into a fairly small area.  However,
this patch unfortunately also reduces the size of the checkbox itself,
proably because it adapts itself to the small optionfont.  I don't know
how to fix this other than increasing the size of optionfont, so I'll
let someone else do this, if this is desired.

 gitk-git/gitk |   24 ++++++++++++------------
 1 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/gitk-git/gitk b/gitk-git/gitk
index 9a4d9c4..ef8dcc0 100644
--- a/gitk-git/gitk
+++ b/gitk-git/gitk
@@ -8015,14 +8015,14 @@ proc doprefs {} {
     spinbox $top.maxpct -from 1 -to 100 -width 4 -textvariable maxgraphpct
     grid x $top.maxpctl $top.maxpct -sticky w
     frame $top.showlocal
-    label $top.showlocal.l -text [mc "Show local changes"] -font optionfont
-    checkbutton $top.showlocal.b -variable showlocalchanges
-    pack $top.showlocal.b $top.showlocal.l -side left
+    checkbutton $top.showlocal.b -variable showlocalchanges \
+	-text [mc "Show local changes"] -font optionfont
+    pack $top.showlocal.b -side left
     grid x $top.showlocal -sticky w
     frame $top.autoselect
-    label $top.autoselect.l -text [mc "Auto-select SHA1"] -font optionfont
-    checkbutton $top.autoselect.b -variable autoselect
-    pack $top.autoselect.b $top.autoselect.l -side left
+    checkbutton $top.autoselect.b -variable autoselect \
+	-text [mc "Auto-select SHA1"] -font optionfont
+    pack $top.autoselect.b -side left
     grid x $top.autoselect -sticky w
 
     label $top.ddisp -text [mc "Diff display options"]
@@ -8031,14 +8031,14 @@ proc doprefs {} {
     spinbox $top.tabstop -from 1 -to 20 -width 4 -textvariable tabstop
     grid x $top.tabstopl $top.tabstop -sticky w
     frame $top.ntag
-    label $top.ntag.l -text [mc "Display nearby tags"] -font optionfont
-    checkbutton $top.ntag.b -variable showneartags
-    pack $top.ntag.b $top.ntag.l -side left
+    checkbutton $top.ntag.b -variable showneartags \
+	-text [mc "Display nearby tags"] -font optionfont
+    pack $top.ntag.b -side left
     grid x $top.ntag -sticky w
     frame $top.ldiff
-    label $top.ldiff.l -text [mc "Limit diffs to listed paths"] -font optionfont
-    checkbutton $top.ldiff.b -variable limitdiffs
-    pack $top.ldiff.b $top.ldiff.l -side left
+    checkbutton $top.ldiff.b -variable limitdiffs \
+	-text [mc "Limit diffs to listed paths"] -font optionfont
+    pack $top.ldiff.b -side left
     grid x $top.ldiff -sticky w
 
     label $top.cdisp -text [mc "Colors: press to choose"]
-- 
1.5.5.1

^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2008-05-07 22:44 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-05-07 22:43 [PATCH] gitk: Allow toggling of checkboxes by clicking on the label Jean-Luc Herren

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).