small comment Diff comments:
> === modified file 'src/app/main.cpp' > --- src/app/main.cpp 2015-03-23 17:46:34 +0000 > +++ src/app/main.cpp 2016-01-25 11:26:52 +0000 > @@ -174,12 +174,18 @@ > qDebug() << "Trying to load QML from:" << path + > "/qml/ubuntu-terminal-app.qml"; > if (fi.exists()) { > qmlfile = path + "/qml/ubuntu-terminal-app.qml"; > - keyboardLayouts << getProfileFromDir(path + > "/qml/KeyboardRows/Layouts/"); > break; > } > } > } > > + // Look for default layouts > + QDir keybLayoutDir = QFileInfo(qmlfile).dir(); > + if (keybLayoutDir.cd("KeyboardRows/Layouts")) { > + keyboardLayouts << getProfileFromDir(keybLayoutDir.canonicalPath() + > "/"); > + } maybe add a debug string here? > + > + // Look for user-defined layouts > QStringList configLocations = > QStandardPaths::standardLocations(QStandardPaths::ConfigLocation); > foreach (const QString &path, configLocations) { > QString fullPath = path + "/com.ubuntu.terminal/Layouts/"; -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-terminal-app/layouts-q-argument/+merge/283786 Your team Ubuntu Terminal Developers is requested to review the proposed merge of lp:~verzegnassi-stefano/ubuntu-terminal-app/layouts-q-argument into lp:ubuntu-terminal-app. -- Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers More help : https://help.launchpad.net/ListHelp