Merged in notes-read-only (pull request #8)

fix: disable notes from toolbar in read only mode

app/components/SimpleEditor/ContainerEditor.js

@@ -48,10 +48,12 @@ class ContainerEditor extends SubstanceContainerEditor {
   }
 
   didMount () {
+    // TODO -- replace with super
     Surface.prototype.didMount.apply(this, arguments)
+
     this.container.on('nodes:changed', this.onContainerChange, this)
-    if (this.isEmpty()) this.createText()
 
+    if (this.isEmpty()) this.createText()
     if (this.isReadOnlyMode()) this.addTargetToLinks()
   }
 

app/components/SimpleEditor/elements/note/NoteTool.js

+import { isUndefined } from 'lodash'
 import { AnnotationTool } from 'substance'
 
-class NoteTool extends AnnotationTool {}
+class NoteTool extends AnnotationTool {
+  renderButton ($$) {
+    const el = super.renderButton($$)
+    const readOnly = this.isSurfaceReadOnly()
+
+    if (!isUndefined(readOnly)) el.attr('disabled', 'true')
+    return el
+  }
+
+  getSurface () {
+    const surfaceManager = this.context.surfaceManager
+    const containerId = this.context.controller.props.containerId
+
+    return surfaceManager.getSurface(containerId)
+  }
+
+  isSurfaceReadOnly () {
+    const surface = this.getSurface()
+    if (!surface) return
+
+    return surface.isReadOnlyMode()
+  }
+}
 
 NoteTool.type = 'note'