[bug] fix group name special character support when adding links

zombieFox · zombieFox · commit 89cd24568e5f · 2019-12-02T10:53:41.000Z
diff --git a/src/js/link.js b/src/js/link.js
@@ -819,7 +819,15 @@ var link = (function(){
 var makeGroupOptions = function(){
 if (bookmarks.get().length > 0){
 bookmarks.get().forEach(function(arrayItem, index){
- groupExistingGroup.appendChild(helper.node("option:" + arrayItem.name + "|value:" + arrayItem.name));
+ var option = helper.makeNode({
+ tag: "option",
+ text: arrayItem.name,
+ attr: [{
+ key: "value",
+ value: arrayItem.name
+ }]
+ });
+ groupExistingGroup.appendChild(option);
 });
 } else{
 groupNewRadio.checked = true;

-Original file line number
+Diff line change
 varmakeGroupOptions=function(){
 if(bookmarks.get().length>0){
 bookmarks.get().forEach(function(arrayItem,index){
 -groupExistingGroup.appendChild(helper.node("option:"+arrayItem.name+"|value:"+arrayItem.name));
 +varoption=helper.makeNode({
 +tag: "option",
 +text: arrayItem.name,
 +attr: [{
 +key: "value",
 +value: arrayItem.name
 +}]
 +});
 +groupExistingGroup.appendChild(option);
 });
 }else{
 groupNewRadio.checked=true;