Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-110934

Make our published QML code as free of qmllint errors/warnings as possible

XMLWordPrintable

    • 6dcaaca3a (dev), 6b2e05d9c (6.5), b9db5246b (6.4), 05c16d8e1 (dev), 5ee0ed790 (6.5), 814d221a3 (dev), 6d5447d2d (6.10)

      Suggestion

      For both our source code and our published examples, establish tools and processes (manual or automatic) to achieve the following:

      1. Identify all the warnings that are now fixable
      2. Avoid introducing new fixable warnings

       

      Why?

      Because our published examples are meant to be used as references, and because https://doc.qt.io/qt-6/qtquickcontrols2-customize.html instructs users to copy our code from $QTDIR/qml/QtQuick/Controls/Basic/

      Some possible outcomes of such processes are:

      • When a new example is added, its warnings are reviewed and the fixable ones are fixed earlier (so QTBUG-110831 would've been prevented or addressed earlier)
      • When a new pragma is introduced, we would have be alerted to the places in our existing code that needs this pragma (so https://codereview.qt-project.org/c/qt/qtdeclarative/+/459949 would've been part of the Qt 6.4.0 release)

        For Gerrit Dashboard: QTBUG-110934
        # Subject Branch Project Status CR V
        459949,2 Add `pragma ComponentBehavior: Bound` to QtQuick.Controls.Basic dev qt/qtdeclarative Status: ABANDONED -1 0
        460453,3 DialogButtonBox: Fix contentWidth warnings dev qt/qtdeclarative Status: MERGED +2 0
        460480,2 Drop useless extra QQmlContext from QQuickDialogButtonBoxPrivate dev qt/qtdeclarative Status: MERGED +2 0
        460843,3 Drop useless extra QQmlContext from QQuickDialogButtonBoxPrivate 6.4 qt/qtdeclarative Status: MERGED +2 0
        460844,2 Drop useless extra QQmlContext from QQuickDialogButtonBoxPrivate 6.5 qt/qtdeclarative Status: MERGED +2 0
        461584,2 DialogButtonBox: Fix contentWidth warnings 6.5 qt/qtdeclarative Status: MERGED +2 0
        461585,2 DialogButtonBox: Fix contentWidth warnings 6.4 qt/qtdeclarative Status: ABANDONED 0 0
        677246,6 quickwidgetgraphs example: Initialize GraphsView in C++ dev qt/qtgraphs Status: MERGED +2 0
        679798,2 quickwidgetgraphs example: Initialize GraphsView in C++ 6.10 qt/qtgraphs Status: MERGED +2 0

            qtqmlteam Qt Qml Team User
            skoh-qt Sze Howe Koh
            Votes:
            2 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated: