[Ssc-dev] latest obscurav1 branch commits

[Derek Halliday]

I agree that just making regions editable at all times is intuitive and
feels OK. It also makes it easier to move a region around, since your
finger isn't competing with the touch area of the region handles. Just
did some testing on the latest codebase and have a few pieces of feedback:

1. looks like there has been some instability introduced into gallery
selection piece of the app. bringing an image back into ObscuraCam's
Image Editor from the gallery results in a black screen of death (and FC
on user action, eventually). Logcat here: http://pastebin.com/Enk1LsNz

2. We should keep image region outlines for all regions present in an
image. In the latest version we've lost that functionality, and instead
it's only visible for the currently focused region. A change in color of
the border would accomplish this well - i.e. white borders for all
regions, yellow border for the currently focused one.

3. Definitely some bugs in pinch-to-change region size, and in general I
couldn't get it to work properly. Will be documenting this shortly and
sending out.

-D

On 10/18/11 5:10 AM, Nathan of Guardian wrote:
> 
> I just committed some fairly big refactoring of the UI code for
> Obscurav1. My goal was to both clean up most of the confusing code
> within ImageRegion about storing scaled and unscaled rect's, and improve
> all the transformation/matrix code in ImageEditor as well.
> 
>>From the user perspective, I also wanted to get rid of the box corner
> mechanism for resizing, and replace it with a more natural pinch
> mechanism. I also removed the need for explicitly selecting "Edit" in
> the menu... instead you can just start moving/resizing a region as soon
> as you touch it.
> 
> There are still some funky bugs to work out over a few more hours, but I
> think my approach is ultimately sound and more intuitive. Would love
> feedback from you, if you can build the code, or once I can post a binary.
> 
> Once we debug this, and add Harlos' "Identify" code handler, Obscurav1.2
> will be ready to be called a beta.
> 
> +n
> _______________________________________________
> Ssc-dev mailing list
> 
> Post: Ssc-dev at lists.mayfirst.org
> List info: https://lists.mayfirst.org/mailman/listinfo/ssc-dev
> 
> To Unsubscribe
>         Send email to:  Ssc-dev-unsubscribe at lists.mayfirst.org
>         Or visit: https://lists.mayfirst.org/mailman/options/ssc-dev/derek%40guardianproject.info
> 
> You are subscribed as: derek at guardianproject.info