Skip to content
This repository was archived by the owner on Jun 28, 2021. It is now read-only.
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
lint happy
  • Loading branch information
naveed-ahmad committed Mar 27, 2017
commit c46c57523652d2664fd2fc03fa64feb8be84e08b
16 changes: 10 additions & 6 deletions src/components/Translation/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,26 +16,28 @@ class Translation extends Component {

componentDidMount() {
const { index } = this.props;
let trans;

if (__CLIENT__) {
var trans = document.getElementById(`trans${index}`).children[1];
trans.addEventListener('click', this.fetchFootNote, true);
trans = document.getElementById(`trans${index}`).children[1]; // eslint-disable-line no-undef
trans.addEventListener('click', this.fetchFootNote, true);
}
}

componentWillUnmount() {
const { index } = this.props;
let trans;

if (__CLIENT__) {
var trans = document.getElementById(`trans${index}`).children[1];
trans = document.getElementById(`trans${index}`).children[1]; // eslint-disable-line no-undef
trans.removeEventListener('click', this.fetchFootNote, true);
}
}

fetchFootNote = (event) => {
const { loadFootNote } = this.props;
const { loadFootNote } = this.props; // eslint-disable-line no-shadow

if(event.target.nodeName == 'SUP' && event.target.attributes.foot_note) {
if (event.target.nodeName === 'SUP' && event.target.attributes.foot_note) {
event.preventDefault();
loadFootNote(event.target.attributes.foot_note.value);
}
Expand All @@ -60,4 +62,6 @@ class Translation extends Component {
}
}

export default connect(state => ({}), { loadFootNote })(Translation);
export default connect(state => ({}), // eslint-disable-line no-unused-vars
{ loadFootNote }
)(Translation);
2 changes: 1 addition & 1 deletion src/components/Verse/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ class Verse extends Component {
const { verse, match } = this.props;
const array = match || verse.translations || [];

return array.map((translation) => (
return array.map(translation => (
<Translation translation={translation} index={translation.id} key={translation.id} />
));
}
Expand Down
6 changes: 3 additions & 3 deletions src/containers/App/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -60,18 +60,18 @@ class App extends Component {
footNote,
loadingFootNote,
removeMedia, // eslint-disable-line no-shadow
removeFootNote,
removeFootNote, // eslint-disable-line no-shadow
...props
} = this.props;
debug('component:APPLICATION', 'Render');
let footNoteText;
if(footNote) {

if (footNote) {
footNoteText = footNote.text;
} else {
footNoteText = <Loader isActive={loadingFootNote} />;
}


return (
<div>
<Helmet {...config.app.head} />
Expand Down
4 changes: 2 additions & 2 deletions src/redux/actions/footNote.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,9 @@ import {
LOAD_FOOT_NOTE_FAIL
} from 'redux/constants/footNote.js';

export const loadFootNote = (footNoteId) => ({
export const loadFootNote = footNoteId => ({
types: [LOAD_FOOT_NOTE, LOAD_FOOT_NOTE_SUCCESS, LOAD_FOOT_NOTE_FAIL],
promise: client => client.get('/api/v3/foot_notes/'+footNoteId)
promise: client => client.get(`/api/v3/foot_notes/${footNoteId}`)
});

export const removeFootNote = () => ({
Expand Down