D22045: [Notifications] Make popup placement more resilient

Kai Uwe Broulik noreply at phabricator.kde.org
Sun Jun 23 15:29:16 BST 2019


broulik created this revision.
broulik added reviewers: Plasma, VDG.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  The popup width is fixed, so don't rely on the actual dialog window width which might be bollocks.
  More importantly, place the popups immediately and before they are shown so they show up correctly and don't fly all over the place.

TEST PLAN
  The first popup might still fly in from outside the screen or something but it's a significant improvement on wayland at least.
  
  (Don't really know which bugreport on bugzilla is that particular issue, there's some others with notification popups overlapping and appearing in random places but might be unrelated)

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D22045

AFFECTED FILES
  applets/notifications/package/contents/ui/global/Globals.qml

To: broulik, #plasma, #vdg
Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190623/e2ec538f/attachment.html>


More information about the Plasma-devel mailing list