on xenial help sign in
Slightly better comment
commit2ea870526cb22d20e08192bbd45e30a61ab7e397 authorGerry Boland <gerry.boland@canonical.com> dateMon 9 Sep 2013, 11:11 (13 years ago) parenta2f764e
1 file changed+6−3
Components/ApplicationManagerWrapper.qml+6−3
Components/ApplicationManagerWrapper.qml+6−3
@@ -67,7 +67,8 @@ Item {
6767 function activateApplication(desktopFile, argument) {
6868 var appId;
6969
70− // HACK: might be called with appId, but mostly with desktopFile
70+ // HACK: Applications identified sometimes with with appId, but mostly with desktopFile.
71+ // TODO: convert entire shell to use appId only.
7172 if (desktopFile.indexOf(".desktop") >= 0) {
7273 appId = desktopFileToAppId(desktopFile);
7374 } else {
@@ -91,7 +92,8 @@ Item {
9192 function stopApplication(application) {
9293 var appId;
9394
94− // HACK: might be called with appId, or else with Application object
95+ // HACK: Applications identified sometimes with with appId, but mostly with desktopFile.
96+ // TODO: convert entire shell to use appId only.
9597 if (typeof application == "string") {
9698 appId = application;
9799 } else {
@@ -126,7 +128,8 @@ Item {
126128 function getApplicationFromDesktopFile(desktopFile, stage) {
127129 var appId;
128130
129− // HACK: might be called with appId, but mostly with desktopFile
131+ // HACK: Applications identified sometimes with with appId, but mostly with desktopFile.
132+ // TODO: convert entire shell to use appId only.
130133 if (desktopFile.indexOf(".desktop") >= 0) {
131134 appId = desktopFileToAppId(desktopFile);
132135 } else {