Add callback to Popups, focus subscribe URL input.

functions.js

@@ -1313,7 +1313,7 @@ function closeInfoBox(cleanup) {
 }
 
 
-function displayDlg(id, param) {
+function displayDlg(id, param, callback) {
 
 	notify_progress("Loading, please wait...", true);
 
@@ -1326,6 +1326,7 @@ function displayDlg(id, param) {
 		parameters: query,
 		onComplete: function (transport) {
 			infobox_callback2(transport);
+			if (callback) callback(transport);
 		} });
 
 	return false;
@@ -1366,14 +1367,6 @@ function infobox_callback2(transport) {
 
 		}
 
-		/* FIXME this needs to be moved out somewhere */
-
-		if ($("tags_choices")) {
-			new Ajax.Autocompleter('tags_str', 'tags_choices',
-				"backend.php?op=rpc&subop=completeTags",
-				{ tokens: ',', paramName: "search" });
-		}
-
 		disableHotkeys();
 
 		notify("");

modules/popup-dialog.php

@@ -101,7 +101,7 @@
 
 			print "<input size=\"40\"
 					onkeypress=\"return filterCR(event, subscribeToFeed)\"
-					name=\"feed_url\"></td></tr>";
+					name=\"feed_url\" id=\"feed_url\"></td></tr>";
 
 			print "<br/>";
 

tt-rss.js

@@ -581,7 +581,8 @@ function quickMenuGo(opid) {
 		}
 	
 		if (opid == "qmcAddFeed") {
-			displayDlg("quickAddFeed");
+			displayDlg('quickAddFeed', '',
+					   function () {$('feed_url').focus();});
 			return;
 		}
 

viewfeed.js

@@ -1468,7 +1468,12 @@ function catchupSelection() {
 }
 
 function editArticleTags(id, feed_id, cdm_enabled) {
-	displayDlg('editArticleTags', id);
+	displayDlg('editArticleTags', id,
+			   function () {
+				   new Ajax.Autocompleter('tags_str', 'tags_choices',
+					   "backend.php?op=rpc&subop=completeTags",
+					   { tokens: ',', paramName: "search" });
+			   });
 }