Skip to content

Commit a8a7214

Browse files
Merge pull request #46 from SAP/pagination-tweaks
fix eslint warnings, remove isRequired for itemsPerPage propType
2 parents e9f2a1a + 4f0d3ca commit a8a7214

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/Pagination/Pagination.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ export class Pagination extends Component {
3131

3232
this.setState(
3333
prevState => {
34-
selectedPage: prevState.selectedPage--;
34+
return { selectedPage: --prevState.selectedPage };
3535
},
3636
() => this.props.onClick(this.state.selectedPage)
3737
);
@@ -45,7 +45,7 @@ export class Pagination extends Component {
4545

4646
this.setState(
4747
prevState => {
48-
selectedPage: prevState.selectedPage++;
48+
return { selectedPage: ++prevState.selectedPage };
4949
},
5050
() => this.props.onClick(this.state.selectedPage)
5151
);
@@ -115,7 +115,7 @@ export class Pagination extends Component {
115115
}
116116

117117
Pagination.propTypes = {
118-
itemsPerPage: PropTypes.number.isRequired,
118+
itemsPerPage: PropTypes.number,
119119
itemsTotal: PropTypes.number.isRequired,
120120
onClick: PropTypes.func.isRequired,
121121
initialPage: PropTypes.number,

0 commit comments

Comments
 (0)