I propose the following change. If using the A command on a page (not on a link), and the page has a title, use that as the auto-suggested bookmark name.
Old: ref=s9_dnav_bw_ir12_s new: Amazon.com: Science Fiction - Science Fiction & Fantasy: Kindle Store I don't have git's email sender setup, so I'll just paste a patch here. I did notice one bug I do'nt know how to fix, though it might not matter much. If a link contains < > or &, they're not turned into < > and &. This also applies to the title. I'm not very familiar with the code, but this seems to work. I'll see how it works when I bookmark more. >From 08d9dbd675aa77146c8f1f072a64929de8639a57 Mon Sep 17 00:00:00 2001 From: Tyler Spivey <[email protected]> Date: Sat, 22 Nov 2014 22:57:43 -0800 Subject: [PATCH] Make A suggest title if it can Make the A command suggest the page title if it has one, if we're not on a link. --- src/buffers.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/buffers.c b/src/buffers.c index 540cb99..4799838 100644 --- a/src/buffers.c +++ b/src/buffers.c @@ -3479,7 +3479,9 @@ static char *showLinks(void) stringAndString(&a, &a_l, "<a href="); stringAndString(&a, &a_l, h); stringAndString(&a, &a_l, ">\n"); - s = (char *)getDataURL(h); + s = cw->ft; + if (!s || !*s) + s = (char *)getDataURL(h); if (!s || !*s) s = h; t = s + strcspn(s, "\1?#"); -- 1.9.1 _______________________________________________ Edbrowse-dev mailing list [email protected] http://lists.the-brannons.com/mailman/listinfo/edbrowse-dev
