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

Zoom page content #20

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

intermittentnrg
Copy link
Collaborator

I originally sent ctrl+ to browser to zoom, but the screenshot unzooms so I found this method instead.

window size and zoom factor should be provided as options obviously...

Copy link
Contributor

@amotl amotl left a comment

Choose a reason for hiding this comment

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

Excellent. I've not run the program yet, to get an idea about the outcome, but I like your attention to details.

Shall I support you on bringing in the zoom factor as option/setting?

@intermittentnrg
Copy link
Collaborator Author

intermittentnrg commented Oct 17, 2023

Yes support would be fantastic thank you!

I've had a lot of feedback that people couldn't read the text when posting widescreen videos on X/Twitter, and given the styling/studio settings do not work this is my solution which works quite well. I'm not even sure of the expected result from the styling as I've not seen it work...


  1. An outstanding issue from feedback is that the labels on the X axis jump around. Just changing dateformat probably improves it, it annoyingly can only be set globally via grafana.ini and it messes up my normal use of grafana.

  2. I also hacked timeutils.py to suit my needs which also needs to be made an option.

def get_freq_delta(every: str) -> RecurrenceInfo:
    rr_interval = 4
    rr_freq = HOURLY
    delta = timedelta(days=30) # - timedelta(seconds=1)
    return RecurrenceInfo(every=every, frequency=rr_freq, interval=rr_interval, duration=delta)

Not sure your usecase was for timeseries panels? Or I may have misunderstood the options. But this was ideal for me to set step size and time range shown.

I originally sent ctrl+ to browser to zoom, but the screenshot unzooms
so I found this method instead.
@amotl amotl self-assigned this Nov 6, 2023
@amotl amotl removed their assignment May 8, 2024
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