Conversation
There was a problem hiding this comment.
Pull request overview
This pull request adds two new Cinnamon desktop applets, though the PR title and description only mention the Screen Snipper applet. The Screen Snipper provides screenshot functionality using gnome-screenshot, while the Resource Monitor displays system resource usage (CPU, RAM, and Swap) in the panel.
Changes:
- Added Screen Snipper applet with area capture and delayed capture features using gnome-screenshot backend
- Added Resource Monitor applet that displays real-time CPU, RAM, and Swap usage by reading from /proc filesystem
Reviewed changes
Copilot reviewed 10 out of 11 changed files in this pull request and generated 13 comments.
Show a summary per file
| File | Description |
|---|---|
| screen-snipper@kprakesh/applet.js | Main ES6 class-based applet implementation for screenshot functionality |
| screen-snipper@kprakesh/metadata.json | Metadata defining applet name, version, and Cinnamon compatibility |
| screen-snipper@kprakesh/settings-schema.json | Settings schema for configurable delay timer |
| screen-snipper@kprakesh/README.md | Documentation with features, dependencies, and usage instructions |
| screen-snipper@kprakesh/LICENSE | GPL-3.0 license file |
| resource-monitor@kprakesh/applet.js | Prototype-based applet that monitors system resources via /proc filesystem |
| resource-monitor@kprakesh/metadata.json | Metadata for resource monitor applet |
| resource-monitor@kprakesh/settings-schema.json | Settings schema for toggling CPU, RAM, and Swap display |
| resource-monitor@kprakesh/README.md | Documentation for resource monitor with technical details |
| resource-monitor@kprakesh/LICENSE | GPL-3.0 license file |
| resource-monitor@kprakesh/utilities-system-monitor.png | PNG icon file (purpose unclear given symbolic icon usage) |
|
Key Changes:
|
Best-practices scannerThis is a regex-based check for API usage that can pose security, performance or This check is not perfect will not replace a normal review.Found 2 potential issue(s): ℹ️ shell_string_spawnscreen-recorder@kprakesh/files/screen-recorder@kprakesh/applet.js:101 Prefer argument vector spawn functions over shell command strings. screen-recorder@kprakesh/files/screen-recorder@kprakesh/applet.js:116 Prefer argument vector spawn functions over shell command strings. Automated pattern check. |
Best-practices scannerThis is a regex-based check for API usage that can pose security, performance or This check is not perfect will not replace a normal review.Found 2 potential issue(s): ℹ️ shell_string_spawnscreen-recorder@kprakesh/files/screen-recorder@kprakesh/applet.js:101 Prefer argument vector spawn functions over shell command strings. screen-recorder@kprakesh/files/screen-recorder@kprakesh/applet.js:116 Prefer argument vector spawn functions over shell command strings. Automated pattern check. |
This PR adds a new applet called Screen Snipper. It is a lightweight panel tool that allows users to quickly capture screenshots using the native gnome-screenshot backend.
Features: