<table><tr><td style="">hein added a comment.
</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/D15410">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D15410#323613" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D15410#323613</a>, <a href="https://phabricator.kde.org/p/graesslin/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@graesslin</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I just want to point out that such applications are not ICCCM compliant: This property must be present when the window leaves the Withdrawn state and may be changed only while the window is in the Withdrawn state. Window managers may examine the property only when they start up and when the window leaves the Withdrawn state, but there should be no need for a client to change its state dynamically.</p>

<p>KWin doesn't support updates to wm_class and e.g. window rules are broken.</p></div>
</blockquote>

<p>Preaching to the choir I'm afraid :(. I consider this bad application behavior and called it a concession to badly written apps on Bugzilla.</p>

<p>Unfortunately the upstream bug report has been ignored however, and there's a large installed base as this is long-standing LibreOffice behavior. I am eventually considering a backport to the LTS branch of this.</p>

<p>Also, on Wayland the appid can change at runtime in theory (and people hooked this up in Mutter and Weston, so I guess it's been encountered), and the affected code is mostly generic over the two windowing systems.</p>

<p>Ah well. At least this takes the shape of a generic "allow hiding tasks matching some id" feature. It's in practice only needed to handle this app misbehavior, but at least it's not brittle weird code to catch ckass changes explicitly.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15410">https://phabricator.kde.org/D15410</a></div></div><br /><div><strong>To: </strong>hein, davidedmundson, broulik, ngraham<br /><strong>Cc: </strong>graesslin, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>