no frame focus if url unchanged

tborychowski · Jan 17, 2017 · ba5c13d · ba5c13d
1 parent cc0f9fb
commit ba5c13d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/app/addressbar/index.js b/app/addressbar/index.js
@@ -45,8 +45,6 @@ function onUrlChanged (webview, issue) {
 	lastShortUrl = shortenUrl(lastFullUrl);
 	el[0].value = lastShortUrl;
 
-	console.log(lastFullUrl, lastShortUrl);
-
 	if (issue && issue.id) {
 		starBox.removeClass('disabled');
 		starsDB.getById(issue.id).then(res => {

diff --git a/app/frame/index.js b/app/frame/index.js
@@ -13,7 +13,7 @@ const wpcss = `${__dirname}/webview.css`;
 
 const ses = session.fromPartition('persist:github');
 
-let webview, isReady = false;
+let webview, isReady = false, lastUrl = '';
 
 const webviewHandlers = {
 	documentClicked () { $.trigger('document-clicked'); },
@@ -91,9 +91,11 @@ function loadingStart () {
 }
 
 function loadingStop () {
+	const newUrl = webview[0].getURL();
 	webview.removeClass('loading');
 	$.trigger('url-change-end');
-	webview[0].focus();
+	if (lastUrl !== newUrl) webview[0].focus();
+	lastUrl = newUrl;
 }