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

Refactor internals – 🎬 2 #21

Merged
merged 2 commits into from
May 25, 2018
Merged

Refactor internals – 🎬 2 #21

merged 2 commits into from
May 25, 2018

Conversation

sandydoo
Copy link
Owner

  • Correctly expose '_trigger' method and allow passing arbitrary
    arguments.
  • Expose place in autocomplete.
  • Remove usage of const.
  • Add assertions to base map-component.
  • Remove unused getPosition method from marker component.
  • Cleanup options processing.
  • Collect extracted event names in _eventAttrs. Use this array to
    filter out attrs when processing options.
  • Ensure events is an object, not an array of event names.
  • Allow registering events using both an events hash and by passing
    attributes.

sandydoo added 2 commits May 25, 2018 18:27
* Correctly expose '_trigger' method and allow passing arbitrary
  arguments.
* Expose `place` in autocomplete.
* Remove usage of `const`.
* Add assertions to base map-component.
* Remove unused `getPosition` method from marker component.
* Cleanup options processing.
* Collect extracted event names in `_eventAttrs`. Use this array to
  filter out attrs when processing options.
* Ensure `events` is an object, not an array of event names.
Allow registering events using both an `events` hash and by passing
attributes.
@sandydoo sandydoo added the enhancement New feature or request label May 25, 2018
@sandydoo sandydoo merged commit 2b23f92 into master May 25, 2018
@sandydoo sandydoo deleted the refactor-second-pass branch May 25, 2018 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant