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