CkNoSFeRaTU commented on a change in pull request #95:
URL: https://github.com/apache/openmeetings/pull/95#discussion_r428166586



##########
File path: 
openmeetings-web/src/main/java/org/apache/openmeetings/web/room/raw-video.js
##########
@@ -391,7 +391,7 @@ var Video = (function() {
                _resizeDlgArea(hasVideo ? size.width : 120
                        , hasVideo ? size.height : 90);
                if (hasVideo && !isSharing && !isRecording) {
-                       VideoUtil.setPos(v, 
VideoUtil.getPos(VideoUtil.getRects(VIDWIN_SEL), sd.width, sd.height + 25));
+                       VideoUtil.arrange();

Review comment:
       Sure. I can rework it so in 0 mode (default) all currently present 
arrange hotkeys and window arrangements will behave exactly like they are 
behave now and wouldn't switch to different mode.
   
   About ways to switch mode I'm planning to add settings to room configuration 
in follow-ups patches. For ability to set different default mode and change 
resolutions for modes 1 & 2. So they are not hardcoded and can be changed on 
per room basis. And also add global settings to what set them on automatic room 
creation.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to