Skip to content

Conversation

lupestro
Copy link
Contributor

Updated recognize-gesture modifier to use latest ember-modifier (fixes one deprecation)
Fixed recognize-gesture invocation of -gestures service during render (fixes the other)

Using ember-modifier requires a minimum of Ember 3.3 (rather than 3.1) for ember-destroyable-plugin, which it calls internally, to function. Not sure if that qualifies this fix to be a sem-ver major bump, but I suspect so.

I'll defer updating the changelog until I hear what the committers want to do.

Fixed invocation of -gestures service during render.
ember-modifiers use now requires minimum Ember 3.3.
@lupestro
Copy link
Contributor Author

lupestro commented Jan 31, 2021

The travis-ci failures in LTS 2.12, 2.16, 2.18 are expected with this change. We're effectively reducing the range of Ember versions with which this will work so that the modifier will work without deprecations in recent versions of Ember. It will definitely need a new major version to insulate users of Ember < 3.3 from the change.

@pieter-v
Copy link
Contributor

@lupestro
Using this branch, I have exceptions in willRemove during rerender my component.
Would it be possible to change

willRemove() {
  this.manager.destroy();
  this.manager = null;
}

to

willRemove() {
  if (this.manager !== null) {
    this.manager.destroy();
    this.manager = null;
  }
}

@lupestro
Copy link
Contributor Author

lupestro commented Sep 6, 2021

Hi @pieter-v - I swung back around here, doing maintenance on a related project, and stumbled over your comment from back in March. I'll make the fix shortly. Sorry for the long delay.

@pieter-v
Copy link
Contributor

@lupestro thanx

galfert added a commit to 67P/hyperchannel that referenced this pull request Dec 3, 2021
Fixes a deprecation warning when used with Ember 3.24.
Taken from html-next/ember-gestures#137
galfert added a commit to 67P/hyperchannel that referenced this pull request Dec 7, 2021
Fixes a deprecation warning when used with Ember 3.24.
Taken from html-next/ember-gestures#137
@veelci
Copy link

veelci commented Nov 14, 2023

@lupestro Any updates on this? I just upgraded an application to Ember 3.28 and am running into #138

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.

3 participants