Skip to content

add support for jQuery 4#75

Merged
n-rodriguez merged 2 commits into
jbox-web:masterfrom
reiern70:reiern70/jQuer40
Jun 17, 2026
Merged

add support for jQuery 4#75
n-rodriguez merged 2 commits into
jbox-web:masterfrom
reiern70:reiern70/jQuer40

Conversation

@reiern70

@reiern70 reiern70 commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@ernestosemedy

Copy link
Copy Markdown

@n-rodriguez Could you please review this PR? We are using your library in our app and it is not adapted to jQuery 4. Thanks in advance!

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates Bootstrap Notify’s callback guards to avoid relying on $.isFunction (removed in jQuery 4), aiming to restore compatibility with newer jQuery versions.

Changes:

  • Introduces a helper intended to replace $.isFunction checks.
  • Replaces several $.isFunction(...) usages with calls to the new helper for onShow, onShown, onClick, onClose, and onClosed.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/bootstrap-notify.coffee Outdated
Comment thread src/bootstrap-notify.coffee
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@n-rodriguez n-rodriguez merged commit 1149af8 into jbox-web:master Jun 17, 2026
2 checks passed
@reiern70

Copy link
Copy Markdown
Contributor Author

@n-rodriguez Thanks for merging. When can we expect a new release?

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