[Differential] [Changed Subscribers] D3822: use a native event filter to notice the screen was swapped

2017-01-01 Thread broulik (Kai Uwe Broulik)
broulik added inline comments. INLINE COMMENTS > screenpool.cpp:37 > { > +qApp->installNativeEventFilter(this); > + No need to install if not on X? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D3822 EMAIL PREFERENCES

[Differential] [Changed Subscribers] D3822: use a native event filter to notice the screen was swapped

2016-12-28 Thread pmuralidharan (PrasannaKumar Muralidharan)
pmuralidharan added inline comments. INLINE COMMENTS > screenpool.cpp:203 > +} > + > + Wondering whether returning false always is correct. It feels wrong but I may be missing something. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D3822 EMAIL PREFERENCES