Skip to content
This repository was archived by the owner on Feb 19, 2025. It is now read-only.

Commit 87bff4c

Browse files
committed
Fixed issue where you can deselect the last page
1 parent 934e8e1 commit 87bff4c

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

src/components/PageExtractionModal/PageExtractionThumbnail.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,8 +87,8 @@ class PageExtractionThumbnail extends React.Component<PageExtractionThumbnailInp
8787
if (this.state.isDisabled) {
8888
return;
8989
}
90-
this.props.onClick(this.props.pageNumber, !this.state.isSelected);
9190
this.setState({ isSelected: !this.state.isSelected });
91+
this.props.onClick(this.props.pageNumber, !this.state.isSelected);
9292
};
9393
render(): JSX.Element {
9494
const { thumbnail, isHover, isSelected } = this.state;
@@ -106,6 +106,7 @@ class PageExtractionThumbnail extends React.Component<PageExtractionThumbnailInp
106106
type="checkbox"
107107
style={{ position: "absolute", top: 0, left: 0 }}
108108
disabled={this.state.isDisabled}
109+
defaultChecked={isSelected}
109110
checked={isSelected}
110111
onClick={this.onClick}
111112
/>

src/components/PageExtractionModal/index.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -72,9 +72,9 @@ class PageExtractionModal extends React.Component<PageExtractionModalInputProps,
7272
pageInput: `${this.state.pageInput},${pageNumber}`
7373
});
7474
} else {
75-
this.setState({
76-
pageInput: this.parsePageInputString(this.state.pageInput.replace(`${pageNumber}`, ""))
77-
});
75+
const parsedInput = this.parsePageInputString(this.state.pageInput.replace(`${pageNumber}`, ""));
76+
this.setState({ pageInput: parsedInput });
77+
this.trigger(parsedInput);
7878
}
7979
};
8080
parsePageInputString = (input: string) => {

0 commit comments

Comments
 (0)