# [PATCH] gitk: Swap positions of 'next' and 'prev' buttons in the 'Find' section.

2 messages from 2008-12-17 to 2008-12-21. Participants: Johannes Sixt, Paul Mackerras.
Thread: https://gitlist.dev/t/16767

## Johannes Sixt, 2008-12-17 11:50

Subject: [PATCH] gitk: Swap positions of 'next' and 'prev' buttons in the 'Find' section.
Message-ID: <4948E779.9080909@viscovery.net>
URL: https://gitlist.dev/e/4948E779.9080909%40viscovery.net

```
From: Johannes Sixt <j6t@kdbg.org>

The button order 'prev' left of 'next' feels more natural than the other
way round, in particular, compared to the order of the forward and backward
arrows that are in the line above.

Signed-off-by: Johannes Sixt <j6t@kdbg.org>
---
 gitk-git/gitk |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/gitk-git/gitk b/gitk-git/gitk
index b63109b..1c8d98e 100644
--- a/gitk-git/gitk
+++ b/gitk-git/gitk
@@ -2052,7 +2052,7 @@ proc makewindow {} {
     button .tf.lbar.fnext -text [mc "next"] -command {dofind 1 1}
     button .tf.lbar.fprev -text [mc "prev"] -command {dofind -1 1}
     label .tf.lbar.flab2 -text " [mc "commit"] "
-    pack .tf.lbar.flabel .tf.lbar.fnext .tf.lbar.fprev .tf.lbar.flab2 \
+    pack .tf.lbar.flabel .tf.lbar.fprev .tf.lbar.fnext .tf.lbar.flab2 \
 	-side left -fill y
     set gdttype [mc "containing:"]
     set gm [tk_optionMenu .tf.lbar.gdttype gdttype \
-- 
1.6.1.rc3.980.ga0bb

```

## Paul Mackerras, 2008-12-21 22:59

Subject: Re: [PATCH] gitk: Swap positions of 'next' and 'prev' buttons in the 'Find' section.
Message-ID: <18766.51818.313516.421046@cargo.ozlabs.ibm.com>
URL: https://gitlist.dev/e/18766.51818.313516.421046%40cargo.ozlabs.ibm.com
In-Reply-To: <4948E779.9080909@viscovery.net>

```
Johannes Sixt writes:

> The button order 'prev' left of 'next' feels more natural than the other
> way round, in particular, compared to the order of the forward and backward
> arrows that are in the line above.

The next/prev labels are a bit awkward in any case, since "next" finds
an earlier commit - one further down the list, and "prev" finds a
later commit - one further back in the list.  A better solution might
be to make images for arrows pointing up and down and use compound
buttons so the buttons say Find ^ and Find v (if you image the ^/v as
up and down arrows).

I also find myself often wanting a "Find first" button, i.e. search
starting at the top of the list.

Paul.

```