Bug 439707

Summary: Setting to add a keyboard shortcut does nothing
Product: [Plasma] plasmashell Reporter: Podagric <kde.podagric>
Component: Pager widgetAssignee: Eike Hein <hein>
Status: CONFIRMED ---    
Severity: minor CC: nate, plasma-bugs-null
Priority: NOR Keywords: usability
Version First Reported In: 5.22.3   
Target Milestone: 1.0   
Platform: Other   
OS: Other   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: screenshot

Description Podagric 2021-07-10 00:10:26 UTC
Created attachment 139978 [details]
screenshot

SUMMARY
Please see the screenshot attached

1. Navigation wraps around: It's already available in systemsettings and turning it on or off in this widget doesn't change anything. I think it can safely be removed from here as it is a global setting and independent of this widget

2. Keyboard shortcuts: setting a shortcut here doesn't do anything either. It could also be removed as actions like displaying the desktop, presenting windows or desktops already have shortcuts defined


SOFTWARE/OS VERSIONS
KDE Plasma Version: 5.22.3
KDE Frameworks Version: 5.83.0
Qt Version: 5.15.2
Graphics Platform: X11
Comment 1 Nate Graham 2021-07-30 22:27:00 UTC
The wrap section in the widget determines whether scrolling over the *widget* wraps. The setting in System Settings determines whether scrolling over the *desktop* wraps.

What do you mean "setting a shortcut here doesn't do anything either"? What kind of shortcut? What are expecting to happen?
Comment 2 Podagric 2021-07-30 23:19:21 UTC
> What do you mean "setting a shortcut here doesn't do anything either"? What kind of shortcut? What are expecting to happen?

I do not know either. the widget has an option to set a shortcut but it does nothing. then should it be removed?
Comment 3 Podagric 2021-07-30 23:21:27 UTC
> The wrap section in the widget determines whether scrolling over the *widget* wraps. The setting in System Settings determines whether scrolling over the *desktop* wraps.

it makes sense. I didn't really get it before because there wasn't any more information about how this worked.
Comment 4 Nate Graham 2021-08-02 17:03:47 UTC
All widgets have a "Keyboard Shortcut" section that lets you define a shortcut that will effectively simulate clicking the widget, because most widgets do something  when clicked. However in the case of the pager, it has no "default action" that will trigger when clicked; instead it overrides the click handler to be able to activate whichever vitrual desktop you happened to have clicked on. We can't simulate that, because we'd be mission cursor position information. Perhaps could add an action to simply move to the next desktop when the keyboard shortcut is activated.