Created attachment 157971 [details] Screenshot of the duplicated entries in the keyboard shortcuts dialog SUMMARY In 'Settings > Configure Keyboard Shortcuts', I see two items called 'show sidebar', but only one of them works (the other one does not affect the sidebar even when a keyboard shortcut is assigned to it). Similarly, there are two 'show sidebar' actions in the list of available toolbar buttons, only one of which works. STEPS TO REPRODUCE Part A (duplicated keyboard shortcut) 1. Settings > Configure Keyboard shortcuts 2. Search for 'sidebar' Part B (duplicated toolbar entry) 1. Right-click on Okular toolbar > Configure toolbars 2. In 'main toolbar <okular_shell>', search for 'show sidebar' 3. In 'main toolbar <okular_part>', search for 'show sidebar' 4. Place the 'show sidebar item in <okular_part> onto the toolbar and save changes 5. Try to find the new 'show sidebar' button on the toolbar OBSERVED RESULT Part A: Two items called 'show sidebar' (see attached screenshot), one of which does nothing when a keyboard shortcut is assigned to it Part B: Only the show sidebar item in okular_shell works, while the other one shows no button even if placed on the toolbar. EXPECTED RESULT Only one 'show sidebar' item SOFTWARE/OS VERSIONS Operating System: Arch Linux KDE Plasma Version: 5.27.4 KDE Frameworks Version: 5.104.0 Qt Version: 5.15.8 Kernel Version: 6.2.10-arch1-1 (64-bit) Graphics Platform: X11 ADDITIONAL INFORMATION This is reproducible even in a fresh user account, so it does not seem to be an issue of stale config files Bug is reproducible on Arch Linux with Okular versions 23.08.80 and 23.08.90 Git bisect points to 2fbab13e1f439f0dd2c6758d51c80e86ee49392a is the first bad commit commit 2fbab13e1f439f0dd2c6758d51c80e86ee49392a Author: Eugene Popov <popov895@ukr.net> Date: Thu Dec 22 22:50:55 2022 +0000 Port sidebar to QDockWidget With this MR, the sidebar can now be (if not locked): * docked to the left or right side * undocked and floated as an independent window * closed with the close button in the header BUG: 455013 interfaces/viewerinterface.h | 7 ++ part/part.cpp | 5 ++ part/part.h | 1 + part/sidebar.cpp | 5 ++ part/sidebar.h | 2 + shell/shell.cpp | 162 ++++++++++++++++++++++++++++++++++++++++++- shell/shell.h | 6 +- shell/shell.rc | 7 +- 8 files changed, 192 insertions(+), 3 deletions(-)
A possibly relevant merge request was started @ https://invent.kde.org/graphics/okular/-/merge_requests/718
Don't change the assignee please
I can also add that the variant that does not work seems to be the one that has been used before the duplicate was introduced, which has the effect that the default keybind (F7) is bound to the non-functioning variant of this action and therefore this default shortcut doesn't work (anymore) after an upgrade to this version of Okular.
Git commit a25e37e6f723106577ff9a2c1a6cf02dccf6b594 by Albert Astals Cid, on behalf of Eugene Popov. Committed on 23/04/2023 at 14:39. Pushed by aacid into branch 'master'. Reuse an existing action to toggle sidebar visibility M +10 -12 shell/shell.cpp M +1 -4 shell/shell.rc https://invent.kde.org/graphics/okular/commit/a25e37e6f723106577ff9a2c1a6cf02dccf6b594
Git commit 12f2228254d3831ed1131eb87e6e99302a230011 by Albert Astals Cid, on behalf of Eugene Popov. Committed on 23/04/2023 at 16:43. Pushed by aacid into branch 'release/23.04'. Reuse an existing action to toggle sidebar visibility (cherry picked from commit a25e37e6f723106577ff9a2c1a6cf02dccf6b594) M +10 -12 shell/shell.cpp M +1 -4 shell/shell.rc https://invent.kde.org/graphics/okular/commit/12f2228254d3831ed1131eb87e6e99302a230011
*** Bug 468882 has been marked as a duplicate of this bug. ***
*** Bug 469108 has been marked as a duplicate of this bug. ***