On 13-12-18 11:04 AM, Marc Branchaud wrote:
> Users often find that "next" and "prev" do the opposite of what they
> expect. For example, "next" moves to the next match down the list, but
> that is almost always backwards in time. Replacing the text with arrows
> makes it clear where the buttons will take the user.
Any opinions on this, either way?
I've grown fond of the down/up arrows. I find them much clearer than the
current next/prev buttons.
My only niggle about this patch is that the buttons are much smaller,
requiring a bit more precision clicking. But the smaller buttons allow more
room for other widgets.
M.
> Signed-off-by: Marc Branchaud <[email protected]>
> ---
>
> Finally got around to drawing some up and down arrows.
>
> M.
>
> gitk | 30 ++++++++++++++++++++++++++++--
> 1 file changed, 28 insertions(+), 2 deletions(-)
>
> diff --git a/gitk b/gitk
> index 33c3a6c..abd2ef3 100755
> --- a/gitk
> +++ b/gitk
> @@ -2263,9 +2263,35 @@ proc makewindow {} {
>
> # build up the bottom bar of upper window
> ${NS}::label .tf.lbar.flabel -text "[mc "Find"] "
> - ${NS}::button .tf.lbar.fnext -text [mc "next"] -command {dofind 1 1}
> - ${NS}::button .tf.lbar.fprev -text [mc "prev"] -command {dofind -1 1}
> +
> + set bm_down_data {
> + #define down_width 16
> + #define down_height 16
> + static unsigned char down_bits[] = {
> + 0x80, 0x01, 0x80, 0x01, 0x80, 0x01, 0x80, 0x01,
> + 0x80, 0x01, 0x80, 0x01, 0x80, 0x01, 0x80, 0x01,
> + 0x87, 0xe1, 0x8e, 0x71, 0x9c, 0x39, 0xb8, 0x1d,
> + 0xf0, 0x0f, 0xe0, 0x07, 0xc0, 0x03, 0x80, 0x01};
> + }
> + image create bitmap bm-down -data $bm_down_data -foreground $uifgcolor
> + ${NS}::button .tf.lbar.fnext -width 26 -command {dofind 1 1}
> + .tf.lbar.fnext configure -image bm-down
> +
> + set bm_up_data {
> + #define up_width 16
> + #define up_height 16
> + static unsigned char up_bits[] = {
> + 0x80, 0x01, 0xc0, 0x03, 0xe0, 0x07, 0xf0, 0x0f,
> + 0xb8, 0x1d, 0x9c, 0x39, 0x8e, 0x71, 0x87, 0xe1,
> + 0x80, 0x01, 0x80, 0x01, 0x80, 0x01, 0x80, 0x01,
> + 0x80, 0x01, 0x80, 0x01, 0x80, 0x01, 0x80, 0x01};
> + }
> + image create bitmap bm-up -data $bm_up_data -foreground $uifgcolor
> + ${NS}::button .tf.lbar.fprev -width 26 -command {dofind -1 1}
> + .tf.lbar.fprev configure -image bm-up
> +
> ${NS}::label .tf.lbar.flab2 -text " [mc "commit"] "
> +
> pack .tf.lbar.flabel .tf.lbar.fnext .tf.lbar.fprev .tf.lbar.flab2 \
> -side left -fill y
> set gdttype [mc "containing:"]
>
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
