Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

send GA event when toggling AND send events on interaction #256

Merged
merged 1 commit into from
Jan 30, 2018

Conversation

ericawright
Copy link
Contributor

@ericawright ericawright commented Jan 29, 2018

This removes the iframe and builds the interactive example within our app, this way we can send click events interacting with the radio buttons.

type = 'click', event = "interaction-with-demo" and I'm also sending details, which is the nearest parent with an id - I'm sending it's id. for this example that is "select-unselect-example"

Fixes: #251.

** merge this OR the other, not both**

Copy link
Contributor

@bwinton bwinton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this more than the previous change, because it keeps all our code in one place, and makes sure it's properly reviewed!

@ericawright ericawright force-pushed the specific-events branch 3 times, most recently from 5509474 to 4d17ab2 Compare January 29, 2018 18:14
@ericawright ericawright changed the title Specific events send GA event when toggling AND send events on interaction Jan 29, 2018
Copy link
Contributor

@aminalhazwani aminalhazwani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left a couple of comments of things we could change if they sound good to you, thanks!

document.getElementById('showExample').style.display='block';
return false;">Interactive example</a> (Currently only renders correctly in Firefox.)
</small>
<div class='interactive'>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here it should probably be <div class="interactive"> instead of <div class='interactive'>, so double quotes instead of single quotes

@@ -440,3 +440,83 @@ div[data-tabs] input[type="radio"]:checked + label {
div[data-tabs] input[type="radio"]:checked + label + div[data-tab] {
display: block;
}

/* Interactive checkbox */
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am wondering if these styles should live in a separate scss partial. I will leave it as it is and we can change it later if you agree 😃

@aminalhazwani aminalhazwani merged commit 4529cab into staging Jan 30, 2018
@ericawright ericawright deleted the specific-events branch January 30, 2018 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants