<table><tr><td style="">davidedmundson updated this revision to Diff 19461.<br />davidedmundson 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/D7780" rel="noreferrer">View Revision</a></tr></table><br /><div><div><div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">
In the old code sometimes the root item would mostly have focus,
sometimes the relevant object. The root item would then perform key
handling for everyone and dispatch events to the relevant place.

Because focus kept ending up in the wrong place and faked, this broke
accessibility (as well as being quite messy).

In the new code items all handle their own key handling. Anything
unhandled propagates to the root view which then Keys.forwardTo the
relevant items.

Behaviour is mostly identical (except I changed escape in the
Applications view)

Change is too massive for 5.11, will delay for master.</pre></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D7780?vs=19436&id=19461" rel="noreferrer">https://phabricator.kde.org/D7780?vs=19436&id=19461</a></div></div><br /><div><strong>BRANCH</strong><div><div>backup</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7780" rel="noreferrer">https://phabricator.kde.org/D7780</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applets/kickoff/package/contents/ui/ApplicationsView.qml<br />
applets/kickoff/package/contents/ui/BaseView.qml<br />
applets/kickoff/package/contents/ui/ComputerView.qml<br />
applets/kickoff/package/contents/ui/FavoritesView.qml<br />
applets/kickoff/package/contents/ui/FullRepresentation.qml<br />
applets/kickoff/package/contents/ui/Header.qml<br />
applets/kickoff/package/contents/ui/KickoffItem.qml<br />
applets/kickoff/package/contents/ui/LeaveView.qml<br />
applets/kickoff/package/contents/ui/OftenUsedView.qml<br />
applets/kickoff/package/contents/ui/RecentlyUsedView.qml<br />
applets/kickoff/package/contents/ui/SearchView.qml</div></div></div><br /><div><strong>To: </strong>davidedmundson, Plasma<br /><strong>Cc: </strong>plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>