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

Quality and compression slider values don't do anything #2994

Closed
ndusek opened this issue Aug 31, 2023 · 3 comments
Closed

Quality and compression slider values don't do anything #2994

ndusek opened this issue Aug 31, 2023 · 3 comments
Milestone

Comments

@ndusek
Copy link
Contributor

ndusek commented Aug 31, 2023

We found that somewhere between 2.x and 3.x, the compression and quality sliders in the session card for VNC apps don't actually affect the quality and compression. To actually change those settings, we have to use the side bar settings that noVNC puts in the actual GUI window.

The reason for this seems the be that the variables passed in the URL are compressionsetting and qualitysetting (see _novnc.html.erb here, for instance) whereas noVNC takes as arguments simply compression and quality. Maybe this was a change from noVNC 1.1 to 1.3?

In any case, we found that manually changing compressionsetting -> compression and qualitysetting -> quality in the noVNC launch URL fixed the problem. Not sure if this is a known issue or not, but we thought we'd report it just in case.

@osc-bot osc-bot added this to the Backlog milestone Aug 31, 2023
@johrstrom johrstrom modified the milestones: Backlog, 3.0 Aug 31, 2023
@johrstrom
Copy link
Contributor

🤦‍♂️ Thanks for report! I'm unaware of the bug. I'll try to get this patched in 3.0.

@ndusek
Copy link
Contributor Author

ndusek commented Aug 31, 2023

No problem. We can submit a PR too, if it's just a matter of doing a quick substitution for those two arguments in _novnc.html.erb? Or is it more complicated than that?

@johrstrom
Copy link
Contributor

No problem. We can submit a PR too, if it's just a matter of doing a quick substitution for those two arguments in _novnc.html.erb? Or is it more complicated than that?

A patch would be most welcome! That appears to be the case yes.

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

No branches or pull requests

3 participants