Skip to content

Commit 52e847d

Browse files
committed
Eliminate some duplicate JS code
1 parent b77ab39 commit 52e847d

1 file changed

Lines changed: 12 additions & 20 deletions

File tree

styles/prosilver/template/ideas.js

Lines changed: 12 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -169,11 +169,7 @@
169169

170170
$this.hide();
171171

172-
$obj.rfcEdit.text(function() {
173-
return value ? $(this).attr('data-l-edit') : $(this).attr('data-l-add');
174-
}).prepend($('<i class="fa fa-fw"></i>').addClass(function() {
175-
return value ? 'fa-pencil' : 'fa-plus-circle';
176-
})).show();
172+
$obj.rfcEdit.toggleAddEdit(value);
177173
}
178174
});
179175
} else if (e.keyCode === keymap.ESC) {
@@ -227,11 +223,7 @@
227223

228224
$this.hide();
229225

230-
$obj.ticketEdit.text(function() {
231-
return value ? $(this).attr('data-l-edit') : $(this).attr('data-l-add');
232-
}).prepend($('<i class="fa fa-fw"></i>').addClass(function() {
233-
return value ? 'fa-pencil' : 'fa-plus-circle';
234-
})).show();
226+
$obj.ticketEdit.toggleAddEdit(value);
235227
}
236228

237229
});
@@ -288,11 +280,7 @@
288280

289281
$this.hide();
290282

291-
$obj.duplicateEdit.text(function() {
292-
return value ? $(this).attr('data-l-edit') : $(this).attr('data-l-add');
293-
}).prepend($('<i class="fa fa-fw"></i>').addClass(function() {
294-
return value ? 'fa-pencil' : 'fa-plus-circle';
295-
})).show();
283+
$obj.duplicateEdit.toggleAddEdit(value);
296284
}
297285
});
298286
} else if (e.keyCode === keymap.ESC) {
@@ -341,11 +329,7 @@
341329

342330
$this.hide();
343331

344-
$obj.implementedEdit.text(function() {
345-
return value ? $(this).attr('data-l-edit') : $(this).attr('data-l-add');
346-
}).prepend($('<i class="fa fa-fw"></i>').addClass(function() {
347-
return value ? 'fa-pencil' : 'fa-plus-circle';
348-
})).show();
332+
$obj.implementedEdit.toggleAddEdit(value);
349333
}
350334
});
351335
} else if (e.keyCode === keymap.ESC) {
@@ -360,6 +344,14 @@
360344
}
361345
});
362346

347+
$.fn.toggleAddEdit = function(value) {
348+
$(this).text(function() {
349+
return value ? $(this).attr('data-l-edit') : $(this).attr('data-l-add');
350+
}).prepend($('<i class="fa fa-fw"></i>').addClass(function() {
351+
return value ? 'fa-pencil' : 'fa-plus-circle';
352+
})).show();
353+
};
354+
363355
/**
364356
* Returns true if idea is a duplicate. Bit hacky.
365357
*/

0 commit comments

Comments
 (0)