Would like some discussion on this please? I think we have to add it based on app store rejection. Could be wrapped up for 1.8 but since it affects cordova-js will need testing on all platforms.
-becky On Wed, May 23, 2012 at 6:15 PM, Shazron <[email protected]> wrote: > The last time I proposed adding a new option to Camera.getPicture it was > shot down (turns out in the end it wasn't needed since we should default to > one option anyway), so I'm interested in what others think of this, since > it is needed (App Store rejection reasons). > > What is the status of this post 2.0 anyway? I don't know if we have any > alternative. > > On Wed, May 23, 2012 at 2:55 PM, Becky Gibson <[email protected] > >wrote: > > > I'd like to fix this open ticket about the placement of the popover used > to > > pick images from an album on ipad: > > https://issues.apache.org/jira/browse/CB-56 > > > > Background: When picking images from the album on an iPad a > > popoverController is required. We currently place that popover at a > fixed > > location on the screen and let iOS place the arrow appropriately. This > has > > caused some apps to get rejected from the appStore due to poor placement > of > > the popover and the arrow (see the ticket for an example). In order to > > fix this we need to allow the developer to specify the placement of the > > popover and the direction of the arrow. Thus I am proposing yet another > > option to camera.getPicture. I don't like adding more options but it > seems > > necessary for proper iPad development. > > > > The popover options object would be added to the current camera options. > > In the getPicture(win,fail, options) JavaScript implementation each of > the > > values in the options parameter are passed to the exec function via the > > parameters array. The iOS exec function is still set up to pass any > > objects found in the parameter array as a dictionary object. So adding > > the popoverOptions object requires minimal changes to the getPicture() > > implementation: > > > > *var popoverOptions = null;* > > * if (typeof options.popoverOptions == "object") {* > > * popoverOptions = options.popoverOptions;* > > * }* > > > > exec(successCallback, errorCallback, "Camera", "takePicture", > [quality, > > destinationType, sourceType, targetWidth, targetHeight, encodingType, > > mediaType, allowEdit, correctOrientation, saveToPhotoAlbum, > > *popoverOptions* > > ]); > > > > Here is a quick proposal for the popoveroptions: > > > > var PopoverOptions = function(x,y,width,height, direction) { > > this.x = (x !== undefined ? x : 0); > > this.y = (y!== undefined ? y : 32); > > this.width = (width !== undefined ? width : 320); > > this.height = (height !== undefined ? height : 480); > > this.arrowDir = (direction !== undefined ? direction : > > PopoverOptions.DIRECTION_ANY); > > } > > > > PopoverOptions.ARROW_DIRECTION_UP= 1; > > PopoverOptions.ARROW_DIRECTION_DOWN=2; > > PopoverOptions.ARROW_DIRECTION_LEFT=4; > > PopoverOptions.ARROW_DIRECTION_RIGHT=8; > > PopoverOptions.ARROW_DIRECTION_ANY=15; > > > > Any objections to adding this? > > > > thanks, > > -becky > > >
