<table><tr><td style="">graesslin created this revision.<br />Restricted Application added a project: KWin.<br />Restricted Application added subscribers: kwin, plasma-devel.
</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/D7425" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Bringing another caption feature from X11 to Wayland. If we have<br />
multiple windows with the same caption, starting from the second window<br />
a suffix <number> is added.</p>

<p>E.g. if we have three windows with caption "foo", the naming is:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">foo</li>
<li class="remarkup-list-item">foo <2></li>
<li class="remarkup-list-item">foo <3></li>
</ul>

<p>The change tries to use as much shared code between the X11 and Wayland<br />
implementation. Unfortunately it's not possible to share completely as<br />
the X11 implementation does X11 specific things like editing the visible<br />
name.</p>

<p>By sharing the code the numbering also works cross windowing system.<br />
That is if a window is called "foo" on X11, a new window on Wayland with<br />
caption "foo" will get adjusted to "foo <2>" and vice versa.</p>

<p>The change also eliminates a duplicated signal for captionChanged in<br />
ShellClient (found by test case).</p>

<p>By using the shared implementation on X11 side a bug gets fixed which<br />
got introduced with the support of "unresponsive", this is no longer<br />
considered and the numbering still works even if there is a window which<br />
is unresponsive.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>New test case and manual testing</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>BRANCH</strong><div><div>caption-numbering-wayland</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7425" rel="noreferrer">https://phabricator.kde.org/D7425</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>abstract_client.cpp<br />
abstract_client.h<br />
autotests/integration/shell_client_test.cpp<br />
client.cpp<br />
client.h<br />
shell_client.cpp<br />
shell_client.h</div></div></div><br /><div><strong>To: </strong>graesslin, KWin, Plasma<br /><strong>Cc: </strong>plasma-devel, kwin, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart, lukas<br /></div>