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

Processor info very big and look messy #18

Closed
Anifyuli opened this issue Jan 15, 2024 · 6 comments
Closed

Processor info very big and look messy #18

Anifyuli opened this issue Jan 15, 2024 · 6 comments
Labels
bug Something isn't working fixed

Comments

@Anifyuli
Copy link

Description

Processor info hover action is very big and look messy and not uniform too with other info

Steps to Reproduce

  1. Show processor info
  2. Hover to processor name
  3. It look messy

Screenshots

image

Environment

  • Astra Monitor Version: 4 (EGOv6)
  • GNOME Version: 45.2
  • Operating System and Version: Ubuntu 23.10
  • Other Relevant System Information: Intel core-i5 8350U, Intel UHD 620

Logs

N/A

Additional Context

N/A

@Anifyuli Anifyuli added the bug Something isn't working label Jan 15, 2024
@ljuzig
Copy link
Collaborator

ljuzig commented Jan 15, 2024

What's your screen resolution? I have to well figure out how to solve the problem of a screen too low to hold all the information. Today I ran some tests for a scrolling container but I don't really like the mouse interaction since it's a popup that disappears when the mouse goes outside the name. A solution might be to selectively remove some lines until the list of information completely fits on the screen.

@fedkad
Copy link

fedkad commented Jan 16, 2024

A solution might be to selectively remove some lines until the list of information completely fits on the screen.

I suggest you split the information into two: Basic and Extended (=Flags and Vulnerability related stuff) and optionally allow the user to disable the Extended info.

@ljuzig
Copy link
Collaborator

ljuzig commented Jan 16, 2024

If I choose that path, I have to set the basic info as default and add an option to enable extended info. I don't want the user to face a bug-like situation at the first extension tryout.

@fedkad
Copy link

fedkad commented Jan 16, 2024

Then, a cascading menu is the second option coming to mind. But, this will make the screen layout and usage more complex.

@ljuzig ljuzig closed this as completed in 4c66655 Jan 16, 2024
@ljuzig ljuzig added the fixed label Jan 16, 2024
@ljuzig
Copy link
Collaborator

ljuzig commented Jan 16, 2024

@Anifyuli This should be fixed in the next release. When the screen is short multi-lines are stripped to a single line. If it's still not ok comment here or open a new ticket.

@Anifyuli
Copy link
Author

What's your screen resolution? I have to well figure out how to solve the problem of a screen too low to hold all the information. Today I ran some tests for a scrolling container but I don't really like the mouse interaction since it's a popup that disappears when the mouse goes outside the name. A solution might be to selectively remove some lines until the list of information completely fits on the screen.

I use 768P screen resolution

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

No branches or pull requests

3 participants