diff --git a/static/js/consos.js b/static/js/consos.js index a9a48a520c11d33ef9fce82df7127c6d562aeebc..9fb0a8cbb22ad03e2ba2c89f2bf8a45de3febd29 100644 --- a/static/js/consos.js +++ b/static/js/consos.js @@ -24,40 +24,56 @@ $(document).ready(function() { }); // Switching in double consumptions mode should update the layout - $("#double_conso").click(function() { + let double_conso_obj = $("#double_conso"); + double_conso_obj.click(function() { $("#consos_list_div").show(); $("#infos_div").attr('class', 'col-sm-5 col-xl-6'); $("#note_infos_div").attr('class', 'col-xl-3'); $("#user_select_div").attr('class', 'col-xl-4'); $("#buttons_div").attr('class', 'col-sm-7 col-xl-6'); - if (buttons.length > 0) { - let note_list_obj = $("#note_list"); + let note_list_obj = $("#note_list"); + if (buttons.length > 0 && note_list_obj.text().length > 0) { $("#consos_list").html(note_list_obj.html()); note_list_obj.html(""); + + buttons.forEach(function(button) { + $("#conso_button_" + button.id).click(removeNote(button, "conso_button", buttons, + "consos_list")); + }); } }); - $("#single_conso").click(function() { + let single_conso_obj = $("#single_conso"); + single_conso_obj.click(function() { $("#consos_list_div").hide(); $("#infos_div").attr('class', 'col-sm-5 col-md-4'); $("#note_infos_div").attr('class', 'col-xl-5'); $("#user_select_div").attr('class', 'col-xl-7'); $("#buttons_div").attr('class', 'col-sm-7 col-md-8'); + let consos_list_obj = $("#consos_list"); if (buttons.length > 0) { - if (notes_display.length === 0) { - let consos_list_obj = $("#consos_list"); + if (notes_display.length === 0 && consos_list_obj.text().length > 0) { $("#note_list").html(consos_list_obj.html()); consos_list_obj.html(""); + buttons.forEach(function(button) { + $("#conso_button_" + button.id).click(removeNote(button, "conso_button", buttons, + "note_list")); + }); } else { buttons.length = 0; - $("#consos_list").html(""); + consos_list_obj.html(""); } } }); + // Ensure we begin in single consumption. Removing these lines may cause problems when reloading. + single_conso_obj.prop('checked', 'true'); + double_conso_obj.removeAttr('checked'); + $("label[for='double_conso']").attr('class', 'btn btn-sm btn-outline-primary'); + $("#consos_list_div").hide(); $("#consume_all").click(consumeAll); diff --git a/templates/note/conso_form.html b/templates/note/conso_form.html index 3925241d440e3884d4045e2d3caddba35ceef3e7..b108a96f83f81df9588c1c992c719b152bc7a0d8 100644 --- a/templates/note/conso_form.html +++ b/templates/note/conso_form.html @@ -121,11 +121,11 @@ <i class="fa fa-edit"></i> {% trans "Edit" %} </a> <div class="btn-group btn-group-toggle float-right" data-toggle="buttons"> - <label class="btn btn-sm btn-outline-primary active"> + <label for="single_conso" class="btn btn-sm btn-outline-primary active"> <input type="radio" name="conso_type" id="single_conso" checked> {% trans "Single consumptions" %} </label> - <label class="btn btn-sm btn-outline-primary"> + <label for="double_conso" class="btn btn-sm btn-outline-primary"> <input type="radio" name="conso_type" id="double_conso"> {% trans "Double consumptions" %} </label> diff --git a/templates/note/transaction_form.html b/templates/note/transaction_form.html index eb85687ee735536964b8b1eb57c8175e16852996..4cbf4e997e99f05848f2f3fd233770cd96bb25df 100644 --- a/templates/note/transaction_form.html +++ b/templates/note/transaction_form.html @@ -10,19 +10,19 @@ SPDX-License-Identifier: GPL-2.0-or-later <div class="row"> <div class="col-xl-12"> <div class="btn-group btn-group-toggle" style="width: 100%; padding: 0 0 2em 0" data-toggle="buttons"> - <label class="btn btn-sm btn-outline-primary active"> + <label for="type_gift" class="btn btn-sm btn-outline-primary active"> <input type="radio" name="transaction_type" id="type_gift" checked> {% trans "Gift" %} </label> - <label class="btn btn-sm btn-outline-primary"> + <label for="type_transfer" class="btn btn-sm btn-outline-primary"> <input type="radio" name="transaction_type" id="type_transfer"> {% trans "Transfer" %} </label> - <label class="btn btn-sm btn-outline-primary"> + <label for="type_credit" class="btn btn-sm btn-outline-primary"> <input type="radio" name="transaction_type" id="type_credit"> {% trans "Credit" %} </label> - <label class="btn btn-sm btn-outline-primary"> + <label type="type_debit" class="btn btn-sm btn-outline-primary"> <input type="radio" name="transaction_type" id="type_debit"> {% trans "Debit" %} </label> @@ -194,6 +194,16 @@ SPDX-License-Identifier: GPL-2.0-or-later return true; }); + + + // Ensure we begin in gift mode. Removing these lines may cause problems when reloading. + $("#type_gift").prop('checked', 'true'); + $("#type_transfer").removeAttr('checked'); + $("#type_credit").removeAttr('checked'); + $("#type_debit").removeAttr('checked'); + $("label[for='type_gift']").attr('class', 'btn btn-sm btn-outline-primary'); + $("label[for='type_transfer']").attr('class', 'btn btn-sm btn-outline-primary'); + $("label[for='type_debit']").attr('class', 'btn btn-sm btn-outline-primary'); }); $("#type_gift").click(function() {