Uploaded image for project: 'Clover'
  1. Clover
  2. CLOV-1817

Fix colors of test passed / failed / error icons in Eclipse

    • Icon: Suggestion Suggestion
    • Resolution: Fixed
    • 4.1.0
    • CEP Plugin
    • None
    • Our product teams collect and evaluate feedback from a number of different sources. To learn more about how we use customer feedback in the planning process, check out our new feature policy.

      Currently we use dark blue color for a failed test. It's not contrast enough compared to the green one.

      Furthermore, we use a red background for a "FAIL" status label, which is not consistent. Also, a green background used for a "PASS" label is different from the green dot in the icon.

      Current colors used: green-pass, blue-fail, red-error are the same as in Eclipse's JUnit view.

      Consider also changing a dot to three different figures to make it accessible for color-blinded people. For instance:

      • light green circle - pass
      • dark blue triangle - fail
      • red square - error

            [CLOV-1817] Fix colors of test passed / failed / error icons in Eclipse

            Katherine Yabut made changes -
            Workflow Original: JAC Suggestion Workflow [ 3342030 ] New: JAC Suggestion Workflow 3 [ 3584597 ]
            Status Original: RESOLVED [ 5 ] New: Closed [ 6 ]
            Michael Andreacchio made changes -
            Workflow Original: New Clover Workflow [ 971561 ] New: JAC Suggestion Workflow [ 3342030 ]
            Assignee Original: Marek Parfianowicz [ mparfianowicz ]
            Issue Type Original: Improvement [ 4 ] New: Suggestion [ 10000 ]
            Status Original: Closed [ 6 ] New: Resolved [ 5 ]
            Marek Parfianowicz made changes -
            Remote Link Original: This issue links to "Page (Extranet)" [ 135622 ]
            Marek Parfianowicz made changes -
            Remote Link New: This issue links to "Page (Atlassian Documentation)" [ 136515 ]
            Marek Parfianowicz made changes -
            Resolution New: Fixed [ 1 ]
            Status Original: In Progress [ 3 ] New: Closed [ 6 ]

            Marek Parfianowicz added a comment - - edited

            Not changing the green background for the "PASS" label - new green dot (it's a derivative of the ADG base green color - with higher saturation) is nearly identical as the text background.

            I'm not changing the red background for "FAIL" label. A reason is that we use red-green color scheme on the Clover Dashboard for test passes/failures. Changing to blue would make it inconsistent. Furthermore, blue-gray scheme is used for covered-uncovered bars on a dashboard.

            So we'll stay with a blue triangle in the icon (to be consistent with a blue test fail icon in the JUnit view) and at the same time with a red "FAIL" text (to be consistent with a Clover dashboard as well as code coloring in editor tabs).

            Marek Parfianowicz added a comment - - edited Not changing the green background for the "PASS" label - new green dot (it's a derivative of the ADG base green color - with higher saturation) is nearly identical as the text background. I'm not changing the red background for "FAIL" label. A reason is that we use red-green color scheme on the Clover Dashboard for test passes/failures. Changing to blue would make it inconsistent. Furthermore, blue-gray scheme is used for covered-uncovered bars on a dashboard. So we'll stay with a blue triangle in the icon (to be consistent with a blue test fail icon in the JUnit view) and at the same time with a red "FAIL" text (to be consistent with a Clover dashboard as well as code coloring in editor tabs).
            Marek Parfianowicz made changes -
            Status Original: Open [ 1 ] New: In Progress [ 3 ]
            Marek Parfianowicz made changes -
            Fix Version/s New: 4.1.0 [ 29105 ]
            Fix Version/s Original: long term roadmap [ 54394 ]
            Marek Parfianowicz made changes -
            Sprint New: 4.1.0 Sprint 3 [ 2710 ]
            Marek Parfianowicz made changes -
            Story Points Original: 1 New: 0.5

              Unassigned Unassigned
              mparfianowicz Marek Parfianowicz
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: