Merge lp:~fcole90/ubuntu-docviewer-app/include-file-plugin into lp:ubuntu-docviewer-app/trunk
Status: | Superseded |
---|---|
Proposed branch: | lp:~fcole90/ubuntu-docviewer-app/include-file-plugin |
Merge into: | lp:ubuntu-docviewer-app/trunk |
Diff against target: |
496 lines (+434/-0) 11 files modified
CMakeLists.txt (+2/-0) plugins/CMakeLists.txt (+1/-0) plugins/file-qml-plugin/CMakeLists.txt (+45/-0) plugins/file-qml-plugin/backend.cpp (+22/-0) plugins/file-qml-plugin/backend.h (+43/-0) plugins/file-qml-plugin/docviewerFile.cpp (+105/-0) plugins/file-qml-plugin/docviewerFile.h (+56/-0) plugins/file-qml-plugin/fileReader.cpp (+82/-0) plugins/file-qml-plugin/fileReader.h (+39/-0) plugins/file-qml-plugin/main.pro (+38/-0) plugins/file-qml-plugin/qmldir (+1/-0) |
To merge this branch: | bzr merge lp:~fcole90/ubuntu-docviewer-app/include-file-plugin |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ubuntu Document Viewer Developers | Pending | ||
Review via email: mp+237543@code.launchpad.net |
This proposal has been superseded by a proposal from 2014-10-08.
Commit message
Adds the file plugin to the source tree.
Description of the change
Adds the file plugin to the source tree.
Note: while developing, to run the app on the desktop, or on any device, it requires an argument. The only way to pass that argument is by adding it as parameter to the .desktop(.in.in) file's Exec key. E.g., here's how I run it on my desktop:
Exec=@EXEC@ /home/dpm/
In the future, we will want to improve this, so that the app can be launched standalone. A workaround for now would be to ship a README.txt file that is loaded by default if no argument has been specified.
Note: autopilot tests still need to be adapted. I'll do that in this same merge proposal. In the meantime, the code changes in cmake and app+plugin can already be reviewed.