fix dynamic requires

Author: aminya

lib/ui.js

@@ -1,16 +1,27 @@
 'use babel'
 import { CompositeDisposable, Disposable } from 'atom';
 
+// TODO use babel to export ... from ...
+import notifications from './ui/notifications'
+import * as selector from './ui/selector'
+import views from './ui/views'
+import progress from './ui/progress'
+import * as layout from './ui/layout'
+import * as docpane from './ui/docs'
+import * as focusutils from './ui/focusutils'
+import * as cellhighlighter from './ui/cellhighlighter'
+
 export default {
-  // TODO Fix all of these dynamic requires and circular dependencies
-  notifications: require('./ui/notifications'),
-  selector:      require('./ui/selector'),
-  views:         require('./ui/views'),
-  progress:      require('./ui/progress'),
-  layout:        require('./ui/layout'),
-  docpane:       require('./ui/docs'),
-  focusutils:    require('./ui/focusutils'),
-  cellhighlighter:    require('./ui/cellhighlighter'),
+  // TODO remove these from the export default and export them directly (prevents expensive copy)
+  // TODO don't use this.message use message directly (prevents expensive copy)
+  notifications:     notifications,
+  selector:          selector,
+  views:             views,
+  progress:          progress,
+  layout:            layout,
+  docpane:           docpane,
+  focusutils:        focusutils,
+  cellhighlighter:   cellhighlighter,
 
   activate(client) {
     this.client = client;