even better messaging

Author: aviatesk

lib/runtime/refactor.js

@@ -36,7 +36,7 @@ class Refactor {
       const _new = items['Rename']
       if (!isValidWordToInspect(_new) || _new.match(wordRegexWithoutDotAccessor) != _new) {
         atom.notifications.addWarning('Julia Client: Rename Refactor', {
-          description: `\`${_new}\` is not a valid identifier`
+          description: `\`${_new}\` isn't a valid identifier`
         })
         return
       }
@@ -66,6 +66,12 @@ class Refactor {
         if (result.text) {
           editor.setTextInBufferRange(range, result.text)
         }
+        if (result.success) {
+          atom.notifications.addSuccess('Julia Client: Rename Refactor', {
+            description: result.success,
+            dismissable: true
+          })
+        }
         if (result.info) {
           atom.notifications.addInfo('Julia Client: Rename Refactor', {
             description: result.info,
@@ -82,11 +88,6 @@ class Refactor {
             description: result.error
           })
         }
-        if (result.success) {
-          atom.notifications.addSuccess('Julia Client: Rename Refactor', {
-            description: result.success
-          })
-        }
       })
     }).catch((err) => {
       if (err) console.error(err)