Summary: | Advanced editor doesn't show keyboard shortcuts | ||
---|---|---|---|
Product: | Koha | Reporter: | Katrin Fischer <katrin.fischer> |
Component: | Cataloging | Assignee: | Nick Clemens (kidclamp) <nick> |
Status: | CLOSED FIXED | QA Contact: | Katrin Fischer <katrin.fischer> |
Severity: | major | ||
Priority: | P5 - low | CC: | andrew, arthur.suzuki, lucas, m.de.rooy, nick, victor |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
22.11.00, 22.05.07, 21.11.14
|
|
Circulation function: | |||
Bug Depends on: | |||
Bug Blocks: | 33318 | ||
Attachments: |
Bug 31818: Enable table elements in keyboard shortcuts popover
Bug 31818: Enable table elements in keyboard shortcuts popover Bug 31818: Enable table elements in keyboard shortcuts popover |
Description
Katrin Fischer
2022-10-16 11:00:36 UTC
I wanted to test the clipboard when running into this - might also be worth having a look. Created attachment 143135 [details] [review] Bug 31818: Enable table elements in keyboard shortcuts popover This patch updates the whitelist for the popover to ensrue our shortcuts table displays To test: 1 - Open the advanced cataloging editor 2 - Click 'Keyboard shortcuts' 3 - Is empty, be sad 4 - Apply patch 5 - Reload 6 - Click it again 7 - Shortcuts! Created attachment 143138 [details] [review] Bug 31818: Enable table elements in keyboard shortcuts popover This patch updates the whitelist for the popover to ensrue our shortcuts table displays To test: 1 - Open the advanced cataloging editor 2 - Click 'Keyboard shortcuts' 3 - Is empty, be sad 4 - Apply patch 5 - Reload 6 - Click it again 7 - Shortcuts! Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org> (In reply to Andrew Fuerste-Henry from comment #3) > Created attachment 143138 [details] [review] [review] > Bug 31818: Enable table elements in keyboard shortcuts popover > > This patch updates the whitelist for the popover to ensrue our shortcuts > table displays > > To test: > 1 - Open the advanced cataloging editor > 2 - Click 'Keyboard shortcuts' > 3 - Is empty, be sad > 4 - Apply patch > 5 - Reload > 6 - Click it again > 7 - Shortcuts! > > Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org> In my testing, I was not able to feel actual sadness in step 3, but maybe we can handle that in a followup. Created attachment 143493 [details] [review] Bug 31818: Enable table elements in keyboard shortcuts popover This patch updates the whitelist for the popover to ensrue our shortcuts table displays To test: 1 - Open the advanced cataloging editor 2 - Click 'Keyboard shortcuts' 3 - Is empty, be sad 4 - Apply patch 5 - Reload 6 - Click it again 7 - Shortcuts! Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Pushed to master for 22.11. Nice work everyone, thanks! Backported to 22.05.x for upcoming 22.05.07 release applied to 21.11 for 21.11.14 21.05.x not affected, no backport. Nothing to document, marking resolved. |