try proper order of corona()->availableScreenRect()

Authored by mart on Mar 9 2017, 5:31 PM.

Description

try proper order of corona()->availableScreenRect()

Summary:
connect to Plasma::Corona::availableScreenRectChanged
instead of using kscreen, this way we should be more sure
that signal is emitted when it's safe to call
corona()->availableScreenRect() as the change of rect
may happen after a screen removal, so may mean
accessing a desktop view while being deleted
BUG:377298

Test Plan:
notifications still pop up in the proper place,
couldn't reproduce the crash neither before nor after tough

Reviewers: Plasma, davidedmundson

Reviewed By: Plasma, davidedmundson

Subscribers: plasma-devel

Tags: Plasma

Differential Revision: https://phabricator.kde.org/D4991

Details

Committed
martMar 10 2017, 11:16 AM
Reviewer
Plasma
Differential Revision
D4991: try proper order of corona()->availableScreenRect()
Parents
R120:8028a500e3b2: Ignore NoDisplay=true .desktop entries when matching by Name.
Branches
Unknown
Tags
Unknown