openmeetings-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Marius van Voorden (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (OPENMEETINGS-570) Some issues with device settings and attach Camera
Date Sun, 17 Mar 2013 10:08:12 GMT

    [ https://issues.apache.org/jira/browse/OPENMEETINGS-570?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13604563#comment-13604563
] 

Marius van Voorden commented on OPENMEETINGS-570:
-------------------------------------------------

Greatly improved, cheers!
I can confirm point a you mention.

One suggestion I would still make to improve the intuitiveness of the UI:
The preview box shows the webcam feed as it will be in the conference. This gives immediate
visual feedback, showing if the right settings have been chosen.
This means that when the user selects "Audio only" or "No audio/video", the video feed should
*not* be shown. One could replace it with either the background color, or with the default
avatar picture, or with the user's avatar. Any of those would work.
                
> Some issues with device settings and attach Camera
> --------------------------------------------------
>
>                 Key: OPENMEETINGS-570
>                 URL: https://issues.apache.org/jira/browse/OPENMEETINGS-570
>             Project: Openmeetings
>          Issue Type: Bug
>    Affects Versions: 2.1 Apache Release
>            Reporter: SebastianWagner
>            Assignee: SebastianWagner
>
> The problem is that the attachCamera method is invoked multiple times when the video
device settings initialize:
> 1) As soon as the two combobox'es rendered completely each will call "attachCamera" and
re-attach the video to the view (2 calls)
> 2) as soon as the user clicks "Allow" in the flash settings, the attachCamera function
is invoked again (what for ?) (1 call)
> Somehow this leads to a bug in the application that creates two videos and one is no
more resizable.
> Also the video view that the attachCamera method is using is the wrong videoview.
> There are two preview video objects in the device settings:
> a) One for the actual current webcam view (aka "_publisher")
> b) One for the playback of the 5 second self recording test video (aka "_viewer")
> The attachCamera function attaches the stream to the _viewer instead of the _publisher.
> There is also a 3th problem (that is also present when having no "attachCamera" function
enabled, but simply doing a 5 second recording test). But the auto-attaching of the Camera
to the videoview makes it more obvious:
> When you click "no Audo/Video" as device settings, your webcam still shows the "red status
light" as if some application access it. Actually this should not happen. The status light
should turn off when you don't use your webcam. The issue is that the cam is somehow not correctly
unattached from the videoview.
> The issues of the 3th problem is in the "_stop" method in the "baseVideoStream.lzx"
> _stop will only be execute a "clear" (that will un-attach the Camera from the videoview)
> if (_ns != null) //if there is a netstream attached to the video
> => In the device settings _ns is null, cause you just want to show a preview
> The effect is that _stop will never call "clear"
> By adding a manual "clear" to the attachCamera function (and when the window closes),
the camera status light goes off when you choose "no Audio/Video" as setting and click on
"start conference".
> I will try to bundle a patch for those issues now.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message