D14288: Initial version of Clazy analyzer plugin

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Fri Jul 27 11:47:42 BST 2018


kossebau added inline comments.

INLINE COMMENTS

> commandlinewidget.cpp:34
> +    m_ui->setupUi(this);
> +    m_ui->cmdEdit->setFontFamily(QStringLiteral("Monospace"));
> +

See D14424 <https://phabricator.kde.org/D14424> for similar UI where proposal is to  use QFontDatabase::systemFont(QFontDatabase::FixedFont) instead.

> commandlinewidget.ui:13
> +  </property>
> +  <property name="windowTitle">
> +   <string>Form</string>

Unused property injected by Qt designer, please remove (also avoids string to appear for translators).

> commandlinewidget.ui:50
> +      <item>
> +       <widget class="QTextEdit" name="cmdEdit">
> +        <property name="readOnly">

See D14424 <https://phabricator.kde.org/D14424> for similar UI where proposal is to use QPlainTextEdit instead.

> globalconfigpage.ui:13
> +  </property>
> +  <property name="windowTitle">
> +   <string>Form</string>

Unused property injected by Qt designer, please remove (also avoids string to appear for translators).

> projectconfigpage.ui:13
> +  </property>
> +  <property name="windowTitle">
> +   <string>Form</string>

Unused property injected by Qt designer, please remove (also avoids string to appear for translators).

REPOSITORY
  R32 KDevelop

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

To: antonanikin, #kdevelop
Cc: kossebau, kdevelop-devel, antismap, iodelay, vbspam, njensen, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20180727/7b949c17/attachment-0001.html>


More information about the KDevelop-devel mailing list