[Bug 37002] New: Correct several HTML markup errors
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37002 Bug ID: 37002 Summary: Correct several HTML markup errors Change sponsored?: --- Product: Koha Version: Main Hardware: All OS: All Status: ASSIGNED Severity: minor Priority: P5 - low Component: Templates Assignee: oleonard@myacpl.org Reporter: oleonard@myacpl.org QA Contact: testopia@bugs.koha-community.org Blocks: 33195 There are several HTML markup errors reported by the W3C validator when looking at the source of the bibliographic detail page in the staff interface. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33195 [Bug 33195] Reindent the bibliographic details page -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37002 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Trivial patch Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37002 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 167310 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=167310&action=edit Bug 37002: Correct several HTML markup errors This patch makes several minor corrections to HTML markup in the bibliographic detail page template and related include files. This corrects the following validator errors: - Stray end tag a. - Bad value true for attribute disabled on element select. - Bad value true for attribute disabled on element input. - No space between attributes. - Duplicate attribute class. - The value of the for attribute of the label element must be the ID of a non-hidden form control. The patch is simple enough that an inspection of the patch is probably enough, but following is a detailed test plan of the affected areas: - Apply the patch and view the bibliographic detail page in the staff interface. - Under the "Edit" menu in the toolbar, these options should still work correctly: - Modify record using template - Edit items in a batch - Delete items in a batch - Click the "Items" link in the sidebar and find the "Bookable" setting for the title's items. Make at least one item bookable and return to the detail page. - Click the "Place booking" button in the toolbar and confirm that you can successfully place the booking. - If you have access to Novelist, confirm that Novelist content displays correctly on the detail page. - Turn on the "EnableItemGroups" preference. - On the detail page, open the "Item groups" tab and click the "New item group" button. - In the modal, the "Name" and "Display order" labels give focus to the corresponding form field when clicked. - Create an item group. - Under the holdings tab, select one or more items and click "Add/move to item group". - In the modal, clicking the "Item group" label should give focus to the dropdown. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37002 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37002 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #167310|0 |1 is obsolete| | --- Comment #2 from David Nind <david@davidnind.com> --- Created attachment 167316 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=167316&action=edit Bug 37002: Correct several HTML markup errors This patch makes several minor corrections to HTML markup in the bibliographic detail page template and related include files. This corrects the following validator errors: - Stray end tag a. - Bad value true for attribute disabled on element select. - Bad value true for attribute disabled on element input. - No space between attributes. - Duplicate attribute class. - The value of the for attribute of the label element must be the ID of a non-hidden form control. The patch is simple enough that an inspection of the patch is probably enough, but following is a detailed test plan of the affected areas: - Apply the patch and view the bibliographic detail page in the staff interface. - Under the "Edit" menu in the toolbar, these options should still work correctly: - Modify record using template - Edit items in a batch - Delete items in a batch - Click the "Items" link in the sidebar and find the "Bookable" setting for the title's items. Make at least one item bookable and return to the detail page. - Click the "Place booking" button in the toolbar and confirm that you can successfully place the booking. - If you have access to Novelist, confirm that Novelist content displays correctly on the detail page. - Turn on the "EnableItemGroups" preference. - On the detail page, open the "Item groups" tab and click the "New item group" button. - In the modal, the "Name" and "Display order" labels give focus to the corresponding form field when clicked. - Create an item group. - Under the holdings tab, select one or more items and click "Add/move to item group". - In the modal, clicking the "Item group" label should give focus to the dropdown. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37002 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #167316|0 |1 is obsolete| | --- Comment #3 from Lucas Gass <lucas@bywatersolutions.com> --- Created attachment 167357 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=167357&action=edit Bug 37002: Correct several HTML markup errors This patch makes several minor corrections to HTML markup in the bibliographic detail page template and related include files. This corrects the following validator errors: - Stray end tag a. - Bad value true for attribute disabled on element select. - Bad value true for attribute disabled on element input. - No space between attributes. - Duplicate attribute class. - The value of the for attribute of the label element must be the ID of a non-hidden form control. The patch is simple enough that an inspection of the patch is probably enough, but following is a detailed test plan of the affected areas: - Apply the patch and view the bibliographic detail page in the staff interface. - Under the "Edit" menu in the toolbar, these options should still work correctly: - Modify record using template - Edit items in a batch - Delete items in a batch - Click the "Items" link in the sidebar and find the "Bookable" setting for the title's items. Make at least one item bookable and return to the detail page. - Click the "Place booking" button in the toolbar and confirm that you can successfully place the booking. - If you have access to Novelist, confirm that Novelist content displays correctly on the detail page. - Turn on the "EnableItemGroups" preference. - On the detail page, open the "Item groups" tab and click the "New item group" button. - In the modal, the "Name" and "Display order" labels give focus to the corresponding form field when clicked. - Create an item group. - Under the holdings tab, select one or more items and click "Add/move to item group". - In the modal, clicking the "Item group" label should give focus to the dropdown. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Current status: Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37002 Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |matt.blenkinsop@ptfs-europe | |.com --- Comment #4 from Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> --- Lucas did you mean to do a second sign off or was it supposed to be a PQA? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37002 --- Comment #5 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Picked from QA queue for main. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37002 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37002 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to main Version(s)| |24.11.00 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37002 --- Comment #6 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Pushed for 24.11! Well done everyone, thank you! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37002 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lucas@bywatersolutions.com Status|Pushed to main |Pushed to stable Version(s)|24.11.00 |24.11.00,24.05.02 released in| | --- Comment #7 from Lucas Gass <lucas@bywatersolutions.com> --- Backported to 24.05.x for upcoming 24.05.02 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37002 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com Text to go in the| |This fixes several minor release notes| |HTML markup validation | |errors for the | |bibliographic detail page | |in the staff interface. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37002 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fridolin.somers@biblibre.co | |m --- Comment #8 from Fridolin Somers <fridolin.somers@biblibre.com> --- Does not apply on 23.11.x Maybe not needed -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37002 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to stable |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org