Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • Yakuake Yakuake
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
  • Bugzilla
    • Bugzilla
  • Merge requests 5
    • Merge requests 5
  • 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
  • Utilities
  • YakuakeYakuake
  • Merge requests
  • !27

Open
Created Sep 03, 2020 by Nicolas Höft@nicolashoeft
  • Report abuse
Report abuse

Fix issues with multi-monitor setups in X11

  • Overview 16
  • Commits 1
  • Changes 1

The logic that tries to determine the available screen rect and position the main window accordingly, seems to be broken on X11 with multi-monitor setups: The window is only visible on the first screen and when scaling is active the absolute height is not considered properly, especially with scaling being active.

Fix this by unifying wayland/X11 with regard to determining the available screen size, both use the dbus API to determine the available work area. This avoids having to duplicate the calculation in yakuake. Unfortunately, on X11 the returned size needs to be corrected for scaling, compared to Wayland.

As a further note, we would not request the screen size on each window toggle, but install a dbus listener for the signal. In this case we would be able to adjust the window live and with fewer dbus calls.

BUG: 275433

Edited Sep 03, 2020 by Nicolas Höft
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: bugfix/multihead-window-size