Skip to content
This repository has been archived by the owner on Mar 26, 2019. It is now read-only.

Checkbox not working within a Pivot #92

Merged
merged 3 commits into from
Sep 22, 2016

Conversation

ericivarsson
Copy link
Contributor

@ericivarsson ericivarsson commented Aug 24, 2016

I ran into a problem where input type="checkbox" wasn´t working inside of a Pivot because of the onClick eventListener.
Not sure if this is the best way to solve the problem but it does seem to work fine for me.

I ran into a problem where input type="checkbox" wasn´t working inside of a Pivot because of the onClick eventListener.
Not sure if this is the best way to solve the problem but it does seems to work fine for me.
@msftclas
Copy link

Hi @ericivarsson, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@wdo3650
Copy link
Contributor

wdo3650 commented Sep 15, 2016

Approved. Thanks @ericivarsson. I was able to repro the issue and keeping the onClick listener contained to ms-Pivot-links fixes the issue.

Approved with PullApprove

@mikewheaton
Copy link
Contributor

mikewheaton commented Sep 15, 2016

Approved

Approved with PullApprove

@wdo3650
Copy link
Contributor

wdo3650 commented Sep 15, 2016

@ericivarsson this PR is unable to be merge because the base branch is out of date. Can you merge the latest changes from master into your fork?

@ericivarsson
Copy link
Contributor Author

@wdo3650 Sorry for the delay, I´ve updated the branch.

@mikewheaton mikewheaton added this to the 1.2.0 milestone Sep 22, 2016
@mikewheaton mikewheaton merged commit d803591 into OfficeDev:master Sep 22, 2016
@ericivarsson ericivarsson deleted the patch-1 branch September 23, 2016 06:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants