OSDN Git Service

Fix the break of repo hook color check.
authorYi-Ling Chuang <emilychuang@google.com>
Fri, 19 Apr 2019 05:52:25 +0000 (13:52 +0800)
committerYi-Ling Chuang <emilychuang@google.com>
Fri, 19 Apr 2019 06:02:35 +0000 (14:02 +0800)
Add the missing color for notification importance button

Fixes: 130850328
Test: manual
Change-Id: I6c070fad42d5fb0e6d5e3aa00d66b32708dde605

color-check-baseline.xml

index 02b00e2..e0a0206 100644 (file)
             column="5"/>
     </issue>
 
+    <issue
+        id="HardCodedColor"
+        severity="Error"
+        message="Avoid using hardcoded color"
+        category="Correctness"
+        priority="4"
+        summary="Using hardcoded color"
+        explanation="Hardcoded color values are bad because theme changes cannot be uniformly applied.Instead use the theme specific colors such as `?android:attr/textColorPrimary` in attributes.&#xA;This ensures that a theme change from a light to a dark theme can be uniformlyapplied across the app."
+        errorLine1="    android:color=&quot;@color/notification_importance_button_unselected&quot;"
+        errorLine2="    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
+        <location
+            file="res/drawable/button_border_unselected.xml"
+            line="21"
+            column="10"/>
+    </issue>
+
 </issues>