Skip to content

Add settings to control screenshare quality#718

Merged
Airyzz merged 4 commits into
mainfrom
screenshare-quality-settings
Feb 22, 2026
Merged

Add settings to control screenshare quality#718
Airyzz merged 4 commits into
mainfrom
screenshare-quality-settings

Conversation

@Airyzz

@Airyzz Airyzz commented Feb 20, 2026

Copy link
Copy Markdown
Contributor

Related to #450

@snoweuph

Copy link
Copy Markdown

can we get an experimental flatpak build for this? I would like to try this out :D

title: "Stream Resolution",
description:
"The resolution of your stream, higher is better",
options: [

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Some Device aren't 16/9. should there maybe be a free form selection, or some dynamic check, what the maximum possible resolution is of a source and based on that generate suggested lower resolutions?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

in my testing, this setting doesnt even really seem to effect the screenshare. I expected it to essentially pick a vertical resolution and maintain the correct aspect ratio, but not sure why its not having any effect at all

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

okay? thats really interesting :D

description:
"Choose which format to encode your stream in. Different codecs may run faster on certain devices, and may be unsupported on others. Most devices should support vp8 and h264.",
options: [
"h264",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

livekit servers can block/allow only specific codes, is there maybe a way to populate the list dynamicly, by what is supported by the used livekit instance?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The livekit client should automatically fall back to use a different codec if this is not supported, so i guess its more of a 'preferred codec' option

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

prolly should be added into the description then that its just preffered?

Comment thread commet/lib/ui/pages/settings/categories/app/voip_settings/voip_settings_page.dart Outdated
@Airyzz

Airyzz commented Feb 20, 2026

Copy link
Copy Markdown
Contributor Author

can we get an experimental flatpak build for this? I would like to try this out :D

https://github.com/commetchat/commet/actions/runs/22244426928

@snoweuph

snoweuph commented Feb 20, 2026

Copy link
Copy Markdown

can we get an experimental flatpak build for this? I would like to try this out :D

https://github.com/commetchat/commet/actions/runs/22244426928

looks like the flatpak debug build is brocken:

$ flatpak install --reinstall chat.commet.commetapp.flatpak
error: Invalid checksum of length 0 expected 32

@snoweuph

snoweuph commented Feb 20, 2026

Copy link
Copy Markdown

also the protable version is double tared :D

image

this makes normal tar -xzf fail btw, unless you specify another out dir, or rename the file xD

even then, its not like I can actually use the "portable" build, as it has hard deps:

$ ./commet 
./commet: error while loading shared libraries: libdbus-1.so.3: cannot open shared object file: No such file or directory

@snoweuph

Copy link
Copy Markdown

can we get an experimental flatpak build for this? I would like to try this out :D

https://github.com/commetchat/commet/actions/runs/22244426928

looks like the flatpak debug build is brocken:

$ flatpak install --reinstall chat.commet.commetapp.flatpak
error: Invalid checksum of length 0 expected 32

looks like github actions likes to play jokes: -> ive downloaded the flatpak. added the .zip suffix, and then unzipped it and got the real flatpak bundle. looks like for actions everythign is a zip??? the tarball also isnt double tared, but just zipped agian by github actions?? wtf.

Comment thread commet/lib/config/preferences.dart Outdated
DoublePreference("screenshare_fps", defaultValue: 60);

StringPreference streamCodec =
StringPreference("livekit_screenshare_codec", defaultValue: "h264");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I suggest defaulting to av1 #450 (comment)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Agreed, seems reasonable

@snoweuph

Copy link
Copy Markdown

I love this :D

@Airyzz
Airyzz merged commit 7d6e3c5 into main Feb 22, 2026
6 checks passed
@snoweuph snoweuph mentioned this pull request Feb 22, 2026
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