Review of "Strata" version 1

Details Page Preview

A fast clipboard manager. Storage, search and image decoding run in a separate Rust daemon. You need to install the Strata Daemon for this to work. See https://github.com/Edu4rdSHL/Strata#installing-the-daemon

Extension Homepage
https://github.com/Edu4rdSHL/Strata

No comments.

FAQ

Files

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

Shexli (experimental) manual_review 1

Shexli found 1 issue that may need reviewer attention.

EGO-A-005 manual_review

extensions should not access the clipboard directly

Direct clipboard access via `St.Clipboard.get_default()` requires reviewer scrutiny.

Review Guidelines

  • ui/panel.js:580
    St.Clipboard.get_default()

All Versions

Version Status
2 Active
1 Rejected

Previous Reviews on this Version

JustPerfection rejected
1. Please remove line 547-569 `extension.js`. GNOME Shell can load `stylesheet-light.css` and `stylesheet-dark.css` automatically. 2. Please use `connectObject()` and `disconnectObject()` so it is easier to track for cleanup. 3. Timeout should be removed before creating a new one (line 352 `extension.js`): [EGO Review Guidelines: Timeout](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources) 4. You cannot use timeout to delay the disable (line 370 `extension.js`). You can also add `donations` to the `metadata.json`, so people can donate to you if they want: [Extension Anatomy: donations](https://gjs.guide/extensions/overview/anatomy.html#donations)