fix: ignore \n|\r

lvjiaxuan · Oct 29, 2022 · 191c85d · 191c85d
1 parent 6721f75
commit 191c85d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/extension.ts b/src/extension.ts
@@ -39,6 +39,9 @@ export async function activate(context: ExtensionContext) {
   context.subscriptions.push(...disposes, statusBarItem)
 
   log('eslint-disabled initialized!')
+  statusBarItem.text = '$(check) eslint-disabled initialized!'
+  statusBarItem.show()
+  setTimeout(() => statusBarItem.hide(), 5000)
 }
 
 // this method is called when your extension is deactivated
@@ -58,7 +61,7 @@ const disposes = [
     }
 
     if (activeTextEditor.selections.length > 1) {
-      log('Sorry, we can not disable multi-lines for now. Support in later version.', true, 'OK')
+      log('Sorry, we can not disable multi-selections for now. Support it in later version.', true, 'OK')
       return
     }
 
@@ -99,8 +102,9 @@ const disposes = [
 
     activeTextEditor.selections.forEach(selection => {
 
+
       const text = getTextBylines(selection.start.line, selection.end.line)
-      if (!text) {
+      if (!text?.replace(/\n|\r/g, '')) {
         log('No content to disable.', true, 'OK')
         return
       }