From 51c175d160d4d9b0fce6ca9194131c059e6cf90c Mon Sep 17 00:00:00 2001 From: Stephen Birarda Date: Tue, 6 Jan 2015 15:30:38 -0800 Subject: [PATCH 1/2] put input default back after row add --- domain-server/resources/web/js/settings.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/domain-server/resources/web/js/settings.js b/domain-server/resources/web/js/settings.js index 141bd72a26..cf39012d0f 100644 --- a/domain-server/resources/web/js/settings.js +++ b/domain-server/resources/web/js/settings.js @@ -364,7 +364,7 @@ function makeTableInputs(setting) { _.each(setting.columns, function(col) { html += "\ \ + value='" + (col.default ? col.default : "") + "' data-default='" + (col.default ? col.default : "") + "'>\ " }) @@ -504,7 +504,7 @@ function addTableRow(add_glyphicon) { }) input_clone.find('input').each(function(){ - $(this).val('') + $(this).val($(this).attr('data-default')); }); if (isArray) { From 2103e0150fe3b00e620dc5682e36937ec3e7c13f Mon Sep 17 00:00:00 2001 From: Stephen Birarda Date: Tue, 6 Jan 2015 15:42:01 -0800 Subject: [PATCH 2/2] fix for row removal in domain array settings --- domain-server/resources/web/js/settings.js | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/domain-server/resources/web/js/settings.js b/domain-server/resources/web/js/settings.js index cf39012d0f..1e9051d717 100644 --- a/domain-server/resources/web/js/settings.js +++ b/domain-server/resources/web/js/settings.js @@ -525,9 +525,9 @@ function deleteTableRow(delete_glyphicon) { var table = $(row).closest('table') var isArray = table.data('setting-type') === 'array' + row.empty(); + if (!isArray) { - // this is a hash row, so we empty it but leave the hidden input blank so it is cleared when we save - row.empty() row.html(""); } else { @@ -538,7 +538,6 @@ function deleteTableRow(delete_glyphicon) { row.remove() } else { // this is the last row, we can't remove it completely since we need to post an empty array - row.empty() row.removeClass(Settings.DATA_ROW_CLASS).removeClass(Settings.NEW_ROW_CLASS) row.addClass('empty-array-row') @@ -592,7 +591,7 @@ function updateDataChangedForSiblingRows(row, forceTrue) { var initialPanelSettingJSON = Settings.initialValues[panelParentID][tableShortName] // if they are equal, we don't need data-changed - isTrue = _.isEqual(panelSettingJSON, initialPanelSettingJSON) + isTrue = !_.isEqual(panelSettingJSON, initialPanelSettingJSON) } else { isTrue = true }