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

[Request] Improve layout of main Temp chart #5003

Closed
mcstoufer opened this issue Apr 29, 2024 · 2 comments
Closed

[Request] Improve layout of main Temp chart #5003

mcstoufer opened this issue Apr 29, 2024 · 2 comments
Labels
done Done but not yet released request Feature request
Milestone

Comments

@mcstoufer
Copy link

Is your feature request related to a problem? Please describe.

The x-axis labels of past time intervals are bit too noisy. The default 120 values is just a spray of hyphens, values and redundant min labels.
Visibility could be improved.

I'm using a current Chrome browser on Mac OS. The issue does not alleviate itself when the window is stretched horizontally. Reducing the total interval to "70" does make the layout more consistent. Setting it to '240' completely blows out the layout.

Describe the solution you'd like

  • Decrease the kerning between the "-" and the value.
  • Reduce all the min labels down to one axis label centered horizontally on the axis below the interval markers.
Screen Shot 2024-04-29 at 10 44 51 AM

Describe alternatives you've considered

No response

Additional context

No response

@github-actions github-actions bot added the request Feature request label Apr 29, 2024
@foosel
Copy link
Member

foosel commented May 6, 2024

Just to clarify something here, the default is 30 minutes, not 120:

image

However, I agree that it also still look sensible above that, so I'll look what I can do here.

foosel added a commit that referenced this issue May 6, 2024
If there would be more of 10 ticks on the x-axis, add 5min to the tick
interval until that's no longer the case. That way the x-axis on the
graph now automatically scales so that the axis tick labels are still
readable and usable.

Closes #5003
@foosel foosel added the done Done but not yet released label May 6, 2024
@foosel foosel added this to the 1.10.x milestone May 6, 2024
@foosel
Copy link
Member

foosel commented May 6, 2024

Ready for 1.10.1.

Now looks like this, the ticks on the time axis scale automatically to a multiple of 5min that fits:

Screenshot from 2024-05-06 14-11-26

@foosel foosel closed this as completed in d773dc1 May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
done Done but not yet released request Feature request
Projects
Status: Done
Development

No branches or pull requests

2 participants