Skip to content

Commit 5e188a3

Browse files
committed
- ayah's have link (discuss, maybe not needed) (quran#379)
- Fix audio player playing text - Remove unneeded padding on mobile - Hide Share on mobile devices (native share is built into the browsers)
1 parent 1eb6d9c commit 5e188a3

File tree

7 files changed

+20
-7
lines changed

7 files changed

+20
-7
lines changed

src/components/Audioplayer/style.scss

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@
3333
text-align: center;
3434

3535
:global(li){
36-
padding-left: 15px;
3736
padding-right: 15px;
3837
display: table-cell;
3938
}

src/components/Ayah/index.js

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,13 @@ export default class Ayah extends Component {
189189
);
190190
}
191191

192-
return content;
192+
return (
193+
<Link
194+
to={`/${this.props.ayah.surahId}:${this.props.ayah.ayahNum}`}
195+
data-metrics-event-name="Ayah:Link"
196+
>
197+
{content}
198+
</Link>);
193199
}
194200

195201
renderControls() {

src/components/Share/index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ export default class Share extends Component {
1818
const surahUrl = encodeURIComponent(`http://quran.com/${surahId}`);
1919

2020
return (
21-
<div className={Style.shareContainer}>
21+
<div className={`${Style.shareContainer} hidden-xs`}>
2222

2323
<i
2424
onClick={() => this.onClickPopup(`https://www.facebook.com/sharer/sharer.php?u=${surahUrl}`, 'Facebook')}

src/components/Share/style.scss

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,9 @@
1+
@import '../../styles/variables';
2+
13
.shareContainer {
24
position: relative;
3-
bottom: 2px;
45
left: 5px;
6+
top: 2px;
57

68
.iconContainer {
79
padding: 0 30px 10px 0;
@@ -30,5 +32,3 @@
3032
}
3133

3234
}
33-
34-

src/components/SurahInfo/style.scss

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,11 @@ $transition-speed: 0.75s;
99
min-height: 0px;
1010
transition: max-height $transition-speed, height $transition-speed;
1111

12+
13+
@media(max-width: $screen-xs-max) {
14+
margin-bottom: 0;
15+
}
16+
1217
&.show{
1318
max-height: 600px;
1419
max-height: 70vh;

src/containers/Home/style.scss

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,10 @@
55
padding: 50px 0px;
66
font-size: 14px;
77
margin: 0px;
8+
9+
@media(max-width: $screen-xs-max) {
10+
padding: 10px 0;
11+
}
812
}
913

1014
.lastVisit{

src/containers/Surah/index.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -398,7 +398,6 @@ export default class Surah extends Component {
398398
onReadingModeToggle={toggleReadingMode}
399399
/>
400400
</li>
401-
<li>|</li>
402401
<li><Share surah={surah} /></li>
403402
</ul>
404403
</Col>

0 commit comments

Comments
 (0)