<table><tr><td style="">rooty 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/D18848">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D18848#407838" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D18848#407838</a>, <a href="https://phabricator.kde.org/p/hein/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@hein</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>This is a good change, but it's more worth finding out why it doesn't work already. Looking at the code, when <tt style="background: #ebebeb; font-size: 13px;">KickoffItem</tt> is triggered it runs the <tt style="background: #ebebeb; font-size: 13px;">switchToInitial()</tt> function in FullRepresentation.qml, which already switches tabs and is also what sets the state back to <tt style="background: #ebebeb; font-size: 13px;">Normal</tt>. So setting the tab in the state shouldn't be necessary. Can you debug why setting <tt style="background: #ebebeb; font-size: 13px;">currentTab</tt> in <tt style="background: #ebebeb; font-size: 13px;">switchToInitial()</tt> apparently doesn't work?</p></div>
</blockquote>

<p>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.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18848">https://phabricator.kde.org/D18848</a></div></div><br /><div><strong>To: </strong>rooty, VDG, Plasma<br /><strong>Cc: </strong>hein, filipf, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>