Skip to content

Conversation

phil-blais
Copy link

Description

Please explain the changes you made here. Include an example of what your changes fix or how to use the changes.

Checklist

  • Created tests which fail without the change (if possible)
  • All tests passing (yarn test)
  • Extended the README / documentation, if necessary

* gridstack.js may be freely distributed under the MIT license.
*/

// https://developer.mozilla.org/en-US/docs/Web/API/CustomEvent/CustomEvent
Copy link
Member

Choose a reason for hiding this comment

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

site has it wrapped in a function it calls.. maybe to scope the CustomEvent var ? I would do the same please.

Copy link
Author

@phil-blais phil-blais Mar 5, 2020

Choose a reason for hiding this comment

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

My mistake. Thought the whole poly file was scoped for some reason. Updated.

@adumesny adumesny merged commit 989d92e into gridstack:develop Mar 5, 2020
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.

2 participants