Bug 328654

Summary:	Kickoff application navigation is broken
Product:	[Plasma] plasmashell	Reporter:	Alex Fiestas <afiestas>
Component:	general	Assignee:	Sebastian Kügler <sebas>
Status:	RESOLVED FIXED
Severity:	normal	CC:	mklapetek
Priority:	NOR
Version:	master
Target Milestone:	1.0
Platform:	unspecified
OS:	Linux
Latest Commit:	http://commits.kde.org/kde-workspace/3e034d3c1da4b3605b953b94065d1bd3233079fb	Version Fixed In:
Sentry Crash Report:
Bug Depends on:
Bug Blocks:	328586
Attachments:	video of the brokeness

Description Alex Fiestas 2013-12-11 11:06:19 UTC

See attached video for all the issues

Reproducible: Always

Steps to Reproduce:
Open kickoff, try to execute an app

Comment 1 Alex Fiestas 2013-12-11 11:06:56 UTC

Created attachment 84030 [details]
video of the brokeness

Comment 2 Martin Klapetek 2013-12-20 15:28:31 UTC

Git commit 67a3129f74ba466fde46cc92233b15d5f747fa02 by Martin Klapetek.
Committed on 20/12/2013 at 15:25.
Pushed by mklapetek into branch 'master'.

[Kickoff] Disable layoutChange signals; fixes Kickoff's navigation

The breadcrumb issues still remain, apparently it happens because
ListModel is unable to store QVariant(QModelIndex).

M  +10   -3    plasma/desktop/applets/kickoff/core/applicationmodel.cpp

http://commits.kde.org/kde-workspace/67a3129f74ba466fde46cc92233b15d5f747fa02

Comment 3 Martin Klapetek 2013-12-20 19:29:00 UTC

Git commit 3e034d3c1da4b3605b953b94065d1bd3233079fb by Martin Klapetek.
Committed on 20/12/2013 at 19:28.
Pushed by mklapetek into branch 'master'.

[Kickoff] Fix breadcrumb navigation

Workaround for ListModel not being able to store QVariant(QModelIndex)
values. Now it's stored in another model property. Makes the breadcrumb
navigation work again.

M  +5    -2    plasma/desktop/applets/kickoff/package/contents/ui/ApplicationsView.qml
M  +5    -4    plasma/desktop/applets/kickoff/package/contents/ui/Breadcrumb.qml

http://commits.kde.org/kde-workspace/3e034d3c1da4b3605b953b94065d1bd3233079fb