Mailing-List: contact dev-help@flex.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@flex.apache.org
Received-SPF: unknown (nike.apache.org: error in processing during lookup of
 aharui@adobe.com)
From: Alex Harui <aharui@adobe.com>
To: "dev@flex.apache.org" <dev@flex.apache.org>, "commits@flex.apache.org"
	<commits@flex.apache.org>
Subject: Re: git commit: [flex-sdk] [refs/heads/develop] - FLEX-34078 fix
 callouts closing when selecting dates in DateFields
Thread-Topic: git commit: [flex-sdk] [refs/heads/develop] - FLEX-34078 fix
 callouts closing when selecting dates in DateFields
Thread-Index: AQHPctHBeyugxhZHqE6aCKonvlNt6ptTeW0A
Date: Tue, 27 May 2014 05:12:11 +0000
Message-ID: <CFA96B12.22B23%aharui@adobe.com>
References: <03678cf42d824e3b86ac53e2c724a57f@git.apache.org>
In-Reply-To: <03678cf42d824e3b86ac53e2c724a57f@git.apache.org>
Accept-Language: en-US
Content-Language: en-US
user-agent: Microsoft-MacOutlook/14.4.1.140326
received-spf: None (: adobe.com does not designate permitted sender hosts)
Content-Type: text/plain; charset="us-ascii"
Content-ID: <7354DBC815344C489B1E0E15016EA362@namprd02.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0

This change broke Falcon, although Falcon is not reporting the failure
correctly.   And I'm sort of surprised it didn't break the SDK build
because it introduces a dependency from spark.swc to mx.swc and mx.swc is
not listed in the set of libraries used to build experimental.swc.

It is also not a good idea to start dragging mx stuff into spark-only
apps.  Might affect mobile app sizes and might affect those with
spark-only build setups.

I'm not familiar with Callout, but in other components with drop downs,
the drop down's owner property is set to the base component, then you
check to see if the component "owns" the clicked item.  Can similar logic
be used here and eliminate the mx.swc dependency?

Thanks,
-Alex

On 5/18/14 12:30 PM, "jmclean@apache.org" <jmclean@apache.org> wrote:

>Repository: flex-sdk
>Updated Branches:
>  refs/heads/develop 3a087a60e -> 080f5e64d
>
>
>FLEX-34078 fix callouts closing when selecting dates in DateFields
>
>
>Project: http://git-wip-us.apache.org/repos/asf/flex-sdk/repo
>Commit: http://git-wip-us.apache.org/repos/asf/flex-sdk/commit/080f5e64
>Tree: http://git-wip-us.apache.org/repos/asf/flex-sdk/tree/080f5e64
>Diff: http://git-wip-us.apache.org/repos/asf/flex-sdk/diff/080f5e64
>
>Branch: refs/heads/develop
>Commit: 080f5e64d5718d76c51ecf6b07552b8bfc1b112e
>Parents: 3a087a6
>Author: Justin Mclean <jmclean@apache.org>
>Authored: Sun May 18 12:30:41 2014 -0700
>Committer: Justin Mclean <jmclean@apache.org>
>Committed: Sun May 18 12:30:41 2014 -0700
>
>----------------------------------------------------------------------
> .../src/spark/components/supportClasses/DropDownController.as     | 3 +++
> 1 file changed, 3 insertions(+)
>----------------------------------------------------------------------
>
>
>http://git-wip-us.apache.org/repos/asf/flex-sdk/blob/080f5e64/frameworks/p
>rojects/spark/src/spark/components/supportClasses/DropDownController.as
>----------------------------------------------------------------------
>diff --git=20
>a/frameworks/projects/spark/src/spark/components/supportClasses/DropDownCo
>ntroller.as=20
>b/frameworks/projects/spark/src/spark/components/supportClasses/DropDownCo
>ntroller.as
>index e1ff2fd..3641111 100644
>---=20
>a/frameworks/projects/spark/src/spark/components/supportClasses/DropDownCo
>ntroller.as
>+++=20
>b/frameworks/projects/spark/src/spark/components/supportClasses/DropDownCo
>ntroller.as
>@@ -38,6 +38,7 @@ import mx.managers.ISystemManager;
>=20
> import spark.components.DropDownList;
> import spark.events.DropDownEvent;
>+import mx.controls.CalendarLayout;
>=20
> use namespace mx_internal;
>=20
>@@ -613,6 +614,8 @@ public class DropDownController extends
>EventDispatcher
>                 var document:Object =3D (target as IUIComponent).document=
;
>                 if (("hostComponent" in document) &&
>document.hostComponent is DropDownList)
> 			        return;
>+				 if (target.parent is CalendarLayout)
>+			        return;
>             }
>=20
>             if (hitAreaAdditions !=3D null)
>