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

Window size setting. #77

Closed
wants to merge 3 commits into from
Closed

Window size setting. #77

wants to merge 3 commits into from

Conversation

RubenNL
Copy link

@RubenNL RubenNL commented Jul 4, 2022

No description provided.

@CLAassistant
Copy link

CLAassistant commented Jul 4, 2022

CLA assistant check
All committers have signed the CLA.

@MightyTitan
Copy link

The scaling does not always work properly. Not all the elements are scaled. This particular element was working and then just stopped working.
Image 001

@briangann
Copy link
Collaborator

let me test this out, i have seen this also on some dashboards.

@briangann briangann added bug Something isn't working enhancement New feature or request labels Sep 22, 2022
@briangann briangann self-requested a review September 22, 2022 18:26
@briangann briangann self-assigned this Sep 22, 2022
@@ -44,6 +46,8 @@ func ProcessArgs(cfg interface{}) Args {
f.StringVar(&a.Mode, "kiosk-mode", "full", "Kiosk Display Mode [full|tv|disabled]\nfull = No TOPNAV and No SIDEBAR\ntv = No SIDEBAR\ndisabled = omit option\n")
f.StringVar(&a.URL, "URL", "https://play.grafana.org", "URL to Grafana server")
f.StringVar(&a.WindowPosition, "window-position", "0,0", "Top Left Position of Kiosk")
f.StringVar(&a.WindowPosition, "window-size", "1920,1080", "Size of Kiosk in pixels")
Copy link
Collaborator

Choose a reason for hiding this comment

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

this variable should be a.WindowSize or it will overwrite the position setting and not set the size

Copy link
Collaborator

@briangann briangann left a comment

Choose a reason for hiding this comment

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

This is a great addition, it will need some adjustments due to recent merges to main.

@github-actions github-actions bot added the stale label Mar 7, 2023
@briangann briangann removed the stale label Mar 7, 2023
@grafana grafana deleted a comment from github-actions bot Mar 7, 2023
@briangann briangann added the no stalebot exempt from stale bot label Mar 7, 2023
@briangann
Copy link
Collaborator

this has been implemented/merged as part of PR #102

@briangann briangann closed this Mar 25, 2023
@koliyo
Copy link

koliyo commented May 8, 2023

The zoom scaling factor, chrome flag force-device-scale-factor, is not part of the #102 PR

ghost pushed a commit to koliyo/grafana-kiosk that referenced this pull request May 31, 2023
Implementation copied from grafana#77
@ghost ghost mentioned this pull request May 31, 2023
@ghost
Copy link

ghost commented May 31, 2023

New PR here for scaling factor: #109

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request no stalebot exempt from stale bot
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants