Hello folks,
I have noticed that some times the _SURF_URI property is not updated.

A possible way to reproduce this problem:

 1. Run `surf 'https://surf.suckless.org''
 2. Click on a link, e.g. `files/'
 3. Press Ctrl-g - or inspect the _SURF_URI property - and notice that
    it is `https://surf.suckless.org/' instead of
    `https://surf.suckless.org/files/'

The attached patch in this email fix this issue.


Thanks!
From 53b50770b62a2554ec95af50629c5bcbbca72d60 Mon Sep 17 00:00:00 2001
From: Leonardo Taccari <iaml...@gmail.com>
Date: Sat, 15 Aug 2020 16:26:14 +0200
Subject: [PATCH] Set URI atom for WEBKIT_LOAD_COMMITTED event too

---
 surf.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/surf.c b/surf.c
index 2b54e3c..3d00ee1 100644
--- a/surf.c
+++ b/surf.c
@@ -1513,6 +1513,7 @@ loadchanged(WebKitWebView *v, WebKitLoadEvent e, Client *c)
 		seturiparameters(c, uri, loadtransient);
 		break;
 	case WEBKIT_LOAD_COMMITTED:
+		setatom(c, AtomUri, uri);
 		seturiparameters(c, uri, loadcommitted);
 		c->https = webkit_web_view_get_tls_info(c->view, &c->cert,
 		                                        &c->tlserr);
-- 
2.27.0

Reply via email to