Review of "Text Extractor" version 4

Details Page Preview

Extract text from screen using OCR. Select a screen region, and the text is automatically copied to clipboard. Requires Tesseract OCR.

Extension Homepage
https://github.com/Aditya190803/TextExtractor

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

All Versions

Version Status
6 Unreviewed
5 Active
4 Rejected
3 Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

JustPerfection rejected
1. #1 from previous review hasn't been addressed here. 2. Timeout needs to be removed on disable (line 201 `extension.js`). 3. Remove line 54-71 `extension.js` and use this instead: ```js this._cancellable?.cancel(); this._cancellable = null; ``` 4. You forgot to connect the cancellable. At line 372 do this: ```js this._cancellable.connect(() => this._subprocess.force_exit()); ```