-
-
Notifications
You must be signed in to change notification settings - Fork 8k
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
UI, libobs: Add downstream keyer #2846
Conversation
Random bugs i noticed. This adds to scenes instead of global scenes: (in non-studio mode) |
A few points of feedback: Amazing work I tested this on macOS Catalina without issue, I would note that the default location of the dock was to the right of the controls for me, where I see in the ui file it was meant to be the left of Scenes. DSK toggling vs switching It might make sense to support toggling and switching through an option in the settings to change behaviour? DSK transitions User Interface The reason for this is that its not obvious whether you've selected the source or global and it kind of breaks the current UI pattern which is clearly has a hierarchial relationship from the Scenes on the left to the Sources on the right. Current: Scenes > Sources There are a few options you could use: Simpler - might be making it clear which scene the sources list is for by changing the Dock title and maybe having a globe icon for the Global ones. Complex - might be to put global sources in the Sources Dock as a tab, either by showing the tab. |
The bugs mentioned here should be fixed now. |
Adding two global scenes it looks like when the 2nd is selected (2nd being farthest down), both are rendering instead of just the 2nd one. Adding a third global, the 1st is always shown if it was selected before selecting the 2nd or the 3rd. (in non-studio mode) Previous bugs look fixed. |
Found another bug: Possible third bug: Whats weird now is that every time I quit OBS and reopen that 'Bug' global scene increases by one. I've launched OBS 3 times, and now have three 'Bug' global scenes. Likely fourth bug: |
fec5429
to
17a4ef8
Compare
All of the bugs mentioned here should be fixed now. |
How much of the context menu has been touched? Currently I see options that only make sense for standard scenes, like "Show in Multiview" (and this option incorrectly toggles the standard scene with the same index). As there's no "copy/paste" for scenes, maybe it'd be worth having a checkbox in the Duplicate Scene dialog to allow the user to duplicate the scene from standard/global? I also notice that switching between global scenes doesn't respect the globally selected Transition nor the Transition Override. At the very least, I think it should support Transition Override. |
A thought came up in conversation today that it would be cool if you could have the scenes above and below the normal scenes so that you could create a permenant background. |
Closing until RFC is finalized. |
Description
Adds downstream keyer (global scenes), so users can have scenes that are always on top of other scenes. This PR allows for unlimited amount of DSK channels.
TODO:
Motivation and Context
Previous PR:
#1321
Previous PR only allowed for one DSK channel.
RFC:
obsproject/rfcs#11
How Has This Been Tested?
Created DSK scenes and made sure that they were always on top.
Types of changes
Checklist: