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.

Sign up for GitHub

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

Jump to bottom

Add invocation of ready function #347

Open

Flimm wants to merge 1 commit into HubSpot:dev from Flimm:ready_invocation

Flimm commented Jul 12, 2024

The ready JQuery code is:

$(document).ready(function () {});

Currently, the modern equivalent is listed as:

function ready(fn) {
  if (document.readyState !== 'loading') {
    fn();
  } else {
    document.addEventListener('DOMContentLoaded', fn);
  }
}

However, this isn't exactly equivalent, as the ready function does not get called. This pull request makes it equivalent to the JQuery code.


          Add invocation of ready function

dea66a1

This makes it equivalent to the JQuery code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet