From e1593127a1ca55fe1f2a44241c2cdbf63dcb5dc3 Mon Sep 17 00:00:00 2001
From: Mohamed El Mahallawy <mmahalwy@gmail.com>
Date: Sat, 9 Feb 2019 20:07:58 -0800
Subject: [PATCH] generate cleanup

---
 src/components/ChapterInfoPanel.tsx    |  7 ++++---
 src/components/FontText.tsx            |  5 +++--
 src/components/Line.tsx                |  2 +-
 src/components/Share.tsx               |  5 ++++-
 src/components/Verse.tsx               | 20 ++++++++++++++++++--
 src/components/Word.tsx                |  4 ++--
 src/components/audioplayer/Track.tsx   |  7 +++----
 src/components/audioplayer/Wrapper.tsx |  2 +-
 src/components/dls/ButtonLink.tsx      |  3 ++-
 src/components/verse/Badge.tsx         | 11 ++++++++---
 src/helpers/fontsStyle.ts              |  2 +-
 src/redux/reducers/audioplayer.ts      |  3 ++-
 src/redux/reducers/chapterInfos.ts     |  3 ++-
 src/redux/reducers/footNotes.ts        |  3 ++-
 src/redux/reducers/lines.ts            |  3 ++-
 src/redux/reducers/options.ts          |  6 +++---
 src/redux/reducers/search.ts           |  2 +-
 src/styles/main.global.scss            |  4 ----
 18 files changed, 59 insertions(+), 33 deletions(-)

diff --git a/src/components/ChapterInfoPanel.tsx b/src/components/ChapterInfoPanel.tsx
index d1ba0fc81..e38fe4491 100644
--- a/src/components/ChapterInfoPanel.tsx
+++ b/src/components/ChapterInfoPanel.tsx
@@ -37,15 +37,15 @@ const Close = styled.button`
   position: absolute;
   right: 15px;
   top: 15px;
-  background: ${({ theme }) => lighten(0.1, theme.textMuted)};
   height: 26px;
   width: 26px;
   padding: 7px 8px;
   font-size: 10px;
   border-radius: 16px;
-  color: #fff;
+  color: ${({ theme }) => theme.textMuted};
   z-index: 20;
   cursor: pointer;
+  border: none;
   &:hover {
     opacity: 0.8;
   }
@@ -77,6 +77,7 @@ const Info = styled.div`
 const Container = styled.div`
   overflow-y: auto;
   margin-bottom: 30px;
+  position: relative;
   height: 0;
   max-height: 0;
   min-height: 0;
@@ -146,7 +147,7 @@ const ChapterInfo: React.SFC<Props> = ({
     setSetting({ isShowingChapterInfo: !isShowingChapterInfo });
 
   return (
-    <Container className="col-xs-12 chapter-info">
+    <Container>
       {setSetting && <Close className="ss-delete" onClick={handleClose} />}
       <div className="row" style={{ width: '100%', height: '100%', margin: 0 }}>
         {chapter && (
diff --git a/src/components/FontText.tsx b/src/components/FontText.tsx
index a21fae418..d0f20944e 100644
--- a/src/components/FontText.tsx
+++ b/src/components/FontText.tsx
@@ -1,7 +1,7 @@
 import styled from 'styled-components';
 import { darken } from 'polished';
 
-export default styled.div`
+export default styled.div<{ readingMode?: boolean }>`
   white-space: pre-line;
   color: #000;
   width: 100%;
@@ -15,12 +15,13 @@ export default styled.div`
   -moz-user-select: none;
   -ms-user-select: none;
   user-select: none;
+  font-size: 3.5rem;
   b,
   span {
     border-color: transparent;
     border-width: 0 0 1px 0;
     border-style: solid;
-    float: right;
+    float: ${({ readingMode }) => (readingMode ? 'none' : 'right')};
     &.active {
       color: ${({ theme }) => darken(0.05, theme.brandPrimary)};
       border-color: ${({ theme }) => darken(0.15, theme.brandPrimary)};
diff --git a/src/components/Line.tsx b/src/components/Line.tsx
index 7e0bf5866..7261aa869 100644
--- a/src/components/Line.tsx
+++ b/src/components/Line.tsx
@@ -44,7 +44,7 @@ const Line: React.SFC<Props> = ({ line, useTextFont }: Props) => {
   });
 
   return (
-    <FontText className="row text-arabic">
+    <FontText className="row text-arabic" readingMode>
       <div className="col-md-12">
         <StyledLine className="text-center">{text}</StyledLine>
       </div>
diff --git a/src/components/Share.tsx b/src/components/Share.tsx
index def990ddf..8b7586c6c 100644
--- a/src/components/Share.tsx
+++ b/src/components/Share.tsx
@@ -29,12 +29,15 @@ const Container = styled.div<{ inline?: boolean }>`
 const FacebookButton = styled(FacebookShareButton)`
   background-repeat: no-repeat;
   background-size: 12px;
-  padding: 1px 5px 0px 0px;
+  padding-top: 1px;
+  display: inline-block;
+  padding-right: 8px;
 `;
 
 const TwitterButton = styled(TwitterShareButton)`
   background-repeat: no-repeat;
   background-size: 21px;
+  display: inline-block;
 `;
 
 type Props = {
diff --git a/src/components/Verse.tsx b/src/components/Verse.tsx
index 14321c1d1..b1fcfc460 100644
--- a/src/components/Verse.tsx
+++ b/src/components/Verse.tsx
@@ -12,6 +12,20 @@ import { SetCurrentVerseKey, Play, Pause } from '../redux/actions/audioplayer';
 import { FetchFootNote } from '../redux/actions/footNotes';
 import FootNote from './FootNote';
 
+const backgroundColor = ({
+  highlight,
+  isNightMode,
+}: {
+  highlight?: boolean;
+  isNightMode?: boolean;
+}): string => {
+  if (highlight) {
+    return isNightMode ? '#151414' : '#F5FBF7';
+  }
+
+  return '';
+};
+
 // TODO: Change this
 const VerseNode = styled(Element)<{
   highlight?: boolean;
@@ -20,8 +34,7 @@ const VerseNode = styled(Element)<{
 }>`
   padding: 2.5% 0;
   border-bottom: 1px solid rgba(${({ textMuted }) => textMuted}, 0.5);
-  background-color: ${({ highlight, isNightMode }) =>
-    highlight ? (isNightMode ? '#151414' : '#F5FBF7') : ''};
+  background-color: ${backgroundColor};
   .text-info {
     color: ${({ theme }) => theme.brandInfo};
     &:hover {
@@ -54,6 +67,7 @@ const defaultProps: $TsFixMe = {
   match: null,
   currentVerse: null,
   footNote: null,
+  isNightMode: false,
 };
 
 type Props = {
@@ -110,7 +124,9 @@ class Verse extends Component<Props> {
       fetchFootNote,
       isNightMode,
     } = this.props;
+
     const translations: Array<$TsFixMe> = verse.translations || [];
+
     return (
       <VerseNode
         name={`verse:${verse.verseKey}`}
diff --git a/src/components/Word.tsx b/src/components/Word.tsx
index f22165456..ec82edf2e 100644
--- a/src/components/Word.tsx
+++ b/src/components/Word.tsx
@@ -134,11 +134,11 @@ class Word extends Component<Props> {
     } = this.props;
 
     let text = '';
-
+    const { verseKey } = word;
     const className = `${useTextFont ? 'text-' : ''}${word.className} ${
       word.charType
     } ${word.highlight ? word.highlight : ''}`;
-    const id = `word-${word.verseKey.replace(/:/, '-')}-${audioPosition}`;
+    const id = `word-${(verseKey || '').replace(/:/, '-')}-${audioPosition}`;
 
     if (useTextFont) {
       if (word.charType === WORD_TYPES.CHAR_TYPE_END) {
diff --git a/src/components/audioplayer/Track.tsx b/src/components/audioplayer/Track.tsx
index 228a66edb..35ef9fb2f 100644
--- a/src/components/audioplayer/Track.tsx
+++ b/src/components/audioplayer/Track.tsx
@@ -3,11 +3,10 @@ import PropTypes from 'prop-types';
 import styled from 'styled-components';
 
 const Container = styled.div`
-  height: 10px;
+  height: ${({ theme }) => `${theme.unit}px`};
   width: 100%;
   background-color: #f7f7f7;
   cursor: pointer;
-  margin-bottom: 5px;
   position: absolute;
   top: 0;
   left: 0;
@@ -22,8 +21,8 @@ const Progress = styled.div`
 
   &:after {
     content: '';
-    height: 20px;
-    width: 20px;
+    height: ${({ theme }) => `${theme.unit * 2}px`};
+    width: ${({ theme }) => `${theme.unit * 2}px`};
     border-radius: 10px;
     position: absolute;
     right: -3px;
diff --git a/src/components/audioplayer/Wrapper.tsx b/src/components/audioplayer/Wrapper.tsx
index ef86fb5bd..ad7a3c056 100644
--- a/src/components/audioplayer/Wrapper.tsx
+++ b/src/components/audioplayer/Wrapper.tsx
@@ -7,7 +7,7 @@ const Wrapper = styled.div<{ isNightMode?: boolean }>`
   user-select: none;
   height: auto;
   z-index: 1;
-  padding: 20px 20px 10px;
+  padding: 1rem 1rem 0.5rem;
   background: ${({ theme, isNightMode }) =>
     isNightMode ? theme.colors.tuatara : theme.colors.white};
   box-shadow: 0 0 0.5rem 0 rgba(0, 0, 0, 0.2);
diff --git a/src/components/dls/ButtonLink.tsx b/src/components/dls/ButtonLink.tsx
index fff2fb404..6ec554901 100644
--- a/src/components/dls/ButtonLink.tsx
+++ b/src/components/dls/ButtonLink.tsx
@@ -6,7 +6,7 @@ const ButtonLink = styled.button<{ active?: boolean }>`
   border-color: transparent;
   background-color: transparent;
   box-shadow: none;
-  display: inline-block;
+  display: block;
   margin-bottom: 0;
   font-weight: normal;
   text-align: center;
@@ -20,6 +20,7 @@ const ButtonLink = styled.button<{ active?: boolean }>`
   line-height: 1.42857;
   border-radius: 4px;
   user-select: none;
+  padding: 0.25rem 0rem;
 
   ${({ active, theme }) =>
     active ? `color: ${theme.brandPrimary}` : ''} &:hover {
diff --git a/src/components/verse/Badge.tsx b/src/components/verse/Badge.tsx
index b8babdda5..13c28e386 100644
--- a/src/components/verse/Badge.tsx
+++ b/src/components/verse/Badge.tsx
@@ -10,7 +10,7 @@ const Label = styled.span`
   display: inline-block;
   margin-bottom: 15px;
   font-weight: 300;
-  color: ${props => props.theme.textColor};
+  color: ${({ theme }) => theme.textColor};
   &:hover {
     opacity: 0.7;
   }
@@ -21,9 +21,13 @@ const propTypes = {
   verse: VerseShape.isRequired,
 };
 
+const defaultProps = {
+  isSearched: false,
+};
+
 type Props = {
-  isSearched?: boolean,
-  verse: VerseShape,
+  isSearched?: boolean;
+  verse: VerseShape;
 };
 
 const Badge: React.SFC<Props> = ({ isSearched, verse }: Props) => {
@@ -57,5 +61,6 @@ const Badge: React.SFC<Props> = ({ isSearched, verse }: Props) => {
 };
 
 Badge.propTypes = propTypes;
+Badge.defaultProps = defaultProps;
 
 export default Badge;
diff --git a/src/helpers/fontsStyle.ts b/src/helpers/fontsStyle.ts
index d6455183f..45011ae07 100644
--- a/src/helpers/fontsStyle.ts
+++ b/src/helpers/fontsStyle.ts
@@ -4,7 +4,7 @@ import config from '../../config';
 
 const baseUrl = config('fontsURL');
 
-const makeFont = (pageNumber: string | number) => `
+export const makeFont = (pageNumber: string | number) => `
 @font-face {
   font-family: p${pageNumber};
   src: url('${baseUrl}/fonts/ttf/p${pageNumber}.ttf')
diff --git a/src/redux/reducers/audioplayer.ts b/src/redux/reducers/audioplayer.ts
index 82c0e56aa..d2f49e87e 100644
--- a/src/redux/reducers/audioplayer.ts
+++ b/src/redux/reducers/audioplayer.ts
@@ -53,7 +53,8 @@ export default (state = INITIAL_STATE, action: $TsFixMe) => {
       return handle(state, action, {
         success: prevState => {
           const audioFile: $TsFixMe = camelcaseKeys(
-            action.payload.audio_file || {}
+            action.payload.audio_file || {},
+            { deep: true }
           );
 
           return {
diff --git a/src/redux/reducers/chapterInfos.ts b/src/redux/reducers/chapterInfos.ts
index 149d04dae..556f630a4 100644
--- a/src/redux/reducers/chapterInfos.ts
+++ b/src/redux/reducers/chapterInfos.ts
@@ -33,7 +33,8 @@ export default (state = INITIAL_STATE, action: $TsFixMe) => {
           entities: {
             ...state.entities,
             [action.meta.chapterId]: camelcaseKeys(
-              action.payload.chapter_info || {}
+              action.payload.chapter_info || {},
+              { deep: true }
             ),
           },
         }),
diff --git a/src/redux/reducers/footNotes.ts b/src/redux/reducers/footNotes.ts
index 3f1969339..6363a37f6 100644
--- a/src/redux/reducers/footNotes.ts
+++ b/src/redux/reducers/footNotes.ts
@@ -30,7 +30,8 @@ export default (state = INITIAL_STATE, action: $TsFixMe) => {
           entities: {
             ...prevState.entities,
             [action.meta.verseKey]: camelcaseKeys(
-              action.payload.foot_note || {}
+              action.payload.foot_note || {},
+              { deep: true }
             ),
           },
         }),
diff --git a/src/redux/reducers/lines.ts b/src/redux/reducers/lines.ts
index 7764dccb9..ac6bdcd85 100644
--- a/src/redux/reducers/lines.ts
+++ b/src/redux/reducers/lines.ts
@@ -28,7 +28,8 @@ export default (state = INITIAL_STATE, action: $TsFixMe) => {
         }),
         success: prevState => {
           const verses = camelcaseKeys(
-            keyBy(action.payload.verses, 'verse_key')
+            keyBy(action.payload.verses, 'verse_key'),
+            { deep: true }
           );
           const lines = prevState.entities;
 
diff --git a/src/redux/reducers/options.ts b/src/redux/reducers/options.ts
index c6e481a9e..48d116212 100644
--- a/src/redux/reducers/options.ts
+++ b/src/redux/reducers/options.ts
@@ -40,7 +40,7 @@ export default (state = INITIAL_STATE, action: $TsFixMe) => {
         success: prevState => ({
           ...prevState,
           recitations: action.payload.recitations.map((obj: $TsFixMe) =>
-            camelcaseKeys(obj || {})
+            camelcaseKeys(obj || {}, { deep: true })
           ),
         }),
       });
@@ -58,7 +58,7 @@ export default (state = INITIAL_STATE, action: $TsFixMe) => {
         success: prevState => ({
           ...prevState,
           translations: action.payload.translations.map((obj: $TsFixMe) =>
-            camelcaseKeys(obj || {})
+            camelcaseKeys(obj || {}, { deep: true })
           ),
         }),
       });
@@ -76,7 +76,7 @@ export default (state = INITIAL_STATE, action: $TsFixMe) => {
         success: prevState => ({
           ...prevState,
           tafsirs: action.payload.tafsirs.map((obj: $TsFixMe) =>
-            camelcaseKeys(obj || {})
+            camelcaseKeys(obj || {}, { deep: true })
           ),
         }),
       });
diff --git a/src/redux/reducers/search.ts b/src/redux/reducers/search.ts
index d3885f353..60fe2e8bc 100644
--- a/src/redux/reducers/search.ts
+++ b/src/redux/reducers/search.ts
@@ -49,7 +49,7 @@ export default (state = INITIAL_STATE, action: $TsFixMe) => {
           perPage: action.payload.per_page,
           took: action.payload.took,
           query: action.payload.query,
-          entities: camelcaseKeys(action.payload.results || {}),
+          entities: camelcaseKeys(action.payload.results || {}, { deep: true }),
         }),
       });
     }
diff --git a/src/styles/main.global.scss b/src/styles/main.global.scss
index 911a48fdd..a1a6b3624 100644
--- a/src/styles/main.global.scss
+++ b/src/styles/main.global.scss
@@ -19,10 +19,6 @@ body {
   }
 }
 
-.row {
-  margin-right: 0em !important;
-}
-
 .highlight {
   background-color: #f5fbf7;
 }