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

leave message on mouseover if sticky #223

Merged
merged 1 commit into from
Dec 29, 2016

Conversation

sbatson5
Copy link
Collaborator

Fixes: #222

The method to defer our timers wasn't checking to see if that object was sticky. This PR adds an early return to the timer methods and test to ensure they are not called.

@sbatson5 sbatson5 merged commit ebed6f7 into master Dec 29, 2016
@sbatson5 sbatson5 deleted the 222-mouse-over-with-sticky-message branch December 29, 2016 19:01
@mnoble01
Copy link

mnoble01 commented Jan 4, 2017

@sbatson5 any plans for a patch release soon? I just ran into this issue in production code.

@sbatson5
Copy link
Collaborator Author

sbatson5 commented Jan 4, 2017

@mnoble01 The PR is open for 1.4.1. Unfortunately, I don't have npm access, so I can't publish it myself. Will ping @poteto.

In the meantime, you can reference the commit directly in your package.json as a temporary solution.

"ember-cli-flash": "poteto/ember-cli-flash#942e4375c8ffb0a0a93f4adbc236fe94ce708ae1"

@mnoble01
Copy link

mnoble01 commented Jan 4, 2017

Thanks, @sbatson5

@sbatson5
Copy link
Collaborator Author

sbatson5 commented Jan 4, 2017

@mnoble01 looks like 1.4.1 is published thanks to @Dhaulagiri

@mnoble01
Copy link

mnoble01 commented Jan 4, 2017

@sbatson5 Yes, thanks to you both for the quick turnaround. I've already upgraded to 1.4.1 👍.

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.

2 participants