Hide cursor when a browser manipulation is in execution (closes DevEx…

…press#1212)
AlexanderMoskovkin · Feb 22, 2017 · a436fb2 · a436fb2
1 parent 96bf9c4
commit a436fb2
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/src/client/automation/index.js b/src/client/automation/index.js
@@ -19,6 +19,7 @@ import AutomationSettings from './settings';
 import { getOffsetOptions } from './utils/offsets';
 import calculateSelectTextArguments from './playback/select/calculate-select-text-arguments';
 import ERROR_TYPES from './errors';
+import cursor from './cursor';
 
 
 exports.Click                 = ClickAutomation;
@@ -40,6 +41,7 @@ exports.ERROR_TYPES                  = ERROR_TYPES;
 exports.AutomationSettings           = AutomationSettings;
 exports.getOffsetOptions             = getOffsetOptions;
 exports.calculateSelectTextArguments = calculateSelectTextArguments;
+exports.cursor                       = cursor;
 
 exports.get = require;
 

diff --git a/src/client/driver/command-executors/prepare-browser-manipulation.js b/src/client/driver/command-executors/prepare-browser-manipulation.js
@@ -1,5 +1,6 @@
 import hammerhead from '../deps/hammerhead';
 import testCafeCore from '../deps/testcafe-core';
+import { cursor } from '../deps/testcafe-automation';
 import MESSAGE from '../../../test-run/client-messages';
 import DriverStatus from '../status';
 
@@ -30,6 +31,11 @@ export default function prepareBrowserManipulation (browserId) {
 
     document.title = assignedTitle;
 
+    var isCursorVisible = cursor.visible;
+
+    if (isCursorVisible)
+        cursor.hide();
+
     return delay(APPLY_DOCUMENT_TITLE_TIMEOUT)
         .then(() => {
             var message = {
@@ -46,6 +52,9 @@ export default function prepareBrowserManipulation (browserId) {
             nativeMethods.clearInterval.call(window, checkTitleIntervalId);
             document.title = savedDocumentTitle;
 
+            if (isCursorVisible)
+                cursor.show();
+
             return delay(RESTORE_DOCUMENT_TITLE_TIMEOUT);
         })
         .then(() => new DriverStatus({ isCommandResult: true, result }));