D18848: [Kickoff] Return Kickoff to Favorites page after running a search and closing it
Krešimir Čohar
noreply at phabricator.kde.org
Fri Feb 8 12:51:55 GMT 2019
rooty added a comment.
In D18848#407838 <https://phabricator.kde.org/D18848#407838>, @hein wrote:
> This is a good change, but it's more worth finding out why it doesn't work already. Looking at the code, when `KickoffItem` is triggered it runs the `switchToInitial()` function in FullRepresentation.qml, which already switches tabs and is also what sets the state back to `Normal`. So setting the tab in the state shouldn't be necessary. Can you debug why setting `currentTab` in `switchToInitial()` apparently doesn't work?
I think it might be because of the switch from the state "Search" to the state "Normal", because this doesn't happen if the arrow keys are used to navigate to the right of Favorites, only when Search is invoked. There's a "currentTab: searchPage" (line 738) setting in the state "Search" that probably gets stuck. I didn't want to deal with changing "Search" because in my experience, Kickoff's search view can be pretty temperamental. This seemed like the safest course of action.
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D18848
To: rooty, #vdg, #plasma
Cc: hein, filipf, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190208/52e9077a/attachment.html>
More information about the Plasma-devel
mailing list