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

Feat/lawo and atem mono sum #334

Merged
merged 4 commits into from
Sep 10, 2024
Merged

Feat/lawo and atem mono sum #334

merged 4 commits into from
Sep 10, 2024

Conversation

olzzon
Copy link
Contributor

@olzzon olzzon commented Sep 3, 2024

Mono (sum) option for input selector has beed added to Atem and Lawo Ruby mixer protocols.

@@ -168,7 +168,7 @@
outline: none;
-moz-outline: none;
color: white;
height: 62px;
height: 55px;
Copy link
Contributor

Choose a reason for hiding this comment

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

Out of curiosity: What is the motivation behind the pixel change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@LindvedKrvang the commit where it's added is: [add Mono input selector make buttons smaller 2 fit 4 options]
So basically before there were 3 options now there are 4.
A better approach would arguably be to let it be dynamic, but this way, as little code as possible was changed.
🙂

@LindvedKrvang LindvedKrvang merged commit 8824740 into tv2:develop Sep 10, 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.

4 participants