Re: Feature request for somewhen
On (20:58 16/06/07), Adeodato Simó <dato@xxxxxxxxxxxxxx> put forth the
proposition:
> * David Woodfall [Sat, 16 Jun 2007 00:07:50 +0100]:
>
> > When entering an mbox from a folder, and then exiting back to the folder,
> > presently the cursor returns to the top of the folder. It would be nice if
> > the cursor remained opposite the mbox exited from, in the same way that
> > when browsing an mbox and exiting an email/thread does.
>
> > It would save an awful lot of scrolling down when, say, checking new posts
> > in mailing lists.
>
> > Just a thought anyway.
>
> Hello. The feature you want is implemented in the sensible_browser_position
> patch by Haakon Riiser:
>
> http://does-not-exist.org/mail-archives/mutt-dev/msg02623.html
> OR
>
> http://hg.debian.org/hg/pkg-mutt/debian-mutt?cmd=file;file=debian/patches/not-applied/patch-1.5.8.hr.sensible_browser_position.3;filenode=4f3e410fcc58a5721d0c7910a8ab2e2cd1996d22;style=raw
Thanks. I've edited the patch slightly so that it works with out having to
toggle mailbox view (tab). It now works in any view. I'm not a experienced
programmer at all. I just changed a couple of lines:
'if (menu->is_mailbox_list)' to 'if (true)'
'if (menu->is_mailbox_list && last_position >= 0)' to
'if (last_position >= 0)'
I think that is all. Haven't a clue if it breaks something else though.
--- mutt-1.5.8/menu.c.orig 2005-02-12 21:01:55.000000000 +0100
+++ mutt-1.5.8/menu.c 2005-03-09 16:39:49.000000000 +0100
@@ -841,8 +841,12 @@
int mutt_menuLoop (MUTTMENU *menu)
{
+ static int last_position = -1;
int i = OP_NULL;
+ if (last_position >= 0)
+ menu->current = last_position;
+
FOREVER
{
if (option (OPTMENUCALLER))
@@ -1068,6 +1072,8 @@
break;
default:
+ if (true)
+ last_position = menu->current;
return (i);
}
}
--- mutt-1.5.8/browser.c.orig 2005-02-03 19:47:52.000000000 +0100
+++ mutt-1.5.8/browser.c 2005-03-09 16:48:31.000000000 +0100
@@ -54,6 +54,7 @@
int num;
} FOLDER;
+static char OldLastDir[_POSIX_PATH_MAX] = "";
static char LastDir[_POSIX_PATH_MAX] = "";
static char LastDirBackup[_POSIX_PATH_MAX] = "";
@@ -498,19 +499,38 @@
menu->tagged = 0;
if (buffy)
+ {
+ menu->is_mailbox_list = 1;
snprintf (title, titlelen, _("Mailboxes [%d]"), mutt_buffy_check (0));
+ }
else
{
+ menu->is_mailbox_list = 0;
strfcpy (path, LastDir, sizeof (path));
mutt_pretty_mailbox (path);
#ifdef USE_IMAP
- if (state->imap_browse && option (OPTIMAPLSUB))
- snprintf (title, titlelen, _("Subscribed [%s], File mask: %s"),
- path, NONULL (Mask.pattern));
- else
+ if (state->imap_browse && option (OPTIMAPLSUB))
+ snprintf (title, titlelen, _("Subscribed [%s], File mask: %s"),
+ path, NONULL (Mask.pattern));
+ else
#endif
- snprintf (title, titlelen, _("Directory [%s], File mask: %s"),
- path, NONULL(Mask.pattern));
+ {
+ char *p = strrchr (OldLastDir, '/');
+ if (p && p - OldLastDir == mutt_strlen (LastDir) &&
+ mutt_strncmp (LastDir, OldLastDir, p - OldLastDir) == 0)
+ {
+ /* If we get here, it means that LastDir is the parent directory of
+ * OldLastDir. I.e., we're returning from a subdirectory, and we want
+ * to position the cursor on the directory we're returning from. */
+ int i;
+ for (i = 0; i < state->entrymax; i++)
+ if (mutt_strcmp (state->entry[i].name, p + 1) == 0)
+ menu->current = i;
+ }
+ snprintf (title, titlelen, _("Directory [%s], File mask: %s"),
+ path, NONULL(Mask.pattern));
+ }
+
}
menu->redraw = REDRAW_FULL;
}
@@ -685,7 +705,6 @@
#endif
)
{
- char OldLastDir[_POSIX_PATH_MAX];
/* save the old directory */
strfcpy (OldLastDir, LastDir, sizeof (OldLastDir));
--- mutt-1.5.8/mutt_menu.h.orig 2003-10-04 21:29:27.000000000 +0200
+++ mutt-1.5.8/mutt_menu.h 2005-03-09 16:39:49.000000000 +0100
@@ -46,6 +46,7 @@
int offset; /* which screen row to start the index */
int pagelen; /* number of entries per screen */
int tagprefix;
+ int is_mailbox_list;
>
> I suggested it for 1.6, but received no feedback AFAICR.
>
> Cheers,
>
> --
> Adeodato Simó dato at net.com.org.es
> Debian Developer adeodato at debian.org
>
> This is my boyfriend, Chester Fleet. [...] It's kind of a weird day to meet
> him
> because he lost a bet with me, and that means that for the entire day, he can
> only talk in clichés. It is so much fun.
> -- Tana Schrick
--
Universe, n.:
The problem.