Skip to content

Commit

Permalink
Remove TODO for passing state
Browse files Browse the repository at this point in the history
It looks like the page and simultaneousPages can be undefined, so better to avoid overwrite data with undefined
  • Loading branch information
beatrycze-volk committed Aug 2, 2024
1 parent fea6f47 commit 5233709
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion Resources/Public/JavaScript/PageView/Controller.js
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,6 @@ class dlfController {
* @param {dlf.StateChangeDetail} detail
*/
changeState(detail) {
// TODO(client-side): Consider passing full new state in stateChanged event, then reduce usage of currentPageNo and simultaneousPages properties
if (detail.page !== undefined) {
this.doc.state.page = detail.page;
}
Expand Down

0 comments on commit 5233709

Please sign in to comment.