Revert "Merge pull request #853 from yacut/restore-cursor-position-af…

…ter-fix" This reverts commit 27d5570.
AtomLinter · Apr 6, 2017 · e71c8b7 · e71c8b7
1 parent b6bdeb1
commit e71c8b7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 26 deletions.
diff --git a/lib/main.js b/lib/main.js
@@ -70,18 +70,12 @@ module.exports = {
             rules = ignoredRulesWhenFixing;
           }
 
-          // The fix replaces the file content and the cursor jumps automatically
-          // to the beginning of the file, so save current cursor position
-          const cursorPosition = editor.getCursorBufferPosition();
           this.worker.request('job', {
             type: 'fix',
             config: atom.config.get('linter-eslint'),
             rules,
             filePath,
             projectPath
-          }).then(() => {
-            // set cursor to the position before fix job
-            editor.setCursorBufferPosition(cursorPosition);
           }).catch(err => {
             atom.notifications.addWarning(err.message);
           });
@@ -120,19 +114,13 @@ module.exports = {
           rules = ignoredRulesWhenFixing;
         }
 
-        // The fix replaces the file content and the cursor jumps automatically
-        // to the beginning of the file, so save current cursor position
-        const cursorPosition = textEditor.getCursorBufferPosition();
         this.worker.request('job', {
           type: 'fix',
           config: atom.config.get('linter-eslint'),
           rules,
           filePath,
           projectPath
-        }).then(response => atom.notifications.addSuccess(response)).then(() => {
-          // set cursor to the position before fix job
-          textEditor.setCursorBufferPosition(cursorPosition);
-        }).catch(err => {
+        }).then(response => atom.notifications.addSuccess(response)).catch(err => {
           atom.notifications.addWarning(err.message);
         });
       }

diff --git a/src/main.js b/src/main.js
@@ -66,18 +66,12 @@ module.exports = {
             rules = ignoredRulesWhenFixing
           }
 
-          // The fix replaces the file content and the cursor jumps automatically
-          // to the beginning of the file, so save current cursor position
-          const cursorPosition = editor.getCursorBufferPosition()
           this.worker.request('job', {
             type: 'fix',
             config: atom.config.get('linter-eslint'),
             rules,
             filePath,
             projectPath
-          }).then(() => {
-            // set cursor to the position before fix job
-            editor.setCursorBufferPosition(cursorPosition)
           }).catch((err) => {
             atom.notifications.addWarning(err.message)
           })
@@ -110,9 +104,6 @@ module.exports = {
           rules = ignoredRulesWhenFixing
         }
 
-        // The fix replaces the file content and the cursor jumps automatically
-        // to the beginning of the file, so save current cursor position
-        const cursorPosition = textEditor.getCursorBufferPosition()
         this.worker.request('job', {
           type: 'fix',
           config: atom.config.get('linter-eslint'),
@@ -121,10 +112,7 @@ module.exports = {
           projectPath
         }).then(response =>
           atom.notifications.addSuccess(response)
-        ).then(() => {
-          // set cursor to the position before fix job
-          textEditor.setCursorBufferPosition(cursorPosition)
-        }).catch((err) => {
+        ).catch((err) => {
           atom.notifications.addWarning(err.message)
         })
       }