<table><tr><td style="">yuenlim updated this revision to Diff 11839.<br />yuenlim retitled this revision from "Keyboard support + improvements to QuickShare plasmoid" to "Basic keyboard support + fixes to QuickShare plasmoid".<br />yuenlim edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-ccf2pt3ydzo4y2e/" rel="noreferrer">(Show Details)</a><br />yuenlim added a reviewer: apol.
</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/D4803" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: #a6f3a6;">While working on basic keyboard support for QuickShare, I found and addressed some other issues. This patch thus addresses 3 items (let me know if I should make 3 separate diffs instead).<br />
<br />
1. Basic keyboard support: allow navigating the plasmoid paste menu with arrow keys, selection with enter key, and dismissal with escape key. No keyboard support yet for "secondary menus" eg the device picker in send to device. Also makes the plasmoid show the paste menu when activated by the global keyboard shortcut (previously does nothing).<br />
<br />
Note: I'm not 100% on how separation should work between the plasmoid's ui and the purpose plugin mechanism itself, but since the purpose plugin side also has UI elements I thought it made sense to have relevant keyboard support implemented there where applicable so the plasmoid inherits and modifies it. In theory this should also extend basic keyboard support to any other consumers of the purpose plugins.<br />
<br />
2. Fix an issue with the "Paste" action. For urls (when copying file/files), the Paste action was broken and didn't correctly pass the data to the sharedialog, so that the only way to share files previously was to use drag/drop. This patch fixes this.<br />
<br />
3. Fix a crash that happens when the plasmoid is activated with an empty clipboard. The crash actually happens in kquickcontrolsaddons.clipboard, which probably could use a patch so it handles empty-clipboard situations more gracefully, but here I have just avoided trying to access the clipboard if it is empty in the plasmoid. This might be related to bug 358268.</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R495 Purpose Library</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D4803?vs=11818&id=11839" rel="noreferrer">https://phabricator.kde.org/D4803?vs=11818&id=11839</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4803" rel="noreferrer">https://phabricator.kde.org/D4803</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/quick/AlternativesView.qml</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>yuenlim, apol<br /><strong>Cc: </strong>plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>