Merge branch 'track-button-disabled' into 'staging'

fix when track changes is enabled and when it is not

See merge request !2

app/components/SimpleEditor/ContainerEditor.js

@@ -224,8 +224,16 @@ class ContainerEditor extends SubstanceContainerEditor {
   disableToolbar () {
     const commandStates = this.getCommandStates()
 
+    const allowed = [
+      'comment',
+      'redo',
+      'save',
+      'track-change-enable',
+      'track-change-toggle-view',
+      'undo'
+    ]
+
     each(keys(commandStates), key => {
-      const allowed = ['comment', 'save', 'undo', 'redo', 'track-change-toggle-view']
       if (!includes(allowed, key)) commandStates[key].disabled = true
     })
   }

app/components/SimpleEditor/elements/track_change/TrackChangeControlCommand.js

-import {Command} from 'substance'
+import { Command } from 'substance'
 
 class TrackChangeControlCommand extends Command {
   getCommandState (params) {
-    let newState = {
+    const newState = {
       disabled: false,
       active: false
     }

app/components/SimpleEditor/elements/track_change/TrackChangeControlTool.js

 import { Tool } from 'substance'
 
 class TrackChangeControlTool extends Tool {
+  constructor (parent, props, config) {
+    super(parent, props, config)
+    this.disabled = false
+  }
 
   getClassNames () {
     return 'sm-target-track-change-enable'
   }
 
+  willReceiveProps (props) {
+    if (!this.canAct()) this.disabled = true
+  }
+
   renderButton ($$) {
     const el = super.renderButton($$)
 
-    let trackChangesMode = this.isTrackChangesOn()
+    const trackChangesMode = this.isTrackChangesOn()
     if (trackChangesMode === true) el.addClass('track-changes-active')
 
     if (!this.canAct()) el.attr('disabled', 'true')