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

🐛 Buttons disabled check uses boolean #79

Closed
wants to merge 1 commit into from

Conversation

ComiR
Copy link

@ComiR ComiR commented Nov 2, 2017

Fixes #72

I also changed the comparison types to strict.

@xenogew
Copy link

xenogew commented Jun 15, 2018

@KidSysco When will you plan to accept this Pull Request? I got struck with the Next/Previous disabled issue with your latest release.

@KidSysco
Copy link
Owner

Very interesting, give us some time to look at it here and we will keep you posted.

@ComiR
Copy link
Author

ComiR commented Jun 15, 2018

Wow - didn't even remember this PR anymore. Pretty long time ignoring it.
I only was triggered to write this because you asked to "give us some time to look at it" - this PR is from the beginning of NOVEMBER LAST YEAR!
Please don't be offended - it just made my day 🤣.

@benjamin-albert
Copy link
Collaborator

We already have #73 which addresses this issue and other visual glitches that jQuery UI 1.12 introduces.

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.

4 participants