ubuntu-touch-coreapps-reviewers team mailing list archive

[Andrea Bernabei <andrea.bernabei@xxxxxxxxxxxxx>]

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.