9front - general discussion about 9front
 help / color / mirror / Atom feed
From: "Jeremy O'Brien" <neutral@fastmail.com>
To: 9front@9front.org
Subject: Re: [9front] mothra back patch
Date: Thu, 13 Sep 2018 08:57:50 -0400	[thread overview]
Message-ID: <1536843470.41398.1506837768.5DD19306@webmail.messagingengine.com> (raw)
In-Reply-To: <1535998418.787504.1495330080.16E7B4C9@webmail.messagingengine.com>

[-- Attachment #1: Type: text/plain, Size: 263 bytes --]

For all zero of you out there using this patch, attached is a bugfixed version since I didn't realize C modulus could return negative numbers. Also included a patch that adds both go back and go forward functionality with 'f'. The patches are mutually exclusive.

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: mothra_back.diff --]
[-- Type: text/x-patch; name="mothra_back.diff", Size: 2430 bytes --]

diff -r eca280998901 sys/man/1/mothra
--- a/sys/man/1/mothra	Tue Sep 11 00:23:35 2018 -0700
+++ b/sys/man/1/mothra	Thu Sep 13 08:38:08 2018 -0400
@@ -100,6 +100,10 @@
 .B moth mode
 menu option again exits moth mode.
 .TP
+.B back
+Navigate backwards through visited link history.  The order of this
+navigation is independent from the order shown in the alt display.
+.TP
 .B snarf
 Copy the current entry text or selected page text to snarf buffer.
 If nothing is selected, the current URL is copied.
@@ -126,6 +130,9 @@
 .B a
 Toggle alt display.
 .TP
+.BI b
+Navigate backwards through visited link history.
+.TP
 .BI g " url
 Go to the page with the given URL.
 .TP
diff -r eca280998901 sys/src/cmd/mothra/mothra.c
--- a/sys/src/cmd/mothra/mothra.c	Tue Sep 11 00:23:35 2018 -0700
+++ b/sys/src/cmd/mothra/mothra.c	Thu Sep 13 08:38:08 2018 -0400
@@ -79,6 +79,8 @@
 Url *selection=0;
 int mothmode;
 int kickpipe[2];
+int history[NWWW];
+int histindex = -1;
 
 void docmd(Panel *, char *);
 void doprev(Panel *, int, int);
@@ -93,6 +95,7 @@
 char *buttons[]={
 	"alt display",
 	"moth mode",
+	"back",
 	"snarf",
 	"paste",
 	"search",
@@ -556,6 +559,7 @@
 	int i;
 	new=www(index);
 	if(new==current && (tag==0 || tag[0]==0)) return;
+	history[histindex] = index;
 	if(current)
 		current->yoffs=plgetpostextview(text);
 	current=new;
@@ -685,6 +689,9 @@
 		else
 			message("Usage: j index");
 		break;
+	case 'b':
+		hit3(3, 2);
+		break;
 	case 'm':
 		mothon(current, !mothmode);
 		break;
@@ -1054,6 +1061,7 @@
 			}
 			plinitlist(list, PACKN|FILLX, genwww, 8, doprev);
 			if(defdisplay) pldraw(list, screen);
+			histindex = (histindex+1) % NWWW;
 			setcurrent(i, selection->tag);
 			break;
 		case GIF:
@@ -1193,15 +1201,21 @@
 		mothon(current, !mothmode);
 		break;
 	case 2:
+		if (histindex <= 0)
+			break;
+		histindex--;
+		doprev(nil, 1, wwwtop-history[histindex]-1);
+		break;
+	case 3:
 		snarf(plkbfocus);
 		break;
-	case 3:
+	case 4:
 		paste(plkbfocus);
 		break;
-	case 4:
+	case 5:
 		search();
 		break;
-	case 5:
+	case 6:
 		if(!selection){
 			message("no url selected");
 			break;
@@ -1221,11 +1235,11 @@
 		fprint(fd, "<p><a href=\"%s\">%s</a>\n", urlstr(selection), urlstr(selection));
 		close(fd);
 		break;
-	case 6:
+	case 7:
 		snprint(name, sizeof(name), "file:%s/hit.html", mkhome());
 		geturl(name, -1, 1, 0);
 		break;
-	case 7:
+	case 8:
 		if(confirm(3))
 			exits(0);
 		break;

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #3: mothra_back_and_forward.diff --]
[-- Type: text/x-patch; name="mothra_back_and_forward.diff", Size: 2648 bytes --]

diff -r eca280998901 sys/man/1/mothra
--- a/sys/man/1/mothra	Tue Sep 11 00:23:35 2018 -0700
+++ b/sys/man/1/mothra	Thu Sep 13 08:37:17 2018 -0400
@@ -100,6 +100,10 @@
 .B moth mode
 menu option again exits moth mode.
 .TP
+.B back
+Navigate backwards through visited link history.  The order of this
+navigation is independent from the order shown in the alt display.
+.TP
 .B snarf
 Copy the current entry text or selected page text to snarf buffer.
 If nothing is selected, the current URL is copied.
@@ -126,6 +130,12 @@
 .B a
 Toggle alt display.
 .TP
+.BI b
+Navigate backwards through visited link history.
+.TP
+.BI f
+Navigate forwards through visisted link history.
+.TP
 .BI g " url
 Go to the page with the given URL.
 .TP
diff -r eca280998901 sys/src/cmd/mothra/mothra.c
--- a/sys/src/cmd/mothra/mothra.c	Tue Sep 11 00:23:35 2018 -0700
+++ b/sys/src/cmd/mothra/mothra.c	Thu Sep 13 08:37:17 2018 -0400
@@ -79,6 +79,8 @@
 Url *selection=0;
 int mothmode;
 int kickpipe[2];
+int history[NWWW];
+int histindex = -1;
 
 void docmd(Panel *, char *);
 void doprev(Panel *, int, int);
@@ -93,6 +95,7 @@
 char *buttons[]={
 	"alt display",
 	"moth mode",
+	"back",
 	"snarf",
 	"paste",
 	"search",
@@ -556,6 +559,7 @@
 	int i;
 	new=www(index);
 	if(new==current && (tag==0 || tag[0]==0)) return;
+	history[histindex] = index;
 	if(current)
 		current->yoffs=plgetpostextview(text);
 	current=new;
@@ -685,6 +689,15 @@
 		else
 			message("Usage: j index");
 		break;
+	case 'b':
+		hit3(3, 2);
+		break;
+	case 'f':
+		if (histindex+1 == NWWW || history[histindex+1] == 0)
+			break;
+		histindex++;
+		doprev(nil, 1, wwwtop-history[histindex]-1);
+		break;
 	case 'm':
 		mothon(current, !mothmode);
 		break;
@@ -1054,6 +1067,7 @@
 			}
 			plinitlist(list, PACKN|FILLX, genwww, 8, doprev);
 			if(defdisplay) pldraw(list, screen);
+			histindex = (histindex+1) % NWWW;
 			setcurrent(i, selection->tag);
 			break;
 		case GIF:
@@ -1193,15 +1207,21 @@
 		mothon(current, !mothmode);
 		break;
 	case 2:
+		if (histindex <= 0)
+			break;
+		histindex--;
+		doprev(nil, 1, wwwtop-history[histindex]-1);
+		break;
+	case 3:
 		snarf(plkbfocus);
 		break;
-	case 3:
+	case 4:
 		paste(plkbfocus);
 		break;
-	case 4:
+	case 5:
 		search();
 		break;
-	case 5:
+	case 6:
 		if(!selection){
 			message("no url selected");
 			break;
@@ -1221,11 +1241,11 @@
 		fprint(fd, "<p><a href=\"%s\">%s</a>\n", urlstr(selection), urlstr(selection));
 		close(fd);
 		break;
-	case 6:
+	case 7:
 		snprint(name, sizeof(name), "file:%s/hit.html", mkhome());
 		geturl(name, -1, 1, 0);
 		break;
-	case 7:
+	case 8:
 		if(confirm(3))
 			exits(0);
 		break;

  reply	other threads:[~2018-09-13 12:57 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-09-03 18:13 Jeremy O'Brien
2018-09-13 12:57 ` Jeremy O'Brien [this message]
2018-09-13 13:18   ` Steve Simon

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=1536843470.41398.1506837768.5DD19306@webmail.messagingengine.com \
    --to=neutral@fastmail.com \
    --cc=9front@9front.org \
    /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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).