From f7e94260dd4fef2825b0fd6c7f8a6170e8599aea Mon Sep 17 00:00:00 2001 From: "Utilu.com" Date: Tue, 23 Aug 2016 09:30:52 +0200 Subject: [PATCH] Use comma between entries when copying the list of extensions (#195) --- extension/chrome/content/nightly.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/extension/chrome/content/nightly.js b/extension/chrome/content/nightly.js index d16d2db..f6c54b1 100644 --- a/extension/chrome/content/nightly.js +++ b/extension/chrome/content/nightly.js @@ -350,7 +350,7 @@ getExtensionList: function(callback) { + (addon.userDisabled || addon.appDisabled ? " [DISABLED]" : ""); }); strings.sort(nightly.insensitiveSort); - callback(strings.join("\n")); + callback(strings); }); } catch(e) { // old extension manager API - take out after Firefox 3.6 support dropped @@ -384,7 +384,7 @@ getExtensionList: function(callback) { catch (e) { } } text.sort(nightly.insensitiveSort); - callback(text.join("\n")); + callback(text); } }, @@ -396,7 +396,7 @@ insertExtensions: function() { nightly.getExtensionList(function(text) { var newpos = element.selectionStart + text.length; var value = element.value; - element.value = value.substring(0, element.selectionStart) + text + + element.value = value.substring(0, element.selectionStart) + text.join(", ") + value.substring(element.selectionEnd); element.selectionStart = newpos; element.selectionEnd = newpos; @@ -410,7 +410,7 @@ insertExtensions: function() { copyExtensions: function() { nightly.getExtensionList(function(text) { if (text) - nightly.copyText(text); + nightly.copyText(text.join(", ")); }); },