Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • Dolphin Dolphin
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 10
    • Issues 10
    • List
    • Boards
    • Service Desk
    • Milestones
  • Bugzilla
    • Bugzilla
  • Merge requests 38
    • Merge requests 38
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • SystemSystem
  • DolphinDolphin
  • Merge requests
  • !486

Make space shortcut for selection mode view-local instead of global

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Felix Ernst requested to merge felixernst/dolphin:fix_space_bar_hijacking into master Jan 02, 2023
  • Overview 7
  • Commits 2
  • Pipelines 5
  • Changes 5

Before this commit, the "Space" keyboard shortcut was bound to triggering selection mode by default. After this commit, pressing "Space" will only trigger selection mode when the file view area has keyboard focus.

Pros:

  • Other buttons in the UI can be triggered with Space once again just like it is expected from an accessibility point of view (https://phabricator.kde.org/T11520#203511).
  • "Type-ahead" searching works once more when typing the space char for file names containing such a space char.

Cons:

  • "Space" can no longer be used to add the currently underlined item to the selection. Instead "Ctrl+Space" needs to be used. (However, this is the current status anyway unless a user has manually unbound "Space" as a shortcut from Selection Mode.)
  • The Selection Mode action will no longer show "Space" as its shortcut in menus.

Overall, I see solutions to all of these problems, but they seem over-engineered for the issues they are trying to solve, so I believe this somewhat small commit is the best solution for now.

BUG: 458282
BUG: 458281
CCBUG: 463048
FIXED-IN: 23.04

Edited Jan 02, 2023 by Felix Ernst
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix_space_bar_hijacking