markg added inline comments.

INLINE COMMENTS

> tooltipdialog.cpp:116-118
> +    if (KWindowSystem::isPlatformX11()) {
>          flags = flags | Qt::BypassWindowManagerHint;
>      }

Isn't this redundant anyway?
It's being set in the constructor as well.

The constructor only deviates in initial flags (it doesn't explicitly set 
Qt::WindowDoesNotAcceptFocus | Qt::WindowStaysOnTopHint). Hmm, if you add these 
flags to the constructor (don't know if that is allowed) then you can just call 
"return Dialog::event(e);" after the #endif.

REPOSITORY
  R242 Plasma Frameworks

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: broulik, #plasma, davidedmundson
Cc: markg, plasma-devel, #frameworks, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, andreaska, sebas

Reply via email to