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

Added monitor span support #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

BizarreMachinist
Copy link

Hey there!

I've added the ability to have the video stretch across dual monitors (an probably more, but I can't test it).
This is for when you make a video that is the combined width of both monitors. Previously it would squish the width and display a copy on each monitor.
This can be done by adding an optional --span or -s after the file name.
I've also made it throw an error if no arguments are given. This doesn't stop you from adding more than the file name and -s, but it'll simply ignore everything after.

I tried to keep your formatting style while editing. Let me know what you think. This is kinda my first pull request :D

Have a glorious day!

@Ninlives
Copy link
Owner

Hi @BizarreMachinist , sorry for the very late response and thanks for your help, I haven't look into this project for a long time and I decided to update it a bit during this holiday. However I cannot test your pr locally since I do not work with dual monitors, I hope someone else can confirm this pr works :)

@Truemmerer
Copy link

Truemmerer commented Apr 25, 2022

Unfortunately, this does not work for me. I only get a black screen.

GNOME 42
Fedora 36
X11
2 Monitors

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.

3 participants