Review of "Screenshot Pro" version 1

Details Page Preview

Restores the classic 'drag and release' screenshot behavior. Automatically captures and resets selection area.

Extension Homepage
https://github.com/deepan-alve/screenshot-pro

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

  • extension.js:51
    St.Clipboard.get_default()
  • extension.js:172
    St.Clipboard.get_default()
  • extension.js:178
    St.Clipboard.get_default()
  • extension.js:340
    St.Clipboard.get_default()

All Versions

Version Status
1 Unreviewed

Previous Reviews on this Version

Deepanalve posted a review
All four St.Clipboard.get_default() calls are clipboard writes, never reads — the extension never reads the user's clipboard. Lines 172/178 are wl-copy subprocess fallbacks for copying OCR text or file paths. Line 51/329 wrap set_content to optionally suppress the default screenshot UI's auto-copy-to-clipboard behavior in 'save to file only' mode, and restore the original method on disable