Image.rotate() for 90 degrees: incorrect image's size(width,height)

[Pai Peng]

Hello CN1,

by using Image.rotate() function I get problem by reading the rotated image's size.

Image temp = image.rotate(90);

Log.p("image size: " + temp.getWidth() + "-" + temp.getHeight(), Log.DEBUG);

I have read an image file to Image object, which has size: 1800-1198.

Then I am using  the code above to rotate this Image object, and what I am waiting for the result of image size is 1198-1800.

But I get the log:

[EDT] 0:0:16,41 - image size: 1800-1198

After rotate the Image object, the image size doesn't be updated.

Is it a bug, or what I have done wrong?

Tested on simulator.

Regards,

Pai

[Shai Almog]

No, the method assumes a square image.
Rotating to a square degree is a special case where this makes sense but if we rotated to 45 degrees then what?

[Pai Peng]

It depends on what the Image could be after rotated by 45 degrees.

For example, if the Image needs to full display, no crop, then it's size is increased.

Regards,

Pai

[Pai Peng]

Hello Shai,

actually what my problem is with captured images on Android device. When they are taken in portrait format, but always displayed in landscape. It is really not acceptable for my app. What I have to do here is adding an new Form for the user to rotate the picture to 90 degrees.

With iOS and Android SDK, this kind of rotation to 90, 180, 270 degrees is also available and after rotation, the new image size is adjusted automatically. This is what I expect also from CN1.

Regards,

Pai

On Wednesday, August 7, 2013 5:47:25 PM UTC+2, Shai Almog wrote:

[Shai Almog]

That is a different use case from the rotate feature and not one that is currently supported out of the box. You could theoretically use getRGB() and shift the pixels into a new image structure but this is obviously less optimal. You can file an RFE for a feature like this but right now we are completely swamped with pro/enterprise requests and I'm just not sure we can handle non-critical issues.

[Pai Peng]

Hello Shai,

I Do Thank you very much!

I know you are working hard on the CN1 and it is still growing.

Regards,

Pai

By the way I am also a PRO user, maybe I should create  a feature request via your support line.

[Shai Almog]

I'm sorry its hard to keep track of all the new subscribers and associate everyone. Just write to the support email and followup with me (its in the subscription tab for pro accounts) I'll make sure to promote the issue.

[Shai Almog]

I was just looking at this issue again, did you use:

Display.getInstance().setProperty("normalizeImage", "true");

This is related to this issue: http://code.google.com/p/codenameone/issues/detail?id=801

I didn't turn it on by default because I wasn't sure the fix worked as expected.

[Pai Peng]

Hi Shai,

great work!

Thank you for so quick support!

Regards,

Pai

[Shai Almog]

Thanks!
I enabled it by default, should be on in the next update.