Re: [PATCH] Save and restore point explicitly in `notmuch-wash-toggle-invisible-action'.

Subject: Re: [PATCH] Save and restore point explicitly in `notmuch-wash-toggle-invisible-action'.

Date: Tue, 24 May 2011 16:36:35 -0700

To: Dmitry Kurochkin, Austin Clements

Cc: notmuch@notmuchmail.org

From: Carl Worth

On Wed, 25 May 2011 03:27:51 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:
> (button-end cite-button) would move the point outside the button - to
> the next character after it.

OK. Your fix addresses my off-by-one bug. I've just pushed the whole
series, with your final amended fix, (and some updates I made to its
commit message).

Thanks again, Dmitry. This will be a nice refinement in the interface.

-Carl

-- 
carl.d.worth@intel.com

• Previous message (by thread): Re: [PATCH] Save and restore point explicitly in `notmuch-wash-toggle-invisible-action'.

Thread:

• Dmitry Kurochkin—[PATCH 1/3] Use different labels for wash buttons when text is visible or hidden. [inbox, unread]
  • Dmitry Kurochkin—[PATCH 2/3] test: fix expected output for emacs tests after the wash button label changes [inbox, unread]
  • Dmitry Kurochkin—[PATCH 3/3] test: add test for hiding/showing signature in notmuch-show view [inbox, unread]
  • Dmitry Kurochkin—[PATCH] Save and restore point explicitly in `notmuch-wash-toggle-invisible-action'. [inbox, unread]
    • Carl Worth—Re: [PATCH] Save and restore point explicitly in `notmuch-wash-toggle-invisible-action'. [inbox, signed, unread]
      • Dmitry Kurochkin—Re: [PATCH] Save and restore point explicitly in `notmuch-wash-toggle-invisible-action'. [inbox, unread]
        • Carl Worth—Re: [PATCH] Save and restore point explicitly in `notmuch-wash-toggle-invisible-action'. [inbox, signed, unread]
          • Dmitry Kurochkin—Re: [PATCH] Save and restore point explicitly in `notmuch-wash-toggle-invisible-action'. [inbox, unread]
            • Carl Worth—Re: [PATCH] Save and restore point explicitly in `notmuch-wash-toggle-invisible-action'. [inbox, signed, unread]
            • Austin Clements—Re: [PATCH] Save and restore point explicitly in `notmuch-wash-toggle-invisible-action'. [inbox, unread]
              • Carl Worth—Re: [PATCH] Save and restore point explicitly in `notmuch-wash-toggle-invisible-action'. [inbox, signed, unread]
              • Dmitry Kurochkin—Re: [PATCH] Save and restore point explicitly in `notmuch-wash-toggle-invisible-action'. [inbox, unread]
                • Dmitry Kurochkin—[PATCH] Save and restore point explicitly in `notmuch-wash-toggle-invisible-action'. [inbox, unread]
              • Carl Worth—Re: [PATCH] Save and restore point explicitly in `notmuch-wash-toggle-invisible-action'. [inbox, signed, unread]
                • Dmitry Kurochkin—Re: [PATCH] Save and restore point explicitly in `notmuch-wash-toggle-invisible-action'. [inbox, unread]
                  • Carl Worth—Re: [PATCH] Save and restore point explicitly in `notmuch-wash-toggle-invisible-action'. [inbox, signed, unread]