instead of show/hide, enable/disable

This commit is contained in:
Rachid Mrad 2023-10-26 14:39:27 -04:00
parent d07f7b614b
commit 9dd2e733cc
No known key found for this signature in database
GPG key ID: EF38E4CEC4A8F3CF

View file

@ -311,13 +311,14 @@ function prepareDeleteButtons(formLabel) {
// Remove the add more button if we have less than 13 forms // Remove the add more button if we have less than 13 forms
if (isNameserversForm && forms.length <= 13) { if (isNameserversForm && forms.length <= 13) {
addButton.classList.remove("display-none") addButton.classList.remove("display-none");
} }
if (isNameserversForm && forms.length < 3) { if (isNameserversForm && forms.length < 3) {
// Hide the delete buttons on the remaining nameservers // Hide the delete buttons on the remaining nameservers
Array.from(form.querySelectorAll('.delete-record')).forEach((deleteButton) => { Array.from(form.querySelectorAll('.delete-record')).forEach((deleteButton) => {
deleteButton.classList.add("display-none"); // deleteButton.classList.add("display-none");
deleteButton.setAttribute("disabled", "true");
}); });
} }
@ -381,11 +382,12 @@ function prepareDeleteButtons(formLabel) {
}); });
} }
// If the source element we're copying has an invisible delete button, // If the source element we're copying has an disabled delete button,
// show that button // enable that button
let deleteButtonNeedsCleanUp = newForm.querySelector('.delete-record').classList.contains("display-none"); let deleteButton= newForm.querySelector('.delete-record');
if (deleteButtonNeedsCleanUp) { if (deleteButton.hasAttribute("disabled")) {
newForm.querySelector('.delete-record').classList.remove("display-none"); // newForm.querySelector('.delete-record').classList.remove("display-none");
deleteButton.removeAttribute("disabled");
} }
} }
@ -438,16 +440,18 @@ function prepareDeleteButtons(formLabel) {
// Attach click event listener on the delete buttons of the new form // Attach click event listener on the delete buttons of the new form
prepareDeleteButtons(formLabel); prepareDeleteButtons(formLabel);
// Hide the add more button if we have 13 forms // Disable the add more button if we have 13 forms
if (isNameserversForm && formNum == 13) { if (isNameserversForm && formNum == 13) {
addButton.classList.add("display-none") // addButton.classList.add("display-none");
addButton.setAttribute("disabled", "true");
} }
if (isNameserversForm && forms.length >= 2) { if (isNameserversForm && forms.length >= 2) {
// Show the delete buttons on the nameservers // Enable the delete buttons on the nameservers
forms.forEach((form, index) => { forms.forEach((form, index) => {
Array.from(form.querySelectorAll('.delete-record')).forEach((deleteButton) => { Array.from(form.querySelectorAll('.delete-record')).forEach((deleteButton) => {
deleteButton.classList.remove("display-none"); // deleteButton.classList.remove("display-none");
deleteButton.removeAttribute("disabled");
}); });
}); });
} }