Mailing-List: contact callback-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: callback-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of simon.macdonald@gmail.com
 designates 209.85.160.47 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAOBL_k7iZhDPfBnk_suHLKR8_J1ciEs1CGV4sddGhvu0+o0BKA@mail.gmail.com>
References: 
 <CAM1AYQBMNdf9Usv9CD+sJFHL3YsH=YqUNaRD8ON-Hkb=wDaUpA@mail.gmail.com>
 <CAOBL_k6N0QyJOFRTn58bYrPxfCnS4Nj1X5WM0nNuV6hDHfqsqg@mail.gmail.com>
 <CANpVHHwdC4kB465fMopKA=uJ=CDcFRM5uLnkY+AWTujTLfKnOw@mail.gmail.com>
 <CAM1AYQD=maZvxHWpOQwEJ2qfJUopadmZr9yL-7FMiJwL_1yizg@mail.gmail.com>
 <CANpVHHzF3bYijEvNCBmYtiMchAg_i5z0__kwg4Sb8emas1RtAQ@mail.gmail.com>
 <CAM1AYQAT0qeu8wtn_KUPodOL28f3+FWHSJvjqDdZvWckkjuqTw@mail.gmail.com>
 <CAM1AYQBb2igFzBifbK26mgFeAPeEqLBwJpsOg7xR-pD2=xr9xg@mail.gmail.com>
 <CAOBL_k7iZhDPfBnk_suHLKR8_J1ciEs1CGV4sddGhvu0+o0BKA@mail.gmail.com>
From: Simon MacDonald <simon.macdonald@gmail.com>
Date: Tue, 25 Sep 2012 14:24:33 -0400
Message-ID: 
 <CAM1AYQC-14EiMCCf-L78q+C4mnUuvtCUtY=O1KMRy3pXc0HhxQ@mail.gmail.com>
Subject: Re: Own. The. Camera.
To: callback-dev@incubator.apache.org
Content-Type: text/plain; charset=ISO-8859-1

Cool Joe. Thanks for the heads up on the UI. I kinda think we will
need a preview as that is what folks expect.

Simon Mac Donald
http://hi.im/simonmacdonald


On Tue, Sep 25, 2012 at 2:19 PM, Joe Bowser <bowserj@gmail.com> wrote:
> Hey
>
> I'm currently hacking on the UI. This really reminds me of why this
> project exists in the first place.  I'm hoping to get a basic layout
> working by the end of the day for phones.
>
> We also need to implement the following:
>
>  * AutoFocus
>  * Zoom
>  * Preview
>
> I'm willing to not have preview, since this could be implemented in
> HTML and Javascript by the developer, even though every camera app
> that I've seen did have preview built-in so far.
>
> Joe
>
> On Tue, Sep 25, 2012 at 11:13 AM, Simon MacDonald
> <simon.macdonald@gmail.com> wrote:
>> Just an update as I continue to work on this. I have the
>> CameraActivity working with the following options:
>>
>> quality : any quality
>> destinationType : DATA_URL or FILE_URI,
>> sourceType : CAMERA or PHOTOALBUM
>> allowEdit : N/A on Android
>> encodingType: JPEG only
>> targetWidth: any width
>> targetHeight: any height
>> mediaType: any media type
>> popoverOptions: N/A on Android
>> saveToPhotoAlbum: true or false
>> correctOrientation: completely broken
>>
>> Things that need to be done include:
>>
>> 1) Figure out why the Camera class does not save any of the EXIF data
>> with orientation being the most important so we can do
>> correctOrientation properly.
>> 2) Move a lot of the code in CameraLauncher into CameraActivity. That
>> way we won't need to write a File in CameraActivity then read it in
>> CameraLauncher. By working with the raw bytes in CameraActivity to do
>> rotation/resizing/compression we should be able to save on memory.
>> 3) Make the Camera GUI look so much nicer than my current utilitarian mode.
>> 4) Move away from a camera layout file to putting the layout in code
>> so we don't have to make Cordova a library project.
>>
>> Simon Mac Donald
>> http://hi.im/simonmacdonald
>>
>>
>> On Fri, Sep 21, 2012 at 8:50 PM, Simon MacDonald
>> <simon.macdonald@gmail.com> wrote:
>>> That should already be fixed by this commit:
>>>
>>> https://git-wip-us.apache.org/repos/asf?p=incubator-cordova-android.git;a=commit;h=f78af9f27baa82a351fb4b8a317ad50d54d99e56
>>>
>>> Simon Mac Donald
>>> http://hi.im/simonmacdonald
>>>
>>>
>>> On Fri, Sep 21, 2012 at 7:11 PM, Anis KADRI <anis.kadri@gmail.com> wrote:
>>>>
>>>> Indeed.
>>>>
>>>> I got this with the latest target http://pastebin.com/wj11TGhh
>>>>
>>>> and the errors above with an old target and I then I recalled
>>>> that renderscript.opt.level thingy. Anyway we can fix this for good ? I
>>>> know it has nothing to do with the topic! :-)
>>>>
>>>> On Fri, Sep 21, 2012 at 3:59 PM, Simon MacDonald
>>>> <simon.macdonald@gmail.com>wrote:
>>>>
>>>> > Looks like you are using an older version of Android before those Exif
>>>> > properties existed.
>>>> >
>>>> > Simon Mac Donald
>>>> > http://hi.im/simonmacdonald
>>>> >
>>>> >
>>>> > On Fri, Sep 21, 2012 at 6:56 PM, Anis KADRI <anis.kadri@gmail.com>
>>>> > wrote:
>>>> >
>>>> > > Tried it out. Not compiling. http://pastebin.com/3KK96gCu
>>>> > > Didn't spend too much time on it. So solution can be trivial :)
>>>> > >
>>>> > > On Fri, Sep 21, 2012 at 2:04 PM, Joe Bowser <bowserj@gmail.com> wrote:
>>>> > >
>>>> > > > That sounds like a good idea.  Do we want auto-focus on or off on
>>>> > > > the
>>>> > > > camera?
>>>> > > >
>>>> > > > Joe
>>>> > > >
>>>> > > > On Fri, Sep 21, 2012 at 12:43 PM, Simon MacDonald
>>>> > > > <simon.macdonald@gmail.com> wrote:
>>>> > > > > Okay,
>>>> > > > >
>>>> > > > > So I've done a bit of work today on taking the camera code back in
>>>> > > house.
>>>> > > > > Right now I have the very basic take a picture and return a file
>>>> > > > > URI
>>>> > > > > working. It does not respect and options or do any image
>>>> > manipulation.
>>>> > > > I'm
>>>> > > > > going to take the time to add that back in slowly.
>>>> > > > >
>>>> > > > >
>>>> > > >
>>>> > >
>>>> >
>>>> > https://github.com/macdonst/cordova-android/commit/ac5dbf3f54ac6e43f6c346130ac2ac86aed26eb5
>>>> > > > >
>>>> > > > > Also, I'm using a layout file which means you have to import
>>>> > > > > Cordova
>>>> > > into
>>>> > > > > your test app as a library project. I'm just doing that for now as
>>>> > > > > eventually that layout will get converted to code so we can go
>>>> > > > > back
>>>> > to
>>>> > > > > making it a jar. I am not a UI designer so Joe volunteered Yohei
>>>> > > > > to
>>>> > > > design
>>>> > > > > the camera interface so it doesn't look like a deranged
>>>> > > > > coffee fuelled monster designed it. Thanks Yohei!
>>>> > > > >
>>>> > > > > Simon Mac Donald
>>>> > > > > http://hi.im/simonmacdonald
>>>> > > >
>>>> > >
>>>> >
>>>
>>>