gsmartcontrol icon indicating copy to clipboard operation
gsmartcontrol copied to clipboard

Self test execution bar color should change to RED when error are present (currently remains green) and title should state "test completed with errors"

Open ohadschn opened this issue 2 years ago • 3 comments

Version and Environment

  • GSmartControl version: 1.1.4 (latest released)
  • OS: Windows 10 22H2

Is your feature request related to a problem? Please describe. The color green is universally interpreted as "successful". I run self tests regularly and recently one of them failed. However I almost missed it as everything seemed normal at first glance.

  1. The reported result was simply "Test completed"
  2. The execution bar was green

It was only upon further inspection that I realized that the subtitle mentioned the test "completed with read failure".

Describe the solution you'd like Upon any test failure:

  1. The title should change to "test completed with errors"
  2. The execution bar color should change to RED

Additional context image

ohadschn avatar Jul 08 '23 09:07 ohadschn

Hi,

Unfortunately, the progress bar color is determined by the theme and it would be problematic to change it.

However, I agree that the failure should be more "in your face". Actually, there is a "warning" image displayed, but due to a bug in Windows package, it's missing.

So, fixing this bug requires:

  • Fixing the missing image in Windows package.
  • Maybe changing the color of the text to red to make it stand out.

ashaduri avatar Jul 15 '23 09:07 ashaduri

What about changing the test "test completed" to something like "TEST FAILED" in bold and/or red?

ohadschn avatar Jul 15 '23 09:07 ohadschn

What about changing the test "test completed" to something like "TEST FAILED" in bold and/or red?

Yes, this is also a good idea.

ashaduri avatar Jul 15 '23 10:07 ashaduri