Class context menu option

Remove alert logic from openInNewWindow method

Author: davidgross

lib/tab-bar-view.coffee

@@ -150,9 +150,6 @@ class TabBarView extends HTMLElement
     tab ?= @querySelector('.right-clicked')
     item = tab.item
     return unless item?
-    unless ['TextEditor', 'TestView'].indexOf(item.constructor.name) > -1
-      alert("'Open In New Window' only works for TextEditor tabs, for now.")
-      return 
     if typeof item.getURI is 'function'
       itemURI = item.getURI()
     else if typeof item.getPath is 'function'

lib/tab-view.coffee

@@ -7,6 +7,8 @@ class TabView extends HTMLElement
     if typeof @item.getPath is 'function'
       @path = @item.getPath()
 
+    if ['TextEditor', 'TestView'].indexOf(item.constructor.name) > -1
+      @classList.add('texteditor')
     @classList.add('tab', 'sortable')
 
     @itemTitle = document.createElement('div')

menus/tabs.cson

@@ -22,9 +22,11 @@ menu: [
     {label: 'Split Left', command: 'tabs:split-left'}
     {label: 'Split Right', command: 'tabs:split-right'}
 
+  ]
+  '.tab.texteditor': [
     {type: 'separator'}
-    
     {label: 'Open In New Window', command: 'tabs:open-in-new-window'}
+    {type: 'separator'}
   ]
   '.tab.pending-tab': [
     {label: 'Keep Pending Tab', command: 'tabs:keep-pending-tab'}