report/log: accessible pass/fail/skip colors

yanne · yanne · commit 65b018c451e9 · 2020-10-14T11:06:41.000+03:00
diff --git a/atest/robot/cli/runner/report_background.robot b/atest/robot/cli/runner/report_background.robot
@@ -20,7 +20,7 @@ Invalid Colors
 
 *** Keywords ***
 Report should have correct background
-    [Arguments]  ${opt}  ${pass}=#9e9  ${fail}=#f66  ${skip}=#ec971f
+    [Arguments]  ${opt}  ${pass}=#9e9  ${fail}=#f66  ${skip}=#fed84f
     Run Tests  ${opt} --report rep.html  misc/pass_and_fail.robot
     ${report} =  Get File  ${OUTDIR}/rep.html
     Should Contain  ${report}  "background":{"fail":"${fail}","pass":"${pass}","skip":"${skip}"},
diff --git a/src/robot/conf/settings.py b/src/robot/conf/settings.py
@@ -54,7 +54,7 @@ class _BaseSettings(object):
                  'LogTitle'         : ('logtitle', None),
                  'ReportTitle'      : ('reporttitle', None),
                  'ReportBackground' : ('reportbackground',
-                                       ('#9e9', '#f66', '#ec971f')),
+                                       ('#9e9', '#f66', '#fed84f')),
                  'SuiteStatLevel'   : ('suitestatlevel', -1),
                  'TagStatInclude'   : ('tagstatinclude', []),
                  'TagStatExclude'   : ('tagstatexclude', []),
@@ -245,7 +245,7 @@ def _process_report_background(self, colors):
             raise DataError("Invalid report background colors '%s'." % colors)
         colors = colors.split(':')
         if len(colors) == 2:
-            return colors[0], colors[1], '#ec971f'
+            return colors[0], colors[1], '#fed84f'
         return tuple(colors)
 
     def _process_tag_stat_combine(self, pattern):
diff --git a/src/robot/htmldata/rebot/common.css b/src/robot/htmldata/rebot/common.css
@@ -80,7 +80,7 @@ h2 {
     color: #393 !important;
 }
 .skip {
-    color: #ec971f !important;
+    color: #fed84f !important;
     font-weight: bold;
 }
 .label {
@@ -93,7 +93,7 @@ h2 {
     border-radius: 3px;
 }
 .label.skip {
-    background-color: #ec971f;
+    background-color: #fed84f;
     color: white !important;
 }
 .label.debug, .label.trace, .label.error, .label.keyword {
@@ -104,13 +104,13 @@ h2 {
     font-weight: bold;
 }
 .label.error, .label.fail {
-    background-color: #d9534f;
+    background-color: #ee5919;
 }
 .label.pass {
-    background-color: #5cb85c;
+    background-color: #97bd61;
 }
 .label.warn {
-    background-color: #ec971f;
+    background-color: #b62109;
 }
 /* Top right header */
 #top-right-header {
@@ -206,13 +206,13 @@ th.stats-col-graph:hover {
     height: 100%;
 }
 .fail-bar {
-    background: #f33;
+    background: #ee5919;
 }
 .pass-bar {
-    background: #1d4;
+    background: #97bd61;
 }
 .skip-bar {
-    background:  #ec971f;
+    background:  #fed84f;
 }
 /* Tablesorter - adapted from provided Blue Skin */
 .tablesorter-header {

Original file line number	Diff line number	Diff line change
`@@ -80,7 +80,7 @@ h2 {`
`80`	`80`	`color: #393 !important;`
`81`	`81`	`}`
`82`	`82`	`.skip {`
`83`		`- color: #ec971f !important;`
	`83`	`+ color: #fed84f !important;`
`84`	`84`	`font-weight: bold;`
`85`	`85`	`}`
`86`	`86`	`.label {`
`@@ -93,7 +93,7 @@ h2 {`
`93`	`93`	`border-radius: 3px;`
`94`	`94`	`}`
`95`	`95`	`.label.skip {`
`96`		`- background-color: #ec971f;`
	`96`	`+ background-color: #fed84f;`
`97`	`97`	`color: white !important;`
`98`	`98`	`}`
`99`	`99`	`.label.debug, .label.trace, .label.error, .label.keyword {`
`@@ -104,13 +104,13 @@ h2 {`
`104`	`104`	`font-weight: bold;`
`105`	`105`	`}`
`106`	`106`	`.label.error, .label.fail {`
`107`		`- background-color: #d9534f;`
	`107`	`+ background-color: #ee5919;`
`108`	`108`	`}`
`109`	`109`	`.label.pass {`
`110`		`- background-color: #5cb85c;`
	`110`	`+ background-color: #97bd61;`
`111`	`111`	`}`
`112`	`112`	`.label.warn {`
`113`		`- background-color: #ec971f;`
	`113`	`+ background-color: #b62109;`
`114`	`114`	`}`
`115`	`115`	`/* Top right header */`
`116`	`116`	`#top-right-header {`
`@@ -206,13 +206,13 @@ th.stats-col-graph:hover {`
`206`	`206`	`height: 100%;`
`207`	`207`	`}`
`208`	`208`	`.fail-bar {`
`209`		`- background: #f33;`
	`209`	`+ background: #ee5919;`
`210`	`210`	`}`
`211`	`211`	`.pass-bar {`
`212`		`- background: #1d4;`
	`212`	`+ background: #97bd61;`
`213`	`213`	`}`
`214`	`214`	`.skip-bar {`
`215`		`- background: #ec971f;`
	`215`	`+ background: #fed84f;`
`216`	`216`	`}`
`217`	`217`	`/* Tablesorter - adapted from provided Blue Skin */`
`218`	`218`	`.tablesorter-header {`