Review of "Clipman" version 66

Details Page Preview

Simple but powerful clipboard manager.

Extension Homepage
https://github.com/popov895/Clipman

No comments.

Diff Against

Files

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

Shexli (experimental)

Shexli has not produced a result yet.

All Versions

Previous Reviews on this Version

JustPerfection rejected
All dynamically stored memory must be cleared or freed in `disable()` (line 1377 `extension.js`): [EGO Review Guidelines: Static Objects](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#only-use-initialization-for-static-resources)
popov895 posted a review
The `panelIndicator.state.history` is a simple array of JS objects (line 1073 in `extension.js`).
popov895 posted a review
This code is needed to avoid losing clipboard history when locking the screen.
JustPerfection posted a review
That is still **dynamically stored memory** that should be cleared. If you want to restore the history, you can store those as JSON in gsettings and restore it on enable.
popov895 posted a review
How can I delete my extension from EGO?
JustPerfection posted a review
You can inactivate all versions from your extension's page: https://extensions.gnome.org/extension/4958/clipman/