Mailing-List: contact dev-help@flex.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@flex.apache.org
From: Peter Ent <pent@adobe.com>
To: "dev@flex.apache.org" <dev@flex.apache.org>
Subject: Re: [FlexJS] Click Event (Att: Alex and Peter) (was: Re: [FlexJS]
 RadioButton click event)
Thread-Topic: [FlexJS] Click Event (Att: Alex and Peter) (was: Re: [FlexJS]
 RadioButton click event)
Thread-Index: AQHSe6HJrcklGbygQ0KMHVIv2+tv86FSnt2A
Date: Tue, 31 Jan 2017 19:05:25 +0000
Message-ID: <D4B64510.205FE%pent@adobe.com>
References: <CAMFKFRjq6uz3d3=NLFXHOi-iovcFAcC1xCmPdTNzb6CO+WU-0g@mail.gmail.com>
In-Reply-To: <CAMFKFRjq6uz3d3=NLFXHOi-iovcFAcC1xCmPdTNzb6CO+WU-0g@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
user-agent: Microsoft-MacOutlook/14.7.1.161129
x-ms-exchange-messagesentrepresentingtype: 1
x-originating-ip: [24.98.132.247]
x-ms-office365-filtering-correlation-id: f458aa62-6437-45f0-0584-08d44a0c1d1d
x-ms-office365-filtering-ht: Tenant
x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001)(48565401078);SRVR:MWHPR02MB2816;
x-microsoft-exchange-diagnostics: 1;MWHPR02MB2816;7:gkRl0SNqzqD+HbQf15m2xC72iYgJT+EOR3/IeUw31ZPpJ73bMni7N3azyVR6IU8mrnca5sxT4O8sQoU8PIVjENZd4JT14f9I7e0SpWSZcsjkXMck1Bi3mtQsEWrni0G3D4ggq5fJsDc44inkObAMgJm18GpJbaK+uHgG/rpyiNu/csYmQF33PoJE8+DoD54tBihEDizVPYWn6EOvJ+0O6gcCSnObmAbcOdtMQLXKTNh653LZsNbJud4xQa16ZIE5fHszruIWyaNklKNFen8cWiSorn3NMvgRjMDvlIbYIJJobadTVf+IFeoHrUTLoatU0SzuWi1DlXNl/y63L7HGSl2UrxNd+FEyAN+LxrpDPwUYiGZbfzNH00iUwAGdPrOHXRrY6vT9L8P1f4dRWJnsLq/QhD4oMWdyq1rDvGbgKvJXrW4ye0djg4WT1p8pM6ctbiJ61Lx/qMvYfUmHH10a+ZeOng7X5zzo7y/7kn4TnKw0BdioINEnTgohq0YnWaQYMMVGCN8yj7r3WBIDS094GA==;20:sM6pN0S3pnYZ1lU8a3LctlymOQSIhUmepgG5x5mBQ4FkJ8tFmlHCpQT7uk624nH++1xGfJF9hUBy2Z++nuzYyVuicRCZNQrM4qN2oJ4TBOn5xOCOSFbqPD5xcwJPTlV43cLcsAo7rfCTb4ZaYoAr8WEQgIy1OIbBSqDwsZPgXa0=
x-microsoft-antispam-prvs: <MWHPR02MB2816CD3043A042077DB677D6D74A0@MWHPR02MB2816.namprd02.prod.outlook.com>
x-exchange-antispam-report-test: UriScan:(185789414550570)(17755550239193);
x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(61425038)(6040375)(601004)(2401047)(5005006)(8121501046)(10201501046)(3002001)(6055026)(61426038)(61427038)(6041248)(20161123560025)(20161123564025)(20161123562025)(20161123555025)(20161123558021)(6072148);SRVR:MWHPR02MB2816;BCL:0;PCL:0;RULEID:;SRVR:MWHPR02MB2816;
x-forefront-prvs: 0204F0BDE2
x-forefront-antispam-report: SFV:NSPM;SFS:(10009020)(6009001)(7916002)(39850400002)(39860400002)(39840400002)(39450400003)(39410400002)(199003)(377454003)(24454002)(189002)(377424004)(2501003)(8936002)(107886002)(66066001)(101416001)(97736004)(1730700003)(189998001)(450100001)(4001350100001)(8676002)(81156014)(305945005)(1720100001)(81166006)(7736002)(53936002)(6306002)(83506001)(10090500001)(6512007)(31430400001)(68736007)(76176999)(3280700002)(966004)(50986999)(54356999)(99286003)(3660700001)(5640700003)(38730400001)(2950100002)(2900100001)(229853002)(92566002)(6916009)(110136003)(6486002)(2906002)(15198665003)(6436002)(86362001)(6506006)(77096006)(25786008)(2351001)(6116002)(3846002)(106116001)(105586002)(106356001)(102836003)(5660300001)(36756003)(15395725005)(122556002);DIR:OUT;SFP:1101;SCL:1;SRVR:MWHPR02MB2816;H:MWHPR02MB2815.namprd02.prod.outlook.com;FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en;
received-spf: None (protection.outlook.com: adobe.com does not designate
 permitted sender hosts)
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="Windows-1252"
Content-ID: <60505424DF5B194CBF4602CBDF88A34A@namprd02.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: adobe.com
X-MS-Exchange-CrossTenant-originalarrivaltime: 31 Jan 2017 19:05:25.3075
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: fa7b1b5a-7b34-4387-94ae-d2c178decee1
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR02MB2816
archived-at: Tue, 31 Jan 2017 19:06:12 -0000

The MDL radio button is composed as:
<label =8A>
	<input =8A>
	<span =8A>Label You See</span>
	<span =8A>
</label>

The first click event has target=3D<span>, currentTarget=3D<label> while th=
e
second click event has target=3D<input>, currentTarget=3D<label>. The click=
 is
being dispatched from the input Radio (or Check) and from the label that's
identifying it.

In DataBindingExample, which uses the HTML.RadioButton, events come
through BrowserEvent which are then relayed to the proper event handler. I
spent some time tracing through this because it is, to me, obscure how
this happens. I'm not sure why MDL isn't automatically get BrowserEvent.

=8Bpeter

On 1/31/17, 4:09 AM, "carlos.rovira@gmail.com on behalf of Carlos Rovira"
<carlos.rovira@gmail.com on behalf of carlosrovira@apache.org> wrote:

>Alex, Peter,
>
>we're having some problems with MDL RadioButton click event.
>
>I implement RadioButton in mdl as a fork of the HTML one (they both share
>the same parent class and structure, so MDL does not extend RB HTML)
>
>As the rest of toggle components all has "change" and "click" events.
>Change use to work ok (no problem at all with that), but click use to give
>us problems. In Checkbox we had the problem that click was throwing two
>times so we make a workaround in the handler with preventDefault and
>manual
>selection/deselection.
>
>In RB Piotr could fix the issue deselecting all RBs in a group. so again
>we
>had to do manual. Maybe we could go with this implementation for now. but
>it motivated me to send this email.
>
>I think we don't understand the base in FlexJS and why is happening this
>issue with click event. For this reason we can't implement a solution that
>could be the best one, or one that does not look a bit "hacky"
>
>Hope you could give us some clue about what's happening here so we could
>understand and act accordingly
>
>
>Thanks!
>
>Carlos
>
>
>
>
>
>2017-01-31 0:28 GMT+01:00 piotrz <piotrzarzycki21@gmail.com>:
>
>> Hi Carlos,
>>
>> I just pushed fixed for double click event on RadioButton, but it's not
>> ideal. With this fix our "selected" property won't work.
>>
>> My solution:
>>
>> - Getting all Radio buttons in group and deselect all radio (I'm using
>>java
>> script function not "selected" property - that's why in unselection
>>process
>> our selected property is untouched) - Look into the "unselectAll"
>>function
>> -
>> maybe you will have some other idea.
>> - I'm selecting clicked element
>>
>>
>> Here is what I see without my fix. Our event listener is fire two times
>>and
>> source of our event is from
>>
>> <http://apache-flex-development.2333347.n4.nabble.
>> com/file/n58814/radio_click.png>
>>
>> goog.events.handleBrowseEvent function inside events.js - This function
>>is
>> fire two times in our ElementWrapper - fireListenerOverride. - Not sure
>> why.
>>
>> Piotr
>>
>>
>>
>>
>> -----
>> Apache Flex PMC
>> piotrzarzycki21@gmail.com
>> --
>> View this message in context: http://apache-flex-
>> development.2333347.n4.nabble.com/FlexJS-RadioButton-click-
>> event-tp58731p58814.html
>> Sent from the Apache Flex Development mailing list archive at
>>Nabble.com.
>>
>
>
>
>--=20
>Carlos Rovira
>http://about.me/carlosrovira