Skip to content
This repository was archived by the owner on Jun 28, 2021. It is now read-only.

Fixed an issue that was causing the wrong word to highlight #831

Merged
naveed-ahmad merged 3 commits into from
Jun 24, 2017
Merged

Fixed an issue that was causing the wrong word to highlight #831

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
f0ff1e7
Fixed an issue that was causing the wrong word to highlight
iMoe037 Jun 24, 2017
809ddcf
Merge branch 'master' into fix/ayah-highlight
naveed-ahmad Jun 24, 2017
d626fd2
Merge branch 'master' into fix/ayah-highlight
naveed-ahmad Jun 24, 2017
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
Next Next commit
Fixed an issue that was causing the wrong word to highlight
  • Loading branch information
@iMoe037
iMoe037 committed Jun 24, 2017
commit f0ff1e7b9ce1b0507b8914342dd46f35a982c85f
8 changes: 6 additions & 2 deletions src/components/Word/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,12 @@ class Word extends Component {
const highlight = currentVerse === word.verseKey && isPlaying
? 'highlight'
: '';
const className = `${useTextFont ? 'text-' : ''}${word.className} ${word.charType} ${highlight} ${word.highlight ? word.highlight : ''}`;
const id = `word-${word.verseKey.replace(/:/, '-')}-${audioPosition}`;
const className = `${useTextFont
? 'text-'
: ''}${word.className} ${word.charType} ${highlight} ${word.highlight
? word.highlight
: ''}`;
const id = `word-${word.verseKey.replace(/:/, '-')}-${word.position}`;

if (useTextFont) {
if (word.charType === CHAR_TYPE_END) {
Expand Down