<table><tr><td style="">marten created this revision.<br />marten added a reviewer: Plasma.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />marten requested review of this revision.
</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/D22740">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Bug <a href="https://bugs.kde.org/show_bug.cgi?id=410190" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=410190</a> describes the inconsistent virtual desktop numbering between the Switch Window popup and the taskbar Move To Desktop list.  This patch fixes the Switch Window popup to use the more natural (to the user) 1-based desktop IDs.</p>

<p>According to virtualdesktopinfo.h, a desktop ID on Wayland is a QString while on X11 it is a uint.  Therefore using toString() here for correct formatting of the desktop ID on both X11 and Wayland.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Built plasma-workspace with this change, observed correct and consistent desktop numbering in both these places.  Not yet able to test on Wayland.</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/D22740">https://phabricator.kde.org/D22740</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>containmentactions/switchwindow/switch.cpp</div></div></div><br /><div><strong>To: </strong>marten, Plasma<br /><strong>Cc: </strong>plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>