raiguard / ModernGadgets

Sleek, minimalist, information-dense gadgets for the modern desktop. Inspired by the AddGadgets.com system monitoring gadgets. Built on the Rainmeter platform.
MIT License
300 stars 32 forks source link

Not enough spacing when using >10 core processor #265

Open Nightliss opened 3 years ago

Nightliss commented 3 years ago

Hey rai,

thanks for updating the suite for HWInfo v7 :)

I've got an issue with spacing with core name, temp and voltage: image

As I imagine making the widget wider would not look good so maybe is there a way to parse the values to drop decimal separator?

SilverAzide commented 3 years ago

There was a recent change to HWiNFO in which it started reporting temps to one decimal place for Ryzen processors (Intel is still no decimals). It is -- unfortunately -- not a simple fix.

raiguard commented 3 years ago

In the meantime, you could disable core voltages or something. Alternatively, you can increase the gadget width by setting bgWidth=200 (default is 150) in GlobalSettings.inc. Just saw that you didn't want to do that. I will look into the decimals thing when I get some time.

Nightliss commented 3 years ago

Not that I didn't want that, just thought it would not scale well with wider container. I didn't find the variable you have mentioned in that file, but I did find gadgetBgWidth in StyleSheet.inc. Changing it to 165 makes it look great. It also changes other gadgets' width so it matches :) Thanks!

image

raiguard commented 3 years ago

Yeah, bgWidth does not exist in GlobalSettings, but adding it there will override what is in StyleSheet, and will be preserved across versions.

Y0dle commented 2 years ago

I noticed this issue as well, another workaround for now: you can change the number of decimals in HWiNFO's sensor settings.

raiguard commented 2 years ago

This will be addressed in the CPU meter rewrite to fix #278. I will be removing the "core" part of each label which will free up a ton of space for the stuff in the middle.