Review of "WinClip" version 1

Details Page Preview

A Windows-style clipboard manager. Super+V opens an overlay at the pointer, over whatever window you are in. • Clipboard — text and image history, with pinned entries kept at the top • GIF — GIFs you have copied, favourites, and animated GIFs found in your Downloads and Pictures folders • Emoji — searchable by name and keyword, with pins, recents and a skin tone setting Choosing an entry pastes it straight into the window underneath. Requires Wayland.

Extension Homepage
https://github.com/stevebushwa/winclip

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) warning 2 manual_review 1

Shexli found 3 issues 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

  • clipboardMonitor.js:27
    St.Clipboard.get_default()

EGO-X-004 warning

extensions should avoid synchronous file IO in shell code

Shell code should avoid synchronous file IO APIs like `GLib.file_get_contents()` and `Gio.File.load_contents()`.

File Operations

  • imageUtil.js:227
    GLib.file_get_contents(path)
  • store.js:70
    GLib.file_get_contents(this._file)

EGO-L-001 warning

extension must not create GObject instances or modify shell before enable()

Resource creation or signal/source setup was found outside `enable()`.

Only use initialization for static resources

  • imageUtil.js:148
    ticker = new Ticker()
  • imageUtil.js:148
    new Ticker()

All Versions

Version Status
2 Unreviewed
1 Rejected

Previous Reviews on this Version

stevebushwa posted a review
this version i uploaded has a pretty bad shell hanging bug in the gif section, it tries to render all of them at once instead of just what is visible in the section visible. i have made improvements to it to fix that in the past few hours, which i will be uploading as a subsequent version
stevebushwa auto- rejected
Auto-rejected because of new version 2 was uploaded