-
-
Notifications
You must be signed in to change notification settings - Fork 219
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
Cannot present/share desktop #19
Comments
I am afraid this is to be pass on to Microsoft. You can up-vote this suggestion: Closing this as the scope of this client is to wrap the functionality of the browser, not implement functionality that not even Microsoft are providing. |
As Microsoft is working on this feature at the moment (desktop sharing via web), we should keep this ticket open/pending. I will actively help get this done in teams-for-linux, cause I need it very much. |
Reopening. Is one of those that is really not a issue for us as, as soon as they implement it for Chrome or Edge, we should get it... but yeah, keeping it open for the moment. By the way, the "working on it" could mean finish next month, in 2 years... is fairly Agile but they should probably have a couple of more states to show progression. |
I also want to believe! in miracles |
It looks like there is an official chrome extension for sharing: |
Talking about miracles :) I downloaded and unzipped the extension. Unfortunately it does nothing special in my opinion. The most interesting part is imho:
But I am not sure how it might work in Electron 🤔 |
That looks promising! Just reading the electron documentation, maybe the following works. https://electronjs.org/docs/api/browser-window#browserwindowaddextensionpath Never tried adding an extension, but it should work. Might need a delay as we are changing the browser from chrome to edge (or the other way round, I don't remember now) Unfortunately I don't know when I will be able to test it. Give it a go and make a pull request. I am happy to package it as 0.1.15 if it works! Thanks again!! |
Reading the comments in the extension, it seems like is not working. Hopefully is just a WIP, not sure. Thanks for keep searching @cameronrmorris and @byteSamurai , at some point we all will be able to share the screen with teams... or will move on to something that allows us to do it. |
Tried a few things but can only add sharing ppt;s into a presentation... not ideal but better than nothing. That will be part of #21 (the code is already there so feel free to |
Thx @IsmaelMartinez It might be a stupid question, but have you ever tried to reverse engineer the MS Teams app? I could imaging we find something useful after downloading the Java-Files, beautify them and study it. But its PITA work :/ In the end of April I have a couple of days and could try something like that, in case you haven't tried it already |
Hi @byteSamurai , I had a quick look (not in deep). You can unasar the "asar" files and you get a bunch of Javascript (lots of lots). I didn't find anything interesting but is fairly lot of code and I didn't look for too long. I tried setting the Feel free to try with the windows/mac client and see if you can get any closer... it should be possible but not sure if we will need to bypass Microsoft implementation and do it with webrtc (if that works) or we can just piggyback on some of theirs. |
0.1.15 now is been released and got the ppt sharing presentation... as said before, not really solving the issue but I hope that improves the situation. |
Screensharing with Teams and chorme/chromium working now for me. The "enable teams calling" do basiclly following:
Compared with the MS Extension, screensharing working without setting a different UserAgent Also tested the options with Rambox-OS and js injection. |
Yes, great, works for me, too. But only in "real" Google Chrome, in Chromium it doesn't work (when not using User Agent Switcher). |
That is promising. Added a few more of those "parameters" and got the following "error" in the console:
I will see if I got time to add the extension to the build and see if that makes it work... if anyone wants to try adding the extension this should work https://electronjs.org/docs/api/browser-window#browserwindowaddextensionpath Do it in branch https://github.com/IsmaelMartinez/teams-for-linux/tree/0.1.16 . I don't know when I will be able to look into this. |
This would be just amazing to be able to share the screen in Linux! |
I have just also tried this version but it does not work: danielhodder/teams-for-linux@3588853 |
Hi,
The basics seems to work... Now to the details :) |
@BlueCase Do you have a branch with your changes? I can check it from my side. |
Pull request to 0.1.16 if you can. We can test it and release it this week if all works fine. Great job!! |
@syepes @IsmaelMartinez
|
0.1.16 should have the screen share button in it. Otherwise we can just tweak it to think is the desktop. We can have a look at the other issues once your code is in. Not sure if @syepes got any changes, but do add them to 0.1.16 if so. Amazing job you all! At this rate we will end up with a better client than the one MS has for Windows/Mac.!! |
Just got it working, I'll be pushing it to the 0.1.16 branch in 10min |
First version #56 |
Merged in 0.1.16. Regarding the issues, here my answers:
I also found that, when you record a video, you need to ctrl+click to be able to download the link. (one of those hacks) Anyone is more than welcome to improve on those points, but, to be honest, is pretty decent stuff. I might do a bit of refactoring around it (nothing mayor, just naming convention is different of the rest), and there are a few/many things in the browser folder... and it might benefit from reorganising it a bit more... but that might need to wait for 0.1.17. If tomorrow I can tests this at work, and it works fine, I am happy to release 0.1.16. I will appreciate if you people can do some testing on this if at all possible! (and/or improve it!) Thanks to (in no particular order) @syepes, @BlueCase, @byteSamurai @cameronrmorris, @mpruefer and @tomicdusan. |
@IsmaelMartinez Thanks for the update!
Notification: Further Bugs:
|
About the Stream quality:
|
If you are using an external monitor you should avoid Chromium or Google Chrome at the moment, cause you cannot choose the desktops separately, see https://bugs.chromium.org/p/chromium/issues/detail?id=396091 jbtw: This issue is several years old
In my opinion: We are using Linux. We tend to know what we do. So I would not limit any options for the user here. Maybe someone would like to share a chat history oder a file stored in teams?
Good idea |
Yesterday, I tested the pull request #56 on my Ubuntu 18.04.2 installation with a colleague (using a macbook). Overall the screen sharing worked really well. As mention before, the initial image quality was very poor. While sharing my screen, the colleague switched to the my camera window and back to the screen share window. This improved the quality of the stream significantly. At the moment I can't say if it was just coincidence or the quality adjusts when switching between the camera and the presentation window. But might be worth looking (or a workaround at least) and something I'd like to share for further testing and improvement. I'll test the current 0.1.16 branch today (aka. typical working day) and see if I could provide more information. Thanks for the great work 🙇♂️ |
I mean the dialog window for selecting the to share window/desktop. I also know about the bug round about the seemingly unsolvable chrome issues to select a external monitor :D
That's exactly the problem why it's hard for new Linux users to get things work and go back to window. But that's a off topic :)
Because of that I wrote "or mark" ;) |
@BlueCase, @byteSamurai and @dbaelz We should probably create a few new "issues" in here so we/someone can tackle them in isolation. Now the long message... apologies ;)
I don't think we can easily use the chrome window picker and the electron implementation relies on creating a desktop picker. True we could just select the "window" by default but having multiple monitors might "break" that usage. So agree with @byteSamurai and we should keep it. Maybe v2 can have a config option where people can put the id that they want to always share.
Agree again with @byteSamurai. People might like to share that to show how to use the app... if we can determine easily that is the presentation screen, we can tag it like that (point 3)
Indeed, even better, we should group them by types like "window" or "screen" and also tag them (not sure how much info we get from the screens.
That sounds like a great idea. Another one for version 2. Alternatively, we can do like the windows desktop app and add a wee screen with what we are sharing
Will need to test this to fully understand what you mean. Maybe point 3 might help?
Something to look into. That could be a blocker for this if the resolution is that bad, but, if there is a work around like the one @dbaelz mentions, then it might be ok for v1. |
Seems the fallback to 640:386 is the reason for the poor quality. |
Thanks again for all the hard work on this subject. The screen selector can be an issue if you have multiple screens, I have tested it with 3 and it can get messy. Basically it only allows to share all 3 screens together or a single app window. Definitively it would be great to have the possibility of selecting one of the external screens. |
But this is standard WebRTC behaviour on Linux, it's the same in GoToMeeting, in Google Hangouts and previously was in Hipchat. So that's okay. When I want to share the whole screen, first I deactivate all other screens except one. |
Not part of this issue: |
@IsmaelMartinez today I'm using teams from |
@maximbaz , could you pull the latest and see if you get it again? I have seen this happening when the app opens in a one-to-one chat. I have increase the timeout back to 3 seconds (instead of one) but I might need to revert it to its original 5 seconds. I have added a wee improvement on the layout (names on it and two per row). That should tackle improvement number 3 (or at least, part of it) Regarding the quality, I have seen it and looks like it depends on the screen that you are sharing (resolution). I haven't played enough with WebRTC but I will see if I can have a look at that one also at some point this week. With regards the "select presentation only", I can confirm is annoying as you don't get a feedback neither of if the use has accepted the share or not... until a wee while but only if they accept. |
@IsmaelMartinez you do such a great work, isn't is time to unfork this repo now? 🙈 |
It might be more discoverable as a fork, because people still stumble upon @IsmaelMartinez it still happening. Could you explain what's the root cause, what kind of timeouts are you trying to add and why they help? Is there a workaround that does not require teams restart? I have now soo many colleagues who are eager to start using screen sharing, but for one or another reasons cannot build from source. Please don't delay the release too much, a version that has screen sharing but occasionally requires restart is still much better than a version with no screen sharing 😉 |
@byteSamurai and @maximbaz . I will wait until there are more stars here than in the ivelkov... if we ever do. Regarding the timeout, the index.js file in the browser folder has a timeout before it applies all the angular inject changes. It use to be 5000 (5 seconds) but I did bring it down to 1second and now to 3. I have added a parameter in the main index.js app to enable the h.264 codec (that I think should be enabled by default). could you guys check it that makes any difference to the video quality? I had mainly good quality images, a part from the odd one here and there. Happy to release tomorrow and open issues for each of the other issues. Still better than no video :) |
I will release 0.1.16 as a Pre-release (as I think is not really that neat) so people can start using and creating issues in here. I will close this issue. Could any of you add new issues as you see/find? I will be out until Tuesday so can't really do much changes to it. Thanks again to everyone involved! |
@IsmaelMartinez Think for a first release it is great.. More as MS delivered till now ;) |
@IsmaelMartinez thank you a lot! Maximum efforttttt! |
Pre-release in here: https://github.com/IsmaelMartinez/teams-for-linux/releases/tag/v0.1.16 I will close this issue, we can create new ones for each case... I have lost track of what is fixed and what is not!! |
Seems like MS is now going to provide all of this "pretty soon" Extract from here: "Enhancements to the Chrome browser meeting experience (coming soon) are getting richer for Windows and Mac users. A year ago, we introduced frictionless meeting join in Chrome with no client needed. Coming soon, we will enhance that zero-footprint experience by adding 3 new capabilities: the ability to see participants video, application sharing, and full desktop sharing. This enables any meeting attendees to have face to face conversations and share content easily, with zero downloads needed." |
I tried to present/share desktop on Ubuntu 16.X and 18.x but i get the message "Sorry your browser isn't supported..."
The text was updated successfully, but these errors were encountered: