<table><tr><td style="">ngraham created this revision.<br />ngraham added reviewers: Plasma, VDG.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />ngraham 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/D16897">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Implements the ability to close windows from the various non-default window switchers using the <kbd style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">F4</kbd> or <kbd style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">w</kbd> keys.</p>

<p>CCBUG: 308331</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Apply patch, Alt+tab to activate window switcher, hit <kbd style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">F4</kbd> or {w}. The current window closes.</p>

<p>This works for all but the "Informative" switcher, which is still broken for all keyboard navigation (unrelated to this patch; once that's fixed, this will start working).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R114 Plasma Addons</div></div></div><br /><div><strong>BRANCH</strong><div><div>close-windows-from-windowswitcher (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16897">https://phabricator.kde.org/D16897</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>windowswitchers/big_icons/contents/ui/main.qml<br />
windowswitchers/compact/contents/ui/main.qml<br />
windowswitchers/informative/contents/ui/main.qml<br />
windowswitchers/present_windows/contents/ui/main.qml<br />
windowswitchers/small_icons/contents/ui/main.qml<br />
windowswitchers/text/contents/ui/main.qml<br />
windowswitchers/thumbnails/contents/ui/main.qml</div></div></div><br /><div><strong>To: </strong>ngraham, Plasma, VDG<br /><strong>Cc: </strong>plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>