scinote-eln/scinote-web
1
1
Fork 0
mirror of https://github.com/scinote-eln/scinote-web.git synced 2024-09-20 23:16:15 +08:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #6014 from wandji20/wb-SCI-9057

Browse source 
Fix bug in new result table event delegation [SCI-9057]
This commit is contained in:
artoscinote 2023-08-18 10:13:36 +02:00 committed by GitHub
parent 664819a7fc cf3cb707fd
commit 45791a764f
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 13 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
app/assets/javascripts/results/result_tables.js.erb
View file

@ -65,8 +65,15 @@
}
// Save result table callback
function resultTableCallback () {
$('result-tables-buttons').on('click', '.save-result', (event) => {
function newResultTableCallback() {
$('.new-result-tables-buttons .save-result').on('click', (event) => {
Results.processResult(event, Results.ResultTypeEnum.TABLE);
});
}
// Save result table callback
function saveResultTableCallback() {
$('.edit-result-tables-buttons .save-result').on('click', (event) => {
Results.processResult(event, Results.ResultTypeEnum.TABLE);
});
}
@ -87,7 +94,6 @@
Results.expandResult($result);
Comments.init();
initNewResultTable();
resultTableCallback();
});
$form.on('ajax:error', function(e, xhr, status, error) {
var data = xhr.responseJSON;
@ -119,7 +125,7 @@
Results.toggleResultEditButtons(false);
$('#result_name').focus();
resultTableCallback();
saveResultTableCallback();
});
}
@ -146,6 +152,7 @@
Results.initCancelFormButton($form, initNewResultTable);
Results.toggleResultEditButtons(false);
$('#result_name').focus();
newResultTableCallback();
},
error: function() {
animateSpinner(null, false);

2
app/views/result_tables/_edit.html.erb
View file

@ -14,7 +14,7 @@
<% end %>
</div>
<hr>
<div class="align-right result-tables-buttons">
<div class="align-right edit-result-tables-buttons">
<button type="button" class="btn btn-secondary cancel-edit">
<%= t("general.cancel")%>
</button>

2
app/views/result_tables/_new.html.erb
View file

@ -13,7 +13,7 @@
</div>
<% end %>
</div>
<div class="align-right result-tables-buttons">
<div class="align-right new-result-tables-buttons">
<button type="button" class="btn btn-secondary cancel-new">
<%= t("general.cancel")%>
</button>