[Bug 22768] New: Basic MARC editor tab keyboard navigation broekn
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22768 Bug ID: 22768 Summary: Basic MARC editor tab keyboard navigation broekn Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: ASSIGNED Severity: normal Priority: P5 - low Component: Templates Assignee: oleonard@myacpl.org Reporter: oleonard@myacpl.org QA Contact: testopia@bugs.koha-community.org CC: dpavlin@rot13.org, testopia@bugs.koha-community.org Depends on: 22045 Blocks: 22399 Target Milestone: --- Dobrica's comment from Bug 22045: This bug broke keyboard shortcuts because of following change: --- a/koha-tmpl/intranet-tmpl/prog/js/staff-global.js +++ b/koha-tmpl/intranet-tmpl/prog/js/staff-global.js @@ -31,7 +31,7 @@ $.fn.tabIndex = function () { return $(this).parent().children('div').index(this); }; $.fn.selectTabByID = function (tabID) { - $(this).tabs("option", "active", $(tabID).tabIndex()); + $(this).tabs("option", "active", $( "#" + tabID ).tabIndex()); }; $(document).ready(function() { without modification to calling which also prefix tabID with #, thus creating invalid ##tabID Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22045 [Bug 22045] Cataloging UX enhancement - Improve access to tabs https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22399 [Bug 22399] Improve responsive behavior of the basic marc editor -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22768 Josef Moravec <josef.moravec@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |josef.moravec@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22768 Bug 22768 depends on bug 22045, which changed state. Bug 22045 Summary: Cataloging UX enhancement - Improve access to tabs https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22045 What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22768 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Basic MARC editor tab |Global search forms' |keyboard navigation broekn |keyboard navigation broken -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22768 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Small 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=22768 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 90810 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=90810&action=edit Bug 22768: Global search form keyboard navigation broken Bug 22045 made changes to the global staff client JavaScript which broken keyboard navigation for the most common forms in the resident search form("Check out, Check in, Renew, Search). This patch reimplements the tab selection code on the basic MARC editor page so that the keyboard shortcut behavior can be restored. To test, apply the patch go to Cataloging. - Search for an existing record and edit it in the basic MARC editor. - Test that the numbered tabs (0-9) at the top work correctly. - Select a tab other than 0 and choose Save -> Save and continue editing. When the page reloads you should be returned to the tab you were on when you saved. - Go to the Circulation home page. Test the keyboard shortcuts for switching tabs in header area: - Alt-u (Check out) - Alt-r (Check in) - Alt-w (Renew) - Alt-q (Search) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22768 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de --- Comment #2 from Owen Leonard <oleonard@myacpl.org> --- *** Bug 23169 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22768 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Patch doesn't apply -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22768 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22768 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #90810|0 |1 is obsolete| | --- Comment #3 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 91486 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=91486&action=edit Bug 22768: Global search form keyboard navigation broken Bug 22045 made changes to the global staff client JavaScript which broken keyboard navigation for the most common forms in the resident search form("Check out, Check in, Renew, Search). This patch reimplements the tab selection code on the basic MARC editor page so that the keyboard shortcut behavior can be restored. To test, apply the patch go to Cataloging. - Search for an existing record and edit it in the basic MARC editor. - Test that the numbered tabs (0-9) at the top work correctly. - Select a tab other than 0 and choose Save -> Save and continue editing. When the page reloads you should be returned to the tab you were on when you saved. - Go to the Circulation home page. Test the keyboard shortcuts for switching tabs in header area: - Alt-u (Check out) - Alt-r (Check in) - Alt-w (Renew) - Alt-q (Search) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22768 Claire Gravely <claire_gravely@hotmail.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=22768 Claire Gravely <claire_gravely@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #91486|0 |1 is obsolete| | --- Comment #4 from Claire Gravely <claire_gravely@hotmail.com> --- Created attachment 91487 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=91487&action=edit Bug 22768: Global search form keyboard navigation broken Bug 22045 made changes to the global staff client JavaScript which broken keyboard navigation for the most common forms in the resident search form("Check out, Check in, Renew, Search). This patch reimplements the tab selection code on the basic MARC editor page so that the keyboard shortcut behavior can be restored. To test, apply the patch go to Cataloging. - Search for an existing record and edit it in the basic MARC editor. - Test that the numbered tabs (0-9) at the top work correctly. - Select a tab other than 0 and choose Save -> Save and continue editing. When the page reloads you should be returned to the tab you were on when you saved. - Go to the Circulation home page. Test the keyboard shortcuts for switching tabs in header area: - Alt-u (Check out) - Alt-r (Check in) - Alt-w (Renew) - Alt-q (Search) Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22768 Nadine Pierre <nadine.pierre@InLibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #91487|0 |1 is obsolete| | --- Comment #5 from Nadine Pierre <nadine.pierre@InLibro.com> --- Created attachment 91502 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=91502&action=edit Bug 22768: Global search form keyboard navigation broken Bug 22045 made changes to the global staff client JavaScript which broken keyboard navigation for the most common forms in the resident search form("Check out, Check in, Renew, Search). This patch reimplements the tab selection code on the basic MARC editor page so that the keyboard shortcut behavior can be restored. To test, apply the patch go to Cataloging. - Search for an existing record and edit it in the basic MARC editor. - Test that the numbered tabs (0-9) at the top work correctly. - Select a tab other than 0 and choose Save -> Save and continue editing. When the page reloads you should be returned to the tab you were on when you saved. - Go to the Circulation home page. Test the keyboard shortcuts for switching tabs in header area: - Alt-u (Check out) - Alt-r (Check in) - Alt-w (Renew) - Alt-q (Search) Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de> Signed-off-by: Nadine Pierre <nadine.pierre@inLibro.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22768 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=22768 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #91502|0 |1 is obsolete| | --- Comment #6 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 91518 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=91518&action=edit Bug 22768: Global search form keyboard navigation broken Bug 22045 made changes to the global staff client JavaScript which broken keyboard navigation for the most common forms in the resident search form("Check out, Check in, Renew, Search). This patch reimplements the tab selection code on the basic MARC editor page so that the keyboard shortcut behavior can be restored. To test, apply the patch go to Cataloging. - Search for an existing record and edit it in the basic MARC editor. - Test that the numbered tabs (0-9) at the top work correctly. - Select a tab other than 0 and choose Save -> Save and continue editing. When the page reloads you should be returned to the tab you were on when you saved. - Go to the Circulation home page. Test the keyboard shortcuts for switching tabs in header area: - Alt-u (Check out) - Alt-r (Check in) - Alt-w (Renew) - Alt-q (Search) Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de> Signed-off-by: Nadine Pierre <nadine.pierre@inLibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22768 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |19.11.00 released in| | Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22768 --- Comment #7 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Nice work! Pushed to master for 19.11.00 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22768 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)|19.11.00 |19.11.00,19.05.03 released in| | Status|Pushed to master |Pushed to stable CC| |fridolin.somers@biblibre.co | |m --- Comment #8 from Fridolin SOMERS <fridolin.somers@biblibre.com> --- Pushed to 19.05.x for 19.05.03 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22768 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lucas@bywatersolutions.com --- Comment #9 from Lucas Gass <lucas@bywatersolutions.com> --- missing dependencies, wont backport to 18.11.x -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22768 Lucas Gass <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