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

Outside-if-not working with version 2.2.1, not anymore with newer versions #29

Closed
rbnvrw opened this issue Feb 20, 2016 · 11 comments
Closed
Labels

Comments

@rbnvrw
Copy link

rbnvrw commented Feb 20, 2016

When I update the plugin to a version > 2.2.1, the outside-if-not attribute stopped working. Small code snippet:

<div  data-click-outside="deselectFormElement()" 
data-outside-if-not="editor__sidebar,option__button">

Downgrading to version 2.2.1 fixes the issue. Could you please fix it in a new version?

Please consider adding unit tests/e2e tests to prevent regressions. Other than that, I really appreciate your plugin!

@IamAdamJowett
Copy link
Owner

@rbnvrw I am putting tests in at the moment, but manual tests in the example app within this repo is not showing the same behavior, do you have a jsfiddle or the like for me to see the behavior?

@IamAdamJowett IamAdamJowett removed the bug label Feb 21, 2016
@rbnvrw
Copy link
Author

rbnvrw commented Feb 21, 2016

That is great. I do not have the time to create a jsfiddle now, but I will make one for you tomorrow.

@elliottregan
Copy link

I can second this. I haven't had time to really debug it, but I had to roll back to 2.2.1 to get it to work again.

@IamAdamJowett
Copy link
Owner

@rbnvrw and @elliottregan a quick question, are by chance either of you on a touchscreen laptop or PC? (not a tablet or mobile)

@IamAdamJowett
Copy link
Owner

@rbnvrw @elliottregan Check out release 2.5.0 and let me know if this fixes the issues you are finding.

@elliottregan
Copy link

Seems to be working again! Thanks!

I was on a regular Mac, no touchscreen.

@elliottregan
Copy link

@IamAdamJowett This bug is back in the latest release. Same thing, no touch screen.

Also in 2.6.2.

@rbnvrw
Copy link
Author

rbnvrw commented Mar 17, 2016

Here also no touchscreen.

@IamAdamJowett
Copy link
Owner

@elliottregan I assume you mean 2.6.2, I will see what has changed that may have brought this back

@IamAdamJowett IamAdamJowett reopened this Mar 18, 2016
@elliottregan
Copy link

Yup. Fixed my comment.

@IamAdamJowett
Copy link
Owner

@elliottregan fixes should be in the latest version 2.8.2 that covers this off. 5c64f22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants