diff --git a/assets/src/bundles/save/index.js b/assets/src/bundles/save/index.js index 9d8bed44..2d42ba27 100644 --- a/assets/src/bundles/save/index.js +++ b/assets/src/bundles/save/index.js @@ -50,26 +50,19 @@ export function maybeRequireExtraInputs() { // inputs or hide them. This also makes the extra inputs required when displayed. const originType = $('#swh-input-visit-type').val(); let cssDisplay; - let requiredBool; + let disabled; if (originType === 'archives') { - requiredBool = true; + disabled = false; cssDisplay = 'flex'; } else { cssDisplay = 'none'; - requiredBool = false; + disabled = true; } - console.log(`css display ${cssDisplay}`); - console.log(`required bool ${requiredBool}`); - $('#optional-origin-forms').css('display', cssDisplay); - document.getElementById('swh-input-artifact-url').required = requiredBool; - document.getElementById('swh-input-artifact-version').required = requiredBool; - - // following did not work - // $('#swh-input-artifact-url').prop('required', requiredBool); - // $('#swh-input-artifact-url').attr('required', requiredBool); + $('#swh-input-artifact-url').prop('disabled', disabled); + $('#swh-input-artifact-version').prop('disabled', disabled); } const userRequestsFilterCheckbox = ` diff --git a/swh/web/templates/misc/origin-save.html b/swh/web/templates/misc/origin-save.html index 585f36b1..29137968 100644 --- a/swh/web/templates/misc/origin-save.html +++ b/swh/web/templates/misc/origin-save.html @@ -55,12 +55,12 @@ See top-level LICENSE file for more information