Skip to content

feat: modal:initialized callback + preload param + $.modal.get method. Fixes #279#293

Open
Zenoo wants to merge 1 commit intokylefox:masterfrom
Zenoo:fix-279
Open

feat: modal:initialized callback + preload param + $.modal.get method. Fixes #279#293
Zenoo wants to merge 1 commit intokylefox:masterfrom
Zenoo:fix-279

Conversation

@Zenoo
Copy link

@Zenoo Zenoo commented Jul 1, 2021

  • New event modal:initialized that runs once the modal content has been loaded.
  • New preload param, defaults to false. When set to true, the modal doesn't open on initialization, and stays preloaded until we .open() it ourselves.
  • New $.modal.get($elm) method that returns the modal instance of the linked element.

These changes should also fix #279.

@Zenoo
Copy link
Author

Zenoo commented Jul 1, 2021

As a side note, I couldn't get gulp to work with the version linked in package.json, so the minification comes from an automated VSCode tool.
Gulp 3 doesn't work with Node 11+, maybe upgrading it would be a good idea.

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.

Embed code inside modal not loading on page load

1 participant