ubuntu-touch-coreapps-reviewers team mailing list archive

Thread
Date

Re: [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/document-hub2 into lp:ubuntu-docviewer-app

To: Stefano Verzegnassi <stefano92.100@xxxxxxxxx>
From: Stefano Verzegnassi <stefano92.100@xxxxxxxxx>
Date: Thu, 26 Feb 2015 20:56:32 -0000
In-reply-to: <20150226202920.13317.2824.launchpad@ackee.canonical.com>
Reply-to: mp+251166@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Ok, got the issue.
In docviewer-application.cpp[1] we use the old code used to find out where the main qml file is located (from line 138).
Since in main.cpp[2] we set the ApplicationName (com.ubuntu.docviewer), the app looks for the qml file in the wrong folder:

"/usr/share/com.ubuntu.docviewer/qml/ubuntu-docviewer-app.qml"
instead of
"/usr/share/ubuntu-docviewer-app/qml/ubuntu-docviewer-app.qml"

I will fix it lately this night.

[1]: http://bazaar.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/document-hub2/view/head:/src/app/docviewer-application.cpp
[2]: http://bazaar.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/document-hub2/view/head:/src/app/main.cpp
-- 
https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/document-hub2/+merge/251166
Your team Ubuntu Document Viewer Developers is requested to review the proposed merge of lp:~verzegnassi-stefano/ubuntu-docviewer-app/document-hub2 into lp:ubuntu-docviewer-app.