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

assign proper icons for temperature items #426

Open
2 tasks done
bam80 opened this issue Apr 21, 2024 · 7 comments
Open
2 tasks done

assign proper icons for temperature items #426

bam80 opened this issue Apr 21, 2024 · 7 comments
Labels
feature New feature or request

Comments

@bam80
Copy link

bam80 commented Apr 21, 2024

Has this issue been covered in the Wiki?

  • I have read the Wiki and my issue is not covered.

Is there an existing issue reported already?

  • I have searched the existing issues, it does not exist.

Describe the new feature you would like

Right now, selected temperature items are indistinguishable and messed up if there are many:
image
It would be cool if we could assign proper icons for them (they also could try to be assigned automatically).

@bam80 bam80 added the feature New feature or request label Apr 21, 2024
@corecoding
Copy link
Owner

Can you be more specific? What is "indistinguishable and messed up"? The screenshot looks correct to me. Are you asking for custom icons? If so, what would you have them be?

@bam80
Copy link
Author

bam80 commented Apr 23, 2024

Indistinguishable in the sense I can't just quickly look at them and say which temperature is which.
Currently, I have to remember that first temperature I've added to the panel is NVMe Sensor 1, the second is Sensor2, then Processor0 and Maximum temperatures are follow.

As you can imagine, this is not what the user wants.

It would be much more easier to catch for eyes if these temperature items would have icons corresponding to their categories: CPU, Disk, etc., as it is in the menu already:
image

@corecoding
Copy link
Owner

I would be glad to review a PR with specific temperature icons. I am not a graphics designer. It actually took me a while to source these icons.

@bam80
Copy link
Author

bam80 commented May 4, 2024

I think existing icons from the screenshot above would suit fine.

@Ticklezz
Copy link

Here is a GPU icon.
GPU ICON

@he-pennypacker
Copy link

I agree with the principle of this issue (temperatures are indistinguishable in the panel). I also appreciate that generating/sourcing additional icons (and managing their use) could be more overhead than desired.

As an alternative suggestion, how about enabling a mouseover/hover tooltip for the sensor names (e.g. processor 0, nvme 1, nvme 2 etc from the screenshot in the OP)?

@bam80
Copy link
Author

bam80 commented Dec 29, 2024

As an alternative suggestion, how about enabling a mouseover/hover tooltip for the sensor names (e.g. processor 0, nvme 1, nvme 2 etc from the screenshot in the OP)?

While it would be better than nothing, I think it's still not enough.
The idea is you should be able to easily distinguish the temperatures items just by looking at them, before you move mouse cursor to hover etc.

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

No branches or pull requests

4 participants