-
Notifications
You must be signed in to change notification settings - Fork 383
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
volume/gain knob in UI #19
Comments
I can hop on this if you can somewhat explain it a bit more detailed it to me pretty please? |
Sure, when you're screen sharing with someone you see their desktop, but now the app automatically hooks up voice chat between both people too. Theres no UI for it currently, it just kind of automagically happens in the background. It would be nice to have a mute toggle and a volume slider. To implement them, there is an One option for the volume level might be to use a range input to set the I would probably add in some code here that creates new audio control html elements and inserts them into the right place in the UI: https://github.com/maxogden/screencat/blob/master/connect.js#L85 You would also need to hook up some events using javascript to make the audio controls affect the values on the |
@maxogden Should the audio controls be on the stream screen? I think that would make more sense as opposed to having to click on the screencat logo in the menu bar? |
@maxogden http://codepen.io/anon/pen/ZbdQXN been playin' |
@jsimplicio that looks pretty sweet! not sure about best placement. if its on the stream screen it would mean the window would have to scroll, or have a top/bottom bar in addition to the remote screen. we could alternatively make the volume control float above the remote screen but that might obscure stuff underneath, but that might not be that big of an issue. on the other hand, i think it might be okay to put it in the menubar popup menu because I have a feeling it will be infrequently accessed |
@maxogden It does make sense that the volume will be accessed not so frequently. If we put it in the menubar menu though we got to share space with the "you are now viewing a screen. stop. show." etc. Maybe you're greeted with "You are now viewing a screen" I gotta do some mockups and figure out how to make this look good. |
@maxogden sup, max. How about something sort of like this? How are you envisioning it? |
@jsimplicio yea that looks awesome, though the placement feels a little awkward. maybe it should go above the buttons? |
@maxogden What about a little menu like this? Or is this a bad idea / not doable? |
whoa super good!! On Thu, Dec 10, 2015 at 3:07 PM, Julia notifications@github.com wrote:
|
@maxogden Thanks! Is this something doable? To perhaps shrink the menu to that size like a more square-y shaped long list of actions? Actually that makes me rethink the size of the menu bar drop down. Did you particularly choose a rectangle that size and that type of layout for a reason? |
no we can totally shrink the window size, its totally arbitrary right now. I was actually wondering if a different window shape would make #37 (comment) look better today, but i'm not sure how I would change it |
@maxogden That comment could totally fit in a small long square if it has like a fixed area where you can scroll down and keep seeing new images of screen? Sort of like an iPhone. Here's how some of the app would look that size (working on it rn): |
@jsimplicio I think that looks pretty awesome. Now that I think about it I think the previous window dimensions were there because i used to put the remote desktop inside the same window, but now I do the separate popup window. So I think the smaller size makes a lot of sense |
I'm changing things to that dimension so we can see how it goes. I'm getting this error when I run Julias-MBP:screencat juliasimplicio$ npm start
fs.js:500 npm ERR! Darwin 15.0.0 npm ERR! Please include the following file with any support request: |
@jsimplicio hmm, try deleting the |
@maxogden That worked 👍 . Also right now if you click on share your window and click to go back, that view still stays open: |
ah crap On Wed, Dec 16, 2015 at 2:35 PM, Julia notifications@github.com wrote:
|
just fixed o/ in latest commit |
@maxogden So I added the little volume in the UI but I'm not too good at javascript. Anyway you can link the stuff you explained to me to the UI? Otherwise I can try to hack around but trust me I don't wanna ruin any code haha |
Yes but I am going on a trip tonight and wont be back till Friday so it On Mon, Jan 11, 2016 at 11:28 AM, Julia notifications@github.com wrote:
|
@maxogden it's no worries whenever you have the time! Just thought I'd ask you |
now that I added audio chat, we should have a way to control audio gain. Not sure the cleanest way to add it to the UI now though, suggestions welcome. This can be implemented using the WebAudio API Gain Control Nodes. There are probably other interesting controls/effects we can surface as well
The text was updated successfully, but these errors were encountered: