Bug #2141

noVNC console mouse synchronisation problems (related to scrollbars)

Added by Rolandas Naujikas about 8 years ago. Updated over 7 years ago.

Status:ClosedStart date:06/26/2013
Priority:HighDue date:
Assignee:-% Done:

0%

Category:Sunstone
Target version:Release 4.2
Resolution:fixed Pull request:
Affected Versions:OpenNebula 4.0

Description

When noVNC console have scrollbars and using tablet/usb device in VM, mouse is synchronised with VM only when scrollbars are showing left topmost part of window. When scrollbars are moved from initial position, mouse is off by the same amount as scrollbars are moved.

History

#1 Updated by Daniel Molina about 8 years ago

  • Status changed from Pending to New

#2 Updated by Daniel Molina about 8 years ago

  • Priority changed from Normal to High

#3 Updated by Ruben S. Montero about 8 years ago

  • Target version set to Release 4.2

#4 Updated by Ruben S. Montero almost 8 years ago

  • Status changed from New to Closed
  • Resolution set to fixed

Solved by detach of VNC screens.

#5 Updated by Rolandas Naujikas over 7 years ago

Problems with detached VNC screen in 4.4RC1.
Mouse could not be moved to topmost position by some small amount
(in windows VM cannot access window bar to close or operate it).

#6 Updated by Daniel Molina over 7 years ago

Hi Rolandas,

Does this happen with the new update of noVNC? this commit is not included in RC1 732528eef61a16cbb1184ea00efff9fcf2e57287

#7 Updated by Rolandas Naujikas over 7 years ago

I was testing latest version from branch one-4.4 (including 732528eef61a16cbb1184ea00efff9fcf2e57287).
I just did testing by downgrading one-4.4 to state before 732528eef61a16cbb1184ea00efff9fcf2e57287 and it works correctly (detached window is accessible with mouse everywhere and title "Connected (encrypted) to: QEMU (one-645)" is in right place).

#8 Updated by Rolandas Naujikas over 7 years ago

If I remove "position: absolute" in
src/sunstone/public/vendor/noVNC/base.css

409: #noVNC_status {
410:   z-index: 0;
411:   position: absolute;
412:   width: 100%;
413:   margin-left: 0px;
414: }

it works better, but looks not so nice.

#9 Updated by Daniel Molina over 7 years ago

Could you try applying this patch on top of HEAD of one-4.4?

diff --git a/src/sunstone/views/vnc.erb b/src/sunstone/views/vnc.erb
index d5f8ec0..c909a22 100644
--- a/src/sunstone/views/vnc.erb
+++ b/src/sunstone/views/vnc.erb
@@ -36,7 +36,9 @@ var INCLUDE_URI = "vendor/noVNC/";
     <div id="noVNC_screen">
             <div id="noVNC_status_bar" class="noVNC_status_bar" style="margin-top: 0px;">
                 <table border=0 width="100%"><tr>
-                    <td><div id="noVNC_status">Loading</div></td>
+                    <td><div id="noVNC_status" style="position: relative; height: auto;">
+                        Loading
+                    </div></td>
                     <td width="1%"><div id="noVNC_buttons">
                         <input type=button value="Send CtrlAltDel" 
                             id="sendCtrlAltDelButton">
@@ -54,8 +56,8 @@ var INCLUDE_URI = "vendor/noVNC/";
         "use strict";

         // Load supporting scripts
-        //Util.load_scripts(["webutil.js", "base64.js", "websock.js", "des.js",
-        //                   "input.js", "display.js", "jsunzip.js", "rfb.js"]);
+        Util.load_scripts(["webutil.js", "base64.js", "websock.js", "des.js",
+                           "input.js", "display.js", "jsunzip.js", "rfb.js"]);

         var rfb;

@@ -100,6 +102,7 @@ var INCLUDE_URI = "vendor/noVNC/";
             }
         }

+        window.onscriptsload = function () {
             var host, port, password, path, token;

             $D('sendCtrlAltDelButton').style.display = "inline";
@@ -146,6 +149,7 @@ var INCLUDE_URI = "vendor/noVNC/";
                                'onPasswordRequired':  passwordRequired});
                 rfb.connect("<%= params['host']%>", "<%= params['port']%>", "<%= params['passwo
             }
+        };

#10 Updated by Rolandas Naujikas over 7 years ago

Daniel Molina wrote:

Could you try applying this patch on top of HEAD of one-4.4?

[...]

Tested - it works and looks OK.

Also available in: Atom PDF