From e973412b5ea29685f4fa260d8eb44baae095fb81 Mon Sep 17 00:00:00 2001 From: Jules Laplace Date: Sat, 4 Jul 2020 16:37:19 +0200 Subject: rename timestamp to annotation --- animism-align/frontend/actions.js | 9 +- animism-align/frontend/api/index.js | 6 - animism-align/frontend/store.js | 5 +- animism-align/frontend/types.js | 4 +- .../frontend/views/align/align.actions.js | 11 +- .../frontend/views/align/align.reducer.js | 21 +++ .../components/annotations/annotation.form.js | 95 ++++++------- .../views/align/containers/timeline.container.js | 2 +- .../views/annotation/annotation.reducer.js | 21 +++ .../views/timestamp/components/timestamp.form.js | 153 --------------------- .../views/timestamp/containers/timestamp.edit.js | 53 ------- .../views/timestamp/containers/timestamp.index.js | 53 ------- .../views/timestamp/containers/timestamp.new.js | 44 ------ .../frontend/views/timestamp/timestamp.reducer.js | 21 --- 14 files changed, 110 insertions(+), 388 deletions(-) create mode 100644 animism-align/frontend/views/annotation/annotation.reducer.js delete mode 100644 animism-align/frontend/views/timestamp/components/timestamp.form.js delete mode 100644 animism-align/frontend/views/timestamp/containers/timestamp.edit.js delete mode 100644 animism-align/frontend/views/timestamp/containers/timestamp.index.js delete mode 100644 animism-align/frontend/views/timestamp/containers/timestamp.new.js delete mode 100644 animism-align/frontend/views/timestamp/timestamp.reducer.js (limited to 'animism-align/frontend') diff --git a/animism-align/frontend/actions.js b/animism-align/frontend/actions.js index 1b5f15e..4153296 100644 --- a/animism-align/frontend/actions.js +++ b/animism-align/frontend/actions.js @@ -1,5 +1,6 @@ import { bindActionCreators } from 'redux' -import { actions as crudActions } from './api' +// import { actions as crudActions } from './api' +import { crud_actions } from './api/crud.actions' import * as audioActions from './views/audio/audio.actions' import * as alignActions from './views/align/align.actions' @@ -7,6 +8,12 @@ import * as siteActions from './views/site/site.actions' import { store } from './store' +const crudActions = [ + 'paragraph', + 'annotation', + 'upload', +].reduce((a,b) => (a[b] = crud_actions(b)) && a, {}) + export default Object.keys(crudActions) .map(a => [a, crudActions[a]]) diff --git a/animism-align/frontend/api/index.js b/animism-align/frontend/api/index.js index 96c4e5c..05f6b8b 100644 --- a/animism-align/frontend/api/index.js +++ b/animism-align/frontend/api/index.js @@ -15,9 +15,3 @@ so you can do ... */ export { util } - -export const actions = [ - 'paragraph', - 'timestamp', - 'upload', -].reduce((a,b) => (a[b] = crud_actions(b)) && a, {}) diff --git a/animism-align/frontend/store.js b/animism-align/frontend/store.js index accf1b1..24542ff 100644 --- a/animism-align/frontend/store.js +++ b/animism-align/frontend/store.js @@ -9,7 +9,7 @@ import uploadReducer from './views/upload/upload.reducer' import alignReducer from './views/align/align.reducer' import audioReducer from './views/audio/audio.reducer' import paragraphReducer from './views/paragraph/paragraph.reducer' -import timestampReducer from './views/timestamp/timestamp.reducer' +import annotationReducer from './views/annotation/annotation.reducer' import siteReducer from './views/site/site.reducer' // import collectionReducer from './views/collection/collection.reducer' @@ -22,8 +22,7 @@ const createRootReducer = history => ( align: alignReducer, audio: audioReducer, paragraph: paragraphReducer, - timestamp: timestampReducer, - // collection: collectionReducer, + annotation: annotationReducer, }) ) diff --git a/animism-align/frontend/types.js b/animism-align/frontend/types.js index d888a0a..45c9caf 100644 --- a/animism-align/frontend/types.js +++ b/animism-align/frontend/types.js @@ -5,11 +5,13 @@ export const api = crud_type('api', []) export const upload = crud_type('upload', []) export const peaks = crud_type('peaks', []) export const text = crud_type('text', []) -export const timestamp = crud_type('timestamp', []) +export const annotation = crud_type('annotation', []) export const paragraph = crud_type('paragraph', []) export const align = crud_type('align', [ 'set_display_setting', 'set_temporary_annotation', + 'update_temporary_annotation', + 'update_temporary_annotation_settings', ]) export const audio = with_type('audio', [ diff --git a/animism-align/frontend/views/align/align.actions.js b/animism-align/frontend/views/align/align.actions.js index 82e4799..b3883ae 100644 --- a/animism-align/frontend/views/align/align.actions.js +++ b/animism-align/frontend/views/align/align.actions.js @@ -25,7 +25,7 @@ export const setCursor = cursor_ts => dispatch => ( dispatch({ type: types.align.set_display_setting, key: 'cursor_ts', value: cursor_ts }) ) -export const showNewTimestampForm = (start_ts, text) => dispatch => { +export const showNewAnnotationForm = (start_ts, text) => dispatch => { let croppedText; if (store.getState().align.annotation.start_ts) { croppedText = store.getState().align.annotation.text @@ -44,6 +44,13 @@ export const showNewTimestampForm = (start_ts, text) => dispatch => { }) } +export const updateAnnotationForm = (key, value) => dispatch => { + dispatch({ type: types.align.update_temporary_annotation, key, value }) +} +export const updateAnnotationSettings = (key, value) => dispatch => { + dispatch({ type: types.align.update_temporary_annotation_settings, key, value }) +} + const cutFirstSentence = text => { const textToCrop = text.trim().replace("\n", " ").split("\n")[0] let cropIndex = textToCrop.indexOf('. ') + 1 @@ -54,7 +61,7 @@ const cutFirstSentence = text => { return croppedText } -export const hideTimestampForm = () => dispatch => { +export const hideAnnotationForm = () => dispatch => { dispatch({ type: types.align.set_temporary_annotation, data: {} diff --git a/animism-align/frontend/views/align/align.reducer.js b/animism-align/frontend/views/align/align.reducer.js index 9064b56..f080c24 100644 --- a/animism-align/frontend/views/align/align.reducer.js +++ b/animism-align/frontend/views/align/align.reducer.js @@ -41,6 +41,27 @@ export default function alignReducer(state = initialState, action) { annotation: action.data, } + case types.align.update_temporary_annotation: + return { + ...state, + annotation: { + ...state.annotation, + [action.key]: action.value, + } + } + + case types.align.update_temporary_annotation_settings: + return { + ...state, + annotation: { + ...state.annotation, + settings: { + ...state.annotation.settings, + [action.key]: action.value, + } + } + } + default: return state } diff --git a/animism-align/frontend/views/align/components/annotations/annotation.form.js b/animism-align/frontend/views/align/components/annotations/annotation.form.js index 03e44e1..9432948 100644 --- a/animism-align/frontend/views/align/components/annotations/annotation.form.js +++ b/animism-align/frontend/views/align/components/annotations/annotation.form.js @@ -11,36 +11,42 @@ import { clamp } from '../../../../util' import { timeToPosition } from '../../align.util' import { Select } from '../../../../common' -const TIMESTAMP_TYPES = ['sentence', 'header'].map(name => ({ name, label: name })) +const ANNOTATION_TYPES = [ + 'sentence', 'header' +].map(name => ({ name, label: name })) class AnnotationForm extends Component { - state = { - data: {}, - } constructor(props){ super(props) this.handleChange = this.handleChange.bind(this) this.handleSelect = this.handleSelect.bind(this) this.handleKeyDown = this.handleKeyDown.bind(this) - } - componentDidMount(){ - this.setState({ - data: { ...this.props.annotation }, - }) - } - componentDidUpdate(prevProps){ - if (this.props.annotation !== prevProps.annotation) { - this.setState({ - data: { - ...this.props.annotation, - text: this.state.data.text, - type: this.state.data.text, - }, - }) - } + this.handleSubmit = this.handleSubmit.bind(this) } handleKeyDown(e) { + if (!e.metaKey && !e.ctrlKey) return + let { start_ts } = this.props.annotation + console.log(e.keyCode) switch (e.keyCode) { + case 38: // up + e.preventDefault() + start_ts -= 0.1 + actions.align.updateAnnotationForm('start_ts', start_ts) + actions.audio.seek(start_ts) + actions.align.setCursor(start_ts) + break + case 40: // down + e.preventDefault() + start_ts += 0.1 + actions.align.updateAnnotationForm('start_ts', start_ts) + actions.audio.seek(start_ts) + actions.align.setCursor(start_ts) + break + case 83: // ctrl-S + e.preventDefault() + this.handleSubmit() + default: + break } } handleChange(e) { @@ -48,62 +54,57 @@ class AnnotationForm extends Component { this.handleSelect(name, value) } handleSelect(name, value) { - this.setState({ - data: { - ...this.state.data, - [name]: value, - } - }) + actions.align.updateAnnotationForm(name, value) } handleSubmit() { - const { data } = this.state - if (data.id === 'new') { - delete data.id - actions.graph.create(data) + const { annotation } = this.props + if (annotation.id === 'new') { + delete annotation.id + actions.annotation.create(annotation) .then(response => { console.log(response) - actions.align.hideTimestampForm() + actions.align.hideAnnotationForm() }) } else { - actions.graph.update(data) + actions.annotation.update(annotation) .then(response => { console.log(response) - actions.align.hideTimestampForm() + actions.align.hideAnnotationForm() }) } } render() { - const { timeline } = this.props - const { data } = this.state - if (!data.start_ts) return
+ const { timeline, annotation } = this.props + if (!annotation.start_ts) return
return (
- {data.type === 'sentence' && this.renderTextarea()} - {data.type === 'header' && this.renderTextarea()} + {annotation.type === 'sentence' && this.renderTextarea()} + {annotation.type === 'header' && this.renderTextarea()}