[Merge] lp:~vrruiz/gallery-app/i18n-strings into lp:gallery-app

Olivier Tilloy olivier.tilloy at canonical.com
Wed Sep 17 11:34:27 UTC 2014


The call to .arg([count]) is going to generate a QML warning in the single case where there is no placeholder to replace:

    QString::arg: Argument missing: …

Not sure how to address that in an elegant way. A clumsy way of doing that would be something along the lines of:

    title: {
        var t = i18n.tr("Delete a photo", "Delete %1 photos", organicEventView.selection.selectedCount)
        if (t.indexOf("%1") !== -1) {
            t = t.arg(organicEventView.selection.selectedCount)
        }
        return t
    }

But that looks rather ugly.
-- 
https://code.launchpad.net/~vrruiz/gallery-app/i18n-strings/+merge/234671
Your team Ubuntu Phablet Team is subscribed to branch lp:gallery-app.



More information about the Ubuntu-reviews mailing list