Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

image_viewer: Add ability to zoom and pan images #24547

Closed

Conversation

kaf-lamed-beyt
Copy link
Contributor

@kaf-lamed-beyt kaf-lamed-beyt commented Feb 10, 2025

This PR introduces zoom and pan actions on images in the image viewer

zed-pan-zoom-controls-ezgif com-video-to-gif-converter

Closes #9584

Release Notes:

  • added zoom and pan controls in image viewer

This PR introduces a new capability to the image viewer crate — zoom and pan actions on images in Zed by tapping into a couple of mouse events exposed by GPUI
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 10, 2025
@kaf-lamed-beyt kaf-lamed-beyt changed the title image_viewer: zoom and pan images Image_viewer: zoom and pan images Feb 10, 2025
@maxdeviant maxdeviant changed the title Image_viewer: zoom and pan images image_viewer: Add ability to zoom and pan images Feb 10, 2025
@Angelk90
Copy link
Contributor

@kaf-lamed-beyt :
Look: #21683.

@kaf-lamed-beyt
Copy link
Contributor Author

Huh. Okay, I'll take a look at the recording and let you know if I have any questions.

@kaf-lamed-beyt
Copy link
Contributor Author

@Angelk90, from the video, most of the capabilities there will be included in Zed's image viewer when this is merged.

One thing that I am not certain of is the removal of the checkered background. In your video, it is called "remove chessboard". I know how well this sits with the Zed team. So I don't exactly have a say on that.

@maxdeviant
Copy link
Member

Thank you for the PR, but we're not moving forward with this feature at this time.

@maxdeviant maxdeviant closed this Mar 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Image Viewer: Pan & Zoom
3 participants