all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Tassilo Horn <tassilo@member.fsf.org>
To: Gideon Stupp <gideon.stupp@gmail.com>
Cc: emacs-devel@gnu.org
Subject: Re: [PATCH] Conkeror like hints for incremental search
Date: Tue, 24 Jan 2012 08:55:35 +0100	[thread overview]
Message-ID: <87aa5dmt54.fsf@tsdh.uni-koblenz.de> (raw)
In-Reply-To: <CAJzWQ4fbMiaZDohH_Dmy11mH8CnEnZqAQAZy3bi_heHdR456rg@mail.gmail.com> (Gideon Stupp's message of "Mon, 23 Jan 2012 19:52:04 +0200")

Gideon Stupp <gideon.stupp@gmail.com> writes:

Hi Gideon,

> This patch adds Conkeror like hints to incremental-search's currently
> matched strings.
> This way it is possible to choose one of the highlighted possibilities
> without completing the search string (useful when using incremental
> search for navigation).

isearch is my main navigation feature, so I've just given your patch a
try.  Here's my feedback:

  - The characters inserted in front of matches are somewhat baffling
    (First I've thought: Why do I have all those typos?!).  At least in
    X11 frames, I'd make them smaller than the usual text, maybe even
    superscript.  And it would be nice if the hint chars where appended
    to matches in forward isearch but prepended in backward isearch.
    That way, the hint char would indicate the position of point when
    selecting that match.

  - Concerning navigation speed, I don't feel that those type hints help
    me.  It might do so if the visible buffer portion contains tons of
    matches, but in my regular use, there are only ~3 matches, so
    hitting C-s or C-r repeatedly is faster for me than switching my
    finger to some other key.

  - I think C-f (and all other keys normally used for navigation) is a
    bad choice, because that's what people usually use for leaving
    isearch.  Maybe M-h would be ok.  It has some mnemonics, and its
    usual binding of `mark-paragraph' isn't too useful in isearch.

Bye,
Tassilo



  reply	other threads:[~2012-01-24  7:55 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-23 17:52 [PATCH] Conkeror like hints for incremental search Gideon Stupp
2012-01-24  7:55 ` Tassilo Horn [this message]
2012-01-24 10:09   ` gideon.stupp
2012-01-24 17:39     ` Stefan Monnier
2012-01-24 18:47       ` Gideon Stupp
2012-01-24 19:12         ` Stefan Monnier
2012-01-24 11:44 ` Juanma Barranquero
2012-01-24 13:31 ` bug#10595: " Gideon Stupp
2016-02-25  6:09   ` Lars Ingebrigtsen
2016-02-25 15:00     ` Gideon Stupp
2016-02-26  5:55       ` Lars Ingebrigtsen
2016-02-26  6:43         ` John Wiegley
2016-02-26  7:02           ` Lars Ingebrigtsen
2012-01-24 22:55 ` Juri Linkov
2012-01-25 11:03   ` gideon.stupp
2012-01-25 11:29     ` Juri Linkov
2012-01-25 11:59       ` gideon.stupp
2012-01-25 14:00         ` Stephen J. Turnbull

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87aa5dmt54.fsf@tsdh.uni-koblenz.de \
    --to=tassilo@member.fsf.org \
    --cc=emacs-devel@gnu.org \
    --cc=gideon.stupp@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.