Skip to content
Snippets Groups Projects
Commit 31a06083 authored by john's avatar john
Browse files

do not open line height for tracked changes if the user cannot resolve them

parent 5e524d1d
No related branches found
No related tags found
No related merge requests found
import { includes, some } from 'lodash'
import { import {
ProseEditor, ProseEditor,
ProseEditorOverlayTools, ProseEditorOverlayTools,
...@@ -6,8 +8,6 @@ import { ...@@ -6,8 +8,6 @@ import {
TOCProvider TOCProvider
} from 'substance' } from 'substance'
import {includes} from 'lodash'
import Comments from './panes/Comments/CommentBoxList' import Comments from './panes/Comments/CommentBoxList'
import CommentsProvider from './panes/Comments/CommentsProvider' import CommentsProvider from './panes/Comments/CommentsProvider'
import ContainerEditor from './ContainerEditor' import ContainerEditor from './ContainerEditor'
...@@ -51,7 +51,10 @@ class Editor extends ProseEditor { ...@@ -51,7 +51,10 @@ class Editor extends ProseEditor {
} }
render ($$) { render ($$) {
var el = $$('div').addClass('sc-prose-editor') const { trackChangesView } = this.state
const canToggleTrackChanges = this.canToggleTrackChanges()
const el = $$('div').addClass('sc-prose-editor')
// left side: editor and toolbar // left side: editor and toolbar
var toolbar = this._renderToolbar($$) var toolbar = this._renderToolbar($$)
...@@ -109,7 +112,10 @@ class Editor extends ProseEditor { ...@@ -109,7 +112,10 @@ class Editor extends ProseEditor {
) )
) )
if (this.state.trackChangesView && !this.isAuthor()) el.addClass('track-changes-mode') if (trackChangesView && canToggleTrackChanges) {
el.addClass('track-changes-mode')
}
return el return el
} }
...@@ -135,10 +141,10 @@ class Editor extends ProseEditor { ...@@ -135,10 +141,10 @@ class Editor extends ProseEditor {
} }
} }
isAuthor () { canToggleTrackChanges () {
const author = includes(this.props.user.roles, 'author') const { user } = this.props
const accepted = ['admin', 'production-editor', 'copy-editor']
return author return some(accepted, (role) => includes(user.roles, role))
} }
scrollTo (nodeId) { scrollTo (nodeId) {
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment