qmllint: Quick plugin: Recognize more acceptable colorsHEADdev

QColor::fromString does not care about the casing of named colors, and
conseqently neither does the QML engine. Unconditionally lower case the
color in the check to accomadate it.

Pick-to: 6.10
Change-Id: I35aa4162bcd8682bbea86776c9929e00f891cdb9
Reviewed-by: Sami Shalayel <[email protected]>
Reviewed-by: Olivier De Cannière <[email protected]>

1 files changed, 5 insertions, 1 deletions

diff --git a/src/plugins/qmllint/quick/quicklintplugin.cpp b/src/plugins/qmllint/quick/quicklintplugin.cpp
index 4b02e49d41..2876477d95 100644
--- a/src/plugins/qmllint/quick/quicklintplugin.cpp
+++ b/src/plugins/qmllint/quick/quicklintplugin.cpp
@@ -706,7 +706,11 @@ void ColorValidatorPass::onBinding(const QQmlSA::Element &element, const QString
     if (!propertyType || propertyType != m_colorType)
         return;
 
-    const QString colorName = binding.stringValue();
+    QString colorName = binding.stringValue();
+    // for "named" colors, QColor::fromString does not care about
+    // the case
+    if (!colorName.startsWith(u'#'))
+        colorName = std::move(colorName).toLower();
     if (s_hexPattern.match(colorName).hasMatch())
         return;