<table><tr><td style="">luebking added a comment.<br />Restricted Application edited projects, added KWin; removed Plasma.
</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/D9699" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>The original implementation based the fullscreen status on the stack position of the window (ie. whenever a window would rise above the plain stack position of the FS window, it would loose the FS status, ie. top layer)<br />
The result was iirc that random notifications would not only show up but also de-fullscreen the window and also virtual desktop switches would constantly kill the FS state.</p>

<p>The group/transient thing was iirc a "make this simpler" thing (assuming the group would be sufficient again, since the switch from stack => active killed the major issues and annoyances that the stack selection brought)</p>

<p>Notice that the new patch does not cover the case of layered transient windows (which iirc was an issue with dolphin at the time, but pls don't nail me on that)</p>

<p>The code in place is certainly wrong, but none of the linked commits actually removed the group check that was used in <a href="https://phabricator.kde.org/R108:476ca65295bfb3f0d90f535d9930250a13a8b323" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/R108:476ca65295bfb3f0d90f535d9930250a13a8b323</a> (the other two commits predate that one)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>BRANCH</strong><div><div>fullscreen-transient-not-group</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9699" rel="noreferrer">https://phabricator.kde.org/D9699</a></div></div><br /><div><strong>To: </strong>graesslin, KWin, Plasma, romangg<br /><strong>Cc: </strong>luebking, romangg, plasma-devel, kwin, iodelay, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart<br /></div>