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

Improve support for smaller screens #54

Merged
merged 4 commits into from
Nov 8, 2024

Conversation

phillipthelen
Copy link
Contributor

Related to #23 some UI changes that make neohtop work better with smaller window widths.

  • Storage, System and Network stat panel have a minimum width of 150px. This prevents issue where text would get wrapped and items cut off. The CPU and memory panels shrink more in these cases, making the usage bars smaller
  • removed the theme name from the theme switcher button. I like the switcher button, but I feel like having the 3 colors show your theme already does a good job of displaying the active theme.
  • use the standard/system clear button that is available for input with the search type to not need the extra clear button.
  • Stack the "Page X of Y" and "(x processes)" labels. They actually fit without making the toolbar higher, saving quite a bit of vertical space
  • center the pagination

These are more just suggestions from me and I can also undo some of the changes if the old style is preferred for p.e. the theme switcher.

New possible minimum width of around 1100px:
Screenshot 2024-11-08 at 12 18 55

with a wider window to show the centering:
Screenshot 2024-11-08 at 12 18 59

@Abdenasser
Copy link
Owner

@phillipthelen would you also want to update tauri.conf.json#L42-L43

Copy link
Owner

@Abdenasser Abdenasser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you!

@Abdenasser Abdenasser merged commit ce69f52 into Abdenasser:main Nov 8, 2024
1 check passed
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.

2 participants