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

Resize and crop with cropping options #72

Closed
pulse00 opened this issue May 19, 2012 · 6 comments
Closed

Resize and crop with cropping options #72

pulse00 opened this issue May 19, 2012 · 6 comments
Labels
Level: New Feature 🆕 This item involves the introduction of new functionality.

Comments

@pulse00
Copy link

pulse00 commented May 19, 2012

I've the following requirement in our app for image resizing and cropping:

  1. If an image exceeeds a given dimension, resize it's smallest side to this dimension
  2. Crop the resulting image, but with the option to specify the side on the image where to crop

So for example, when the maximum dimension is a rectangle with 250x250 (height/width) and the input image is
300x600, resize it to 250x498, and make it optional where to crop the resized image:

  1. crop the left side
  2. crop in the "middle"
  3. crop on the right

Is this possible with the current available filters?

If not, i'd be happy to send you a PR with this feature.

@lsmith77
Copy link
Contributor

i don't think its possible yet ..i assume you want to handle the choosing of the crop area via a GET parameter?

@havvg
Copy link
Contributor

havvg commented Feb 8, 2013

You can achieve this by chaining the crop filter and the resize filter according to your specs.
You can create a new filter, which encapsulates this behavior and passes the "global" configuration to the respective target filters.

Please send a PR, if you got something to share on this!
A generic approach on a ChainFilterLoader would be great.

@makasim
Copy link
Collaborator

makasim commented Dec 15, 2013

I suppose it has to be covered by #72

@havvg
Copy link
Contributor

havvg commented Dec 15, 2013

?

@makasim
Copy link
Collaborator

makasim commented Dec 16, 2013

Ive reread the issue, It seems I misunderstood it. sorry for bothering.

@makasim makasim modified the milestones: v1.1, v1.0.0 May 22, 2014
@robfrawley robfrawley removed this from the v1.1-missed milestone Jan 25, 2017
@michellesanver
Copy link
Contributor

This is an issue from 2012, I thus assume it is no longer relevant. If it is, please open a new issue with details and we will look into that. Thanks a lot :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Level: New Feature 🆕 This item involves the introduction of new functionality.
Projects
None yet
Development

No branches or pull requests

6 participants