D25405: Fix Step infobrowser and examples localization

Albert Astals Cid noreply at phabricator.kde.org
Tue Nov 19 21:22:17 GMT 2019


aacid added inline comments.

INLINE COMMENTS

> infobrowser.cc:156
>          }
> -        QString fileName = QStringLiteral(":/objectinfo/%1.html").arg(className.toLower());
> +        QString fileName = QStandardPaths::locate(QStandardPaths::DataLocation, QStringLiteral("objinfo/%1/%2.html").arg(QLocale::system().uiLanguages().at(0), className.toLower()));
> +        if(fileName.isEmpty()) {

I think using KLocalizedString::languages() here makes more sense, i think QLocale uses a bit of a different naming system for "complex" stuff like pt_BR

REPOSITORY
  R341 Step

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D25405

To: yurchor, #kde_edu, #localization, cordlandwehr
Cc: aacid, cordlandwehr, kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20191119/2d961424/attachment.html>


More information about the kde-edu mailing list