Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
ohadschn opened this issue Jul 8, 2023 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@ohadschn
Copy link

ohadschn commented Jul 8, 2023

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 ohadschn added the enhancement New feature or request label Jul 8, 2023
@ashaduri
Copy link
Owner

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.

@ohadschn
Copy link
Author

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

@ashaduri
Copy link
Owner

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

Yes, this is also a good idea.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants