When you are seeing a specific authorized value, it's no longer possible to go to another authorized value using the drop-down menu "Show category".
Hi Séverine, can you be more specific? I am seeing the "Show category" on this view https://snipboard.io/Cda091.jpg Do you mean when we edit an authorised value?
Hi Jonathan ! Here is a capture made a few minutes ago on Biblibre's sandbox #1 : https://prnt.sc/zmai4z When I select a new category in the menu, it doesn't change. When the page is slow to charge, I can see a "Submit" button that finally disappear.
Caused by commit 38bc9eb2a8c01c8625b1c33c7e098cd0022b2710 Bug 25954: Replace paragraph tags with labels in header search forms Owen, can you have a look please? There are 2 elements with id=searchfield on this page.
This occurrence might be problematic as well: koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt: <input type="text" name="searchfield" id="searchfield" size="4" value="[% searchfield | html %]" />
Created attachment 116923 [details] [review] Bug 27703: Make ID of category select field unique Duplicate element IDs on the authorized values page are invalid and cause a problem with the JavaScript intended to submit the form when the category select field changes. This patch changes the ID of the label and the field and makes the corresponding change to the JavaScript. To test, apply the patch and go to Administration -> Authorized values. - Click an authorized value category to view its contents. - Using mouse or keyboard, make a selection from the "Show category" dropdown menu. - The form should submit automatically and direct you to the corresponding category. - Validate the HTML of the page and confirm that there are no errors.
Created attachment 116924 [details] [review] Bug 27703: Make ID of category select field unique Duplicate element IDs on the authorized values page are invalid and cause a problem with the JavaScript intended to submit the form when the category select field changes. This patch changes the ID of the label and the field and makes the corresponding change to the JavaScript. To test, apply the patch and go to Administration -> Authorized values. - Click an authorized value category to view its contents. - Using mouse or keyboard, make a selection from the "Show category" dropdown menu. - The form should submit automatically and direct you to the corresponding category. - Validate the HTML of the page and confirm that there are no errors.
Created attachment 117231 [details] [review] Bug 27703: Make ID of category select field unique Duplicate element IDs on the authorized values page are invalid and cause a problem with the JavaScript intended to submit the form when the category select field changes. This patch changes the ID of the label and the field and makes the corresponding change to the JavaScript. To test, apply the patch and go to Administration -> Authorized values. - Click an authorized value category to view its contents. - Using mouse or keyboard, make a selection from the "Show category" dropdown menu. - The form should submit automatically and direct you to the corresponding category. - Validate the HTML of the page and confirm that there are no errors. Signed-off-by: David Nind <david@davidnind.com>
Now works as expected when trying to change the category from the dropdown list. Not exactly sure how you want the HTML validated. I used Firefox Developer tools, and the HTML Validator - 30 warnings, along the lines of "Warning: trimming empty <i>", "Warning: trimming empty <b>", "Warning: trimming empty <span>", and "Warning: '<' + '/' + letter not allowed here". The WAVE accessibility checker toolbar shows different errors and warnings.
Created attachment 117560 [details] [review] Bug 27703: Make ID of category select field unique Duplicate element IDs on the authorized values page are invalid and cause a problem with the JavaScript intended to submit the form when the category select field changes. This patch changes the ID of the label and the field and makes the corresponding change to the JavaScript. To test, apply the patch and go to Administration -> Authorized values. - Click an authorized value category to view its contents. - Using mouse or keyboard, make a selection from the "Show category" dropdown menu. - The form should submit automatically and direct you to the corresponding category. - Validate the HTML of the page and confirm that there are no errors. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Works as expected and passes QA scripts. No duplicate ID errors during validation and other validation issues can be caught in new bugs. Passing QA
Pushed to master for 21.05, thanks to everybody involved!
Pushed to 20.11.x for 20.11.04
Pushed to 20.05.x for 20.05.10
Not backported to oldoldstable (19.11.x). Feel free to ask if it's needed.