Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • P Plasma Workspace
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 21
    • Issues 21
    • List
    • Boards
    • Service Desk
    • Milestones
  • Bugzilla
    • Bugzilla
  • Merge requests 108
    • Merge requests 108
  • 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
  • PlasmaPlasma
  • Plasma Workspace
  • Merge requests
  • !1761

[System Tray] Emit pressed signal on compact representation in popup, too

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Kai Uwe Broulik requested to merge work/kbroulik/systemtray-emit-pressed into master May 23, 2022
  • Overview 2
  • Commits 1
  • Pipelines 4
  • Changes 3

Some actions, such as middle click to mute, trigger only on press. Some applets also need a pressed-click because of some legacy autoclose on focus change bug.

Unfortunately, QML does not disambiguate the "pressed" property and "pressed(mouse)" signal with argument, so this has to be done awkwardly through QMetaObject on C++ side.

BUG: 426646 BUG: 452893 FIXED-IN: 5.25.0


  • Middle click to toggle mute now works from popup
  • Opening notification applet from history now works
  • Didn't notice any other issues, could use some more real-world testing

@kmaterka

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: work/kbroulik/systemtray-emit-pressed