[plasmashell] [Bug 363935] Vertical panel covers maximized windows with new Qt multiscreen support

[Martin Gräßlin via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=363935

Martin Gräßlin  changed:

   What|Removed |Added

 Resolution|--- |FIXED
  Latest Commit||b866bbe75e04584f0a54980c2c2
   ||1cba2eadc123b
 Status|UNCONFIRMED |RESOLVED
   Version Fixed In||5.7.0

--- Comment #4 from Martin Gräßlin  ---
Thanks for adding the patch! And sorry that we didn't notice earlier. In future
please post the patch to phabricator.kde.org for Project Plasma. That way it
lands in the inbox of the Plasma devs ;-)

In the mean time this problem was fixed with
https://quickgit.kde.org/?p=plasma-workspace.git=commit=b866bbe75e04584f0a54980c2c21cba2eadc123b
- a quite similar patch to yours

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 363935] Vertical panel covers maximized windows with new Qt multiscreen support

[Martin Gräßlin via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=363935

Martin Gräßlin  changed:

   What|Removed |Added

 CC||mgraess...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 363935] Vertical panel covers maximized windows with new Qt multiscreen support

[Daniel Lichtenberger via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=363935

--- Comment #3 from Daniel Lichtenberger  ---
Created attachment 99378
  --> https://bugs.kde.org/attachment.cgi?id=99378=edit
Patch: use QScreen::virtualGeometry in PanelView::updateStruts

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 363935] Vertical panel covers maximized windows with new Qt multiscreen support

[Daniel Lichtenberger via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=363935

Daniel Lichtenberger  changed:

   What|Removed |Added

 CC||plasma-b...@kde.org
  Component|Panel   |Multi-screen support

--- Comment #2 from Daniel Lichtenberger  ---
The issue is related to my multiscreen setup. I moved the panel to the right
border of my right screen, and the new code in PanelView::updateStruts fails to
take that into account: "wholeScreen" loads the current screen geometry, not
the whole (virtual) screen geometry, and "rightOffset" ends up being negative.

Against the advice in the comments, I changed the initiatlization of
wholeScreen to screen()->virtualGeometry() - it seems to work for me, but I
don't know about the reasons why virtualGeometry() was deemed unreliable.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 363935] Vertical panel covers maximized windows with new Qt multiscreen support

[Daniel Lichtenberger via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=363935

--- Comment #1 from Daniel Lichtenberger  ---
Created attachment 99364
  --> https://bugs.kde.org/attachment.cgi?id=99364=edit
Screenshot showing the issue

-- 
You are receiving this mail because:
You are watching all bug changes.