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

Attached plot panel fixed width plot or sensor panel #1179

Merged

Conversation

KitVanDeBunt
Copy link
Contributor

When scaling the window and the plot panel is attached to the window the width of both panels scale procentualy.
I always use this program with the plot panel attached to the right. I want the sensor panel to have a fixed width so I can freely change the size of the window without having to adjust the width of the sensor panel every time.

In my first commit, I made changes to make it work the way I want it to. But I am not sure everyone agrees with that. In the second commit I added it to the settings and the menu so its optional. the settings also work for when the plot panel is attached to the bottom.

2023-10-12_19-25_6kZXd5uRS3.mp4

@PhyxionNL
Copy link
Collaborator

Please follow title case names like the other menu items. I also think it can be renamed to Plot Scaling Mode.

Edited the names of menu items added for split panel scaling options.

Corrected some variable names to corrospond to menu item name changes.
@KitVanDeBunt
Copy link
Contributor Author

KitVanDeBunt commented Nov 10, 2023

Thanks for the feedback. I fixed the title case names. I changed the name to Split Panel Scaling Mode instead of Plot Scaling Mode does that work? The scaling only works if there are two panels in one window. Naming it Plot panel scaling mode might cause people to think that it has something to do with the scaling for the separate plot window as well.

@PhyxionNL PhyxionNL merged commit 137e91e into LibreHardwareMonitor:master Nov 15, 2023
@PhyxionNL
Copy link
Collaborator

Makes sense, thanks, merged.

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

Successfully merging this pull request may close these issues.

None yet

2 participants