broulik abandoned this revision.
broulik added a comment.
Likely obsoleted by notification rewrite
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D3209
To: broulik, #plasma, mck182, mart
Cc: davidedmundson, mart, plasma-devel, jraleigh, GB_2, ragreen, Pitel,
davidedmundson added a comment.
What's the status of this?
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D3209
To: broulik, #plasma, mck182, mart
Cc: davidedmundson, mart, plasma-devel, ZrenBot, progwolff, lesliezhai,
ali-mohamed, jensreuterberg, abetts, s
mart added inline comments.
INLINE COMMENTS
> notificationshelper.cpp:154
> +
> +popup->show();
> +});
at this point it still thinks the size is 0,0
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D3209
To: broulik, #plasma, mck182, mart
Cc: mart,
mart accepted this revision.
mart added a reviewer: mart.
mart added a comment.
This revision is now accepted and ready to land.
tested locally on x11, it seems to still work correctly
INLINE COMMENTS
> notificationshelper.cpp:150
>
> -QTimer::singleShot(300, popup, &QWindow::show);
> +
broulik created this revision.
broulik added reviewers: Plasma, mck182.
broulik set the repository for this revision to rPLASMAWORKSPACE Plasma
Workspace.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
This ensures the d