• Global community
    • Language:
      • Deutsch
      • English
      • Español
      • Français
      • Português
  • 日本語コミュニティ
    Dedicated community for Japanese speakers
  • 한국 커뮤니티
    Dedicated community for Korean speakers
Exit
Locked
0

[svn:fx-4.x] 14567: Fix for de-mirroring icons correctly including http:// bugs.adobe.com/jira/browse/SDK-25657 and http://bugs.adobe.com/jira/browse/ SDK-25684.

Adobe Employee ,
Mar 03, 2010 Mar 03, 2010

Copy link to clipboard

Copied

Revision: 14567

Revision: 14567

Author: klin@adobe.com

Date: 2010-03-03 18:08:04 -0800 (Wed, 03 Mar 2010)

Log Message:

***********

Fix for de-mirroring icons correctly including http://bugs.adobe.com/jira/browse/SDK-25657 and http://bugs.adobe.com/jira/browse/SDK-25684.

In general, the fix for de-mirroring icons involves saving the original transform matrix before applying our scaling and translation. Our current method only partially does this resulting in bugs such as the vertical flipping. After saving the matrix, we can restore the original transform matrix when applying most properties that would affect our calculations (width, height, scaleX, scaleY, x). We also use backing variables for scaleX and scaleY since they may be affected by the matrix calculations and by setting width and height. I?\226?\128?\153ve also tried to simplify the case where we are switching back to not mirroring by restoring the original matrix and setting properties normally.

-Modified validateTransformMatrix to only handle the mirroring case. It saves the original matrix to a private variable, origMatrix, and applies matrix transformations on the current transform matrix. The transformations are similar to the pattern in AdvancedLayoutFeatures.build2DMatrix. We must use matrix transformations in order to make sure our transformations work with other transformations (e.g. rotation).

-In the addedHandler, I split the logic for mirroring and not mirroring. For the mirroring case, we also set the backing variables in addition to calling validateTransformMatrix. For the normal case, we restore the transform matrix to origMatrix.

-In each of the property setters, we first restore the transform matrix to origMatrix before setting the property. After setting the property, we apply de-mirroring by calling validateTransformMatrix.

QE notes: No

Doc notes: No

Bugs: SDK-25657, SDK-25684

Reviewer: Jason

Tests run: checkintests, Mirroring, Button, Alert, Tree

Is noteworthy for integration: No

Ticket Links:

***********

http://bugs.adobe.com/jira/browse/SDK-25657

http://bugs.adobe.com/jira/browse/SDK-25684

http://bugs.adobe.com/jira/browse/SDK-25657

http://bugs.adobe.com/jira/browse/SDK-25684

Modified Paths:

**************

flex/sdk/branches/4.x/frameworks/projects/framework/src/mx/core/FlexBitmap.as

TOPICS
Commits

Views

485

Translate

Translate

Report

Report
Community guidelines
Be kind and respectful, give credit to the original source of content, and search for duplicates before posting. Learn more
community guidelines