<table><tr><td style="">mvourlakos added inline comments.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D15814">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D15814#inline-85625">View Inline</a><span style="color: #4b4d51; font-weight: bold;">davidedmundson</span> wrote in <span style="color: #4b4d51; font-weight: bold;">dialog.cpp:209</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">How does parentWindow->mask relate to being a dock or not?</p>

<p style="padding: 0; margin: 8px;">Also we need to be a bit careful on names.<br />
_NET_WM_WINDOW_TYPE_DOCK is all panels, including the standard plasma one.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">The idea was first introduced at popupPlacement() in dialog.cpp. <br />
In order to distinguish docks from panels.</p>

<p style="padding: 0; margin: 8px;">It is based on the assumption that by design plasma panels do not use mask() because their shadows are drawn external from the window and most of the docks (e.g. Plank,Latte) are using masking for almost all of their operations.</p>

<p style="padding: 0; margin: 8px;">Do you believe there is a better way to distinguish between docks and panels ?</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15814">https://phabricator.kde.org/D15814</a></div></div><br /><div><strong>To: </strong>mvourlakos, Plasma, broulik, davidedmundson<br /><strong>Cc: </strong>abetts, broulik, ngraham, kde-frameworks-devel, michaelh, bruns<br /></div>