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

Streaming and Recording Indicators #112

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

Conversation

sspain
Copy link
Contributor

@sspain sspain commented Aug 17, 2021

Added indicators to top-bar.vue that light up to show streaming and recording status.

icons.js required the addition of the Circle and Cloud-Upload-Alt FA icons.

top-bar.vue needed just a few changes:

  1. Add two computed props to the component based on the current streaming/recording status of OBS,
  2. Put two indicators in the bar with responsive margins and widths to accommodate small screens,
  3. Hide the indicators when not connected to an instance of OBS, and
  4. Tie the opacity of the two indicators to the two new computed props.

About the only question I have left is the utility of making the indicators click targets that would command OBS to start or stop streaming or recording. I may tack that on, if anybody feels strongly enough about its usefulness. I'm not sure if that adds value to our workflow or not, but maybe it would... I'm open to comment on this.

By using a filter function and the Vue state, this change removes any
scenes from the panel that end in "[Hidden]", "^^", or "~~" and
presents a less-cluttered view with larger touch targets for more
complicated productions.
Add to the top bar live indicators for recording and streaming status.

icons.js required two additions from Font Awesome: Circle and Cloud
Upload Alt.

top-bar.vue required calculated properties based on the stream status
out of the VueX store, as well as the (responsive) spots for the
indicators themselves, and bindings on classes to tie brightness of the
indicators to the actual status.
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.

1 participant