Review of "Screen word translate" version 40

Details Page Preview

Translate word on the screen. Default web address is translate.google.com, you can add the web address for your own language. Also you can contribute your web address to my github repo. Use hotkey Ctrl+Alt+j to toggle the function. Use hotkey Ctrl+Alt+o to show popup window

Extension Homepage
https://github.com/sunwxg/gnome-shell-extension-dict

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

Previous Reviews on this Version

JustPerfection active
JustPerfection rejected
1. Please remove line 5 and 6 (extension.js). 2. You should remove the timeout on destroy (line 221 and 335 extension.js): https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources
JustPerfection posted a review
Also gsettings object shouldn't be in global scope (line 27 prefs.js)