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

Input box missing - description string doesn't wrap #152

Open
Jopp-gh opened this issue Jul 25, 2024 · 5 comments
Open

Input box missing - description string doesn't wrap #152

Jopp-gh opened this issue Jul 25, 2024 · 5 comments
Labels
bug Something isn't working

Comments

@Jopp-gh
Copy link

Jopp-gh commented Jul 25, 2024

Bug Description

the max width of widgets is way too narrow, please see the linear widget (2nd column) in my attachment

German uses pretty long composite names, so I can't even set a value for minimum speed in "linear" curves, because the input box is on the outside of the widget. Also the other input boxes in the same "linear" curves widget are too narrow, barely readable.

FAN-c

Expected Result

To be able to insert all possible values

Steps to Reproduce

Add a linear curve and check the min speed value (German env)

Possible Solutions

I suggest to enlarge the width of widgets and wrap long strings so they break and continue on the next line.

Operating system

Linux Mint Virginia

fan-control version

2024.6

@Jopp-gh Jopp-gh added the bug Something isn't working label Jul 25, 2024
@wiiznokes
Copy link
Owner

I will look into it. In the meantime, you can use the config file

@wiiznokes
Copy link
Owner

I can't fix it currently, due to limitation in libcosmic. But the fix i was thinking was to use ..., witch isn't a good solution for a static translation of the app. I would suggest to change the German translation to something shorter, or don't translate this fields.

For example, in French, even if it's not a perfect solution, i didn't translate it because i think it's clearer having

min_temp = min temp
min_speed = min speed

instead of

min_temp = temp min
min_speed = vitesse min

(to keep the min prefix)

@Jopp-gh
Copy link
Author

Jopp-gh commented Jul 25, 2024

Thanks for your quick answer

Fan control doesn't have a switch to change to another language. Did you mean, I can change language in some configuration file?

Yeah, to truncate long strings is a quick solution.
Also, I could provide a new and shorter German localization of Fan Control's strings, if you're interested.

On the long run however, I'd like to see the width of widgets slightly increased.

@wiiznokes
Copy link
Owner

Fan control doesn't have a switch to change to another language. Did you mean, I can change language in some configuration file?

You can either submit a pr with new german translation (https://github.com/wiiznokes/fan-control/blob/master/CONTRIBUTING.md) or use flatseal to set this env variable LANG=us. This will make fan-control use english translations

On the long run however, I'd like to see the width of widgets slightly increased.

I will consider it

@Jopp-gh
Copy link
Author

Jopp-gh commented Jul 25, 2024

use flatseal to set this env variable LANG=us.

Great

On the long run however, I'd like to see the width of widgets slightly increased.

I will consider it

Thanks.

Here a quick mockup how I would change widget width:
FanC-mockup

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants