+1
On Fri, Dec 20, 2013 at 8:38 AM, Andrew Grieve <agri...@chromium.org> wrote: > Just to recap (correct if I'm wrong) - > 1. targetWidth & targetHeight are not being interpreted consistently across > platforms. > 2. What they should do is: > - Never grow the image > - Shrink the image while maintaining aspect ratio, such that both > dimension of the resized image fit within targetWidth & targetHeight. > > Certainly sounds good to me! > > > On Fri, Dec 20, 2013 at 10:49 AM, Wargo, John <john.wa...@sap.com> wrote: > > > Well I've not really received a consensus on anything. One or two people > > have spoken up, but not much beyond that. > > > > John M. Wargo > > SCN Blog: http://scn.sap.com/blogs/johnwargo > > Twitter: @johnwargo > > > > > > -----Original Message----- > > From: Shazron [mailto:shaz...@gmail.com] > > Sent: Tuesday, December 17, 2013 1:15 AM > > To: dev@cordova.apache.org > > Subject: Re: Camera targetWidth & targetHeight > > > > Thanks John, > > Filing the issue on iOS would be great, summarizing what needs to be done > > (and the consensus). > > > > > > On Mon, Dec 16, 2013 at 5:30 PM, John M. Wargo <jwarg...@gmail.com> > wrote: > > > > > So, does anyone have an opinion on how this discrepancy should be > > > addressed? The methods work differently on different platforms and I > > think > > > they should work the same. > > > > > > To me Android works as I expect it should - setting the properties > along > > > the right axis. Do I need to file JIRA tickets about bringing iOS into > > > alignment? > > > > > > Also, what about when only one parameter is provided? Should we decide > > how > > > this is supposed to work and implement it across the different > platforms? > > > I'm for supporting only one parameter, allows me to set the one side > and > > > let the camera aspect ratio determine the other. > > > > > > > > > -------- Original Message -------- > > > Subject: Camera targetWidth & targetHeight > > > Date: Mon, 02 Dec 2013 22:15:06 -0500 > > > From: John M. Wargo <jwarg...@gmail.com> > > > To: Cordova Dev <dev@cordova.apache.org> > > > > > > > > > > > > A while back I posted a question regarding Camera targetWidth & > > > targetHeight properties and how they worked. After some discussion, the > > > conclusion I reached was that the documentation couldn't be correct > about > > > how it worked since there was no way to determine the camera's > resolution > > > with the current API but the docs said I had to provide both > parameters. > > I > > > said I'd do some testing and I have finally gotten around to completing > > it. > > > Here's what I discovered: > > > > > > I created an application that allowed me to pass in different values > for > > > targetWidth & targetHeight when taking a picture. I tested at the > > following > > > image sizes: 640x480, 800x600, 1024x768 as well as setting only the > > > targetWidth to 1024 or only the targetHeight to 768. > > > > > > Here's the results: > > > > > > Android > > > Portrait Landscape > > > 480x640 640x480 > > > 600x800 800x600 > > > 768x1024 1024x768 > > > 768x1024 1024x768 > > > 768x1024 1024x768 > > > > > > > > > > > > iOS > > > Portrait Landscape > > > 360x480 640x480 > > > 450x600 800x600 > > > 576x768 1024x768 > > > 2448x3264 3264x2448 > > > 2448x3264 3264x2448 > > > > > > > > > > > > Windows Phone 8 > > > Portrait Landscape > > > 1836x3264 3264x1836 > > > 1836x3264 3264x1836 > > > 1836x3264 3264x1836 > > > 1836x3264 3264x1836 > > > 1836x3264 3264x1836 > > > > > > > > > As you can see, Android properly implements the targetWidth & > > targetHeight > > > properties. On iOS, it supports setting both properties, but not > > instances > > > where only one is specified. Windows Phone 8 ignores the parameters > > > completely. On iOS, when you turn the device on its side, the Camera > API > > > applies the target width or height to the wrong axis (Android does this > > > well however). > > > > > > I'm trying to test this on a BlackBerry device, but my development > > > environment is giving me fits right now. I'll work on it in the morning > > and > > > publish my results when I get them. > > > > > > I would suggest that the android implementation is as expected and that > > > the other platforms need their implementations of targetWidth & > > > targetHeight adjusted so it works correctly. The documentation should > be > > > updated as well as it's incorrect today specifying that both properties > > > must be provided. > > > > > > If the group doesn't want to support only providing one of the > > properties, > > > then I would expect that the onError callback is called when only one > is > > > provided rather than simply ignoring them as is the case with iOS and > > > Windows Phone. > > > > > > I posted my sample application and a spreadsheet with my results to > > > https://github.com/johnwargo/camera_res_test > > > > > > -- > > > John M. Wargo > > > @johnwargo <http://twitter.com/johnwargo> > > > www.johnwargo.com <http://www.johnwargo.com> > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------ > > > > > > -- > > > John M. Wargo > > > @johnwargo <http://twitter.com/johnwargo> > > > www.johnwargo.com <http://www.johnwargo.com> > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------------------------------------ > > > ------------------------------ > > > > > > > > > > > >