Skip to content

Conversation

@jackmcdade
Copy link
Member

@jackmcdade jackmcdade commented Feb 10, 2026

Uses cropper.js to add a new image crop feature, complete with aspect ratio and orientation flipping.

CleanShot 2026-02-09 at 22 45 13@2x CleanShot 2026-02-09 at 22 46 06@2x

When finished cropping you can choose to save as a new image or replace the existing one.
CleanShot 2026-02-09 at 22 46 17@2x


Note

Medium Risk
Adds a new client-side crop + upload/overwrite flow for assets and changes ConfirmationModal cancel semantics, which could impact asset replacement behavior and other consumers relying on cancel events.

Overview
Adds an image cropping feature in the asset editor using cropperjs, including a new full-screen CropEditor with aspect-ratio presets, orientation flipping, and keyboard shortcuts.

Wires cropping into Editor.vue via a new Crop toolbar action and a confirmation step that uploads the cropped file either as a new copy (timestamped) or by overwriting the original, including cache-busting reload behavior when replacing.

Updates ConfirmationModal to distinguish explicit cancel-button clicks via a new cancel-clicked event (while still emitting cancel on any close), and adds a new crop icon plus the cropperjs dependency (and lockfile entries).

Written by Cursor Bugbot for commit 239753f. This will update automatically on new commits. Configure here.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@jackmcdade jackmcdade marked this pull request as ready for review February 10, 2026 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants