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

Commit

Permalink
fixes #359
Browse files Browse the repository at this point in the history
  • Loading branch information
mmahalwy committed Jun 25, 2016
1 parent d1a18bc commit d2553eb
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/containers/Surah/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -208,8 +208,8 @@ export default class Surah extends Component {

handleOptionChange = (payload) => {
const { setOption, loadAyahs, surah, ayahIds, options } = this.props; // eslint-disable-line no-shadow, max-len

This comment has been minimized.

Copy link
@mahboob-awan

mahboob-awan Jun 25, 2016

Contributor

remove ayahIds from const{..}

const from = ayahIds.first();
const to = ayahIds.last();
const from = this.getFirst();
const to = this.getLast();

setOption(payload);

Expand Down Expand Up @@ -262,7 +262,7 @@ export default class Surah extends Component {

handleLazyLoadAyahs = (callback) => {
const { loadAyahs, ayahIds, surah, isEndOfSurah, options } = this.props; // eslint-disable-line no-shadow, max-len

This comment has been minimized.

Copy link
@mahboob-awan

mahboob-awan Jun 25, 2016

Contributor

As above

const range = [ayahIds.first(), ayahIds.last()];
const range = [this.getFirst(), this.getLast()];

let size = 10;

Expand Down

2 comments on commit d2553eb

@mahboob-awan
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ayahIds.has(to)) will also change?

@mmahalwy
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nope, that is fine

Please sign in to comment.