[wp-trac] [WordPress Trac] #19393: Image crop position

WordPress Trac wp-trac at lists.automattic.com
Sun Feb 19 15:00:32 UTC 2012


#19393: Image crop position
-------------------------+------------------
 Reporter:  bradt        |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  3.4
Component:  Media        |     Version:  3.3
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |
-------------------------+------------------

Comment (by bradt):

 Replying to [comment:13 azaozz]:
 > Replying to [comment:11 dwenaus]:
 > > there are many times when you have pictures of people, and cropping to
 the middle chops off their head. uncool. :) is this going to make it into
 3.4?
 >
 > Thinking this would need more general solution. Currently users can crop
 the images (or just the thumbs) any way they like by using the image
 editing tab in the image settings.

 Are you saying that every time a user uploads a headshot, screenshot, or
 any other image that requires a different crop position, we should expect
 the user to go into the cropper tool and crop the image and/or its
 thumbnails? I don't see any reason why we shouldn't be able to set a
 different default crop position for each image size and save the user this
 hassle. I can tell you first hand that when the user is a client, this is
 a tall order.

 > Similarly we have two sets of image manipulation functions: one
 "general" and another specific for the image editor. Imho it's worth
 merging them and making a nicer API for image manipulation in core.

 Agreed, but I believe it's outside the scope of this patch. This patch is
 about extending the $crop variable of the the add_image_size API function
 to solve a common problem where the [http://www.google.ca/webhp?sourceid
 =chrome-instant&ix=seb&ie=UTF-8&ion=1#hl=en&output=search&sclient=psy-
 ab&q=wordpress%20crop%20image%20top&pbx=1&oq=&aq=&aqi=&aql=&gs_sm=&gs_upl=&fp=3fb20b071e948d11&ix=seb&ion=1&ix=seb&ion=1&bav=on.2,or.r_gc.r_pw.r_cp.r_qf.,cf.osb&fp=3fb20b071e948d11&biw=1439&bih=963&ix=seb&ion=1
 top solutions found on Google] has long been to *gulp* ...hack the core.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/19393#comment:15>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list