Skip to content
This repository was archived by the owner on Mar 13, 2018. It is now read-only.

Action should be performed only on desired target element#39

Open
redcom wants to merge 1 commit intogooglearchive:masterfrom
redcom:master
Open

Action should be performed only on desired target element#39
redcom wants to merge 1 commit intogooglearchive:masterfrom
redcom:master

Conversation

@redcom
Copy link

@redcom redcom commented Mar 31, 2015

tapAction event is attached to the parent element on which we would like to perform action against. This is exactly how it should be done.
But the delegated event should target individual element we want to alter.

If clicking in "section" parent element, without clicking on the element "paper-shadow" will cause an error.

@googlebot
Copy link

Thanks for your pull request.

It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA) at https://cla.developers.google.com/.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check the information on your CLA or see this help article on setting the email on your git commits.

Once you've done that, please reply here to let us know. If you signed the CLA as a corporation, please let us know the company's name.

@redcom
Copy link
Author

redcom commented Apr 1, 2015

Hello,
I have signed the CLA as individual.

@googlebot
Copy link

We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm.

@redcom
Copy link
Author

redcom commented Apr 2, 2015

I did the changes

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants