Summary: | Authorities editor update broke the feature added by Bug 20154 | ||
---|---|---|---|
Product: | Koha | Reporter: | Owen Leonard <oleonard> |
Component: | Templates | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | david, kyle, marjorie.barry-vila, patrick.robitaille, testopia |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
22.05.00
|
|
Circulation function: | |||
Bug Depends on: | 24415, 30419 | ||
Bug Blocks: | |||
Attachments: |
Bug 30422: Authorities editor update broke the feature added by Bug 20154
Bug 30422: Authorities editor update broke the feature added by Bug 20154 Bug 30422: Authorities editor update broke the feature added by Bug 20154 Bug 30422: Authorities editor update broke the feature added by Bug 20154 |
Description
Owen Leonard
2022-03-31 14:58:18 UTC
Created attachment 132800 [details] [review] Bug 30422: Authorities editor update broke the feature added by Bug 20154 This patch corrects JavaScript in authorities templates so that clicking "Edit" on the authority detail page will take you to the authority edit page while keeping the same tab open. This patch also corrects the JavaScript which should have been automatically moving the cursor focus to the first form field when switching tabs in the editor. To test, apply the patch and locate an authority record in your catalog. - Click any of the numbered tabs besides tab 0. - Choose "Edit record" from the "Edit" dropdown menu. - When the authorities edit page loads the same tab should be preselected. A couple of things noted when testing: - If you are on tab 0 and go to edit, it opens in tab 8 (without the tab 8 highlighted) - It jumps to the first field used in the tab for the record (for example: tab 6 for authority record #98 - it jumps down to field 667, but it is not focused on that field) - There is a noticeable delay for me when you click edit: it jumps to the first field with a value for the tab, then hides the fields not in that tab (not sure that is the best way to describe this) - When switching tabs and if the first fields are collapsed, then it looks slightly odd having the focus on the indicator field. But I guess there is no better way to do this. Created attachment 134822 [details] [review] Bug 30422: Authorities editor update broke the feature added by Bug 20154 This patch corrects JavaScript in authorities templates so that clicking "Edit" on the authority detail page will take you to the authority edit page while keeping the same tab open. This patch also corrects the JavaScript which should have been automatically moving the cursor focus to the first form field when switching tabs in the editor. This patch also adds the "Section" headings which are present in the basic cataloging editor but were not added here at the time of the page's tabs revision. To test, apply the patch and locate an authority record in your catalog. - Without changing tabs, choose "Edit record" from the "Edit" dropdown menu." - The authorities editor should load and display the "Section 0" tab. - Return to the authority detail page and select a tab besides tab 0. - Click back to tab 0. Editing the record at this point should open the editor to Section 0. - Return to the authority detail page. - Click any of the numbered tabs besides tab 0. - Choose "Edit record" from the "Edit" dropdown menu. - When the authorities edit page loads the same tab should be preselected. - Test that when you switch tabs in the authority editor the cursor focus is automatically moved to the first visible input. Created attachment 134835 [details] [review] Bug 30422: Authorities editor update broke the feature added by Bug 20154 This patch corrects JavaScript in authorities templates so that clicking "Edit" on the authority detail page will take you to the authority edit page while keeping the same tab open. This patch also corrects the JavaScript which should have been automatically moving the cursor focus to the first form field when switching tabs in the editor. This patch also adds the "Section" headings which are present in the basic cataloging editor but were not added here at the time of the page's tabs revision. To test, apply the patch and locate an authority record in your catalog. - Without changing tabs, choose "Edit record" from the "Edit" dropdown menu." - The authorities editor should load and display the "Section 0" tab. - Return to the authority detail page and select a tab besides tab 0. - Click back to tab 0. Editing the record at this point should open the editor to Section 0. - Return to the authority detail page. - Click any of the numbered tabs besides tab 0. - Choose "Edit record" from the "Edit" dropdown menu. - When the authorities edit page loads the same tab should be preselected. - Test that when you switch tabs in the authority editor the cursor focus is automatically moved to the first visible input. Signed-off-by: David Nind <david@davidnind.com> Created attachment 134956 [details] [review] Bug 30422: Authorities editor update broke the feature added by Bug 20154 This patch corrects JavaScript in authorities templates so that clicking "Edit" on the authority detail page will take you to the authority edit page while keeping the same tab open. This patch also corrects the JavaScript which should have been automatically moving the cursor focus to the first form field when switching tabs in the editor. This patch also adds the "Section" headings which are present in the basic cataloging editor but were not added here at the time of the page's tabs revision. To test, apply the patch and locate an authority record in your catalog. - Without changing tabs, choose "Edit record" from the "Edit" dropdown menu." - The authorities editor should load and display the "Section 0" tab. - Return to the authority detail page and select a tab besides tab 0. - Click back to tab 0. Editing the record at this point should open the editor to Section 0. - Return to the authority detail page. - Click any of the numbered tabs besides tab 0. - Choose "Edit record" from the "Edit" dropdown menu. - When the authorities edit page loads the same tab should be preselected. - Test that when you switch tabs in the authority editor the cursor focus is automatically moved to the first visible input. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de> Pushed to master for 22.05, thanks to everybody involved [U+1F984] |