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

Ember 3.17 : error in animateOut #1338

Closed
oliverlj opened this issue Mar 6, 2020 · 16 comments
Closed

Ember 3.17 : error in animateOut #1338

oliverlj opened this issue Mar 6, 2020 · 16 comments

Comments

@oliverlj
Copy link

oliverlj commented Mar 6, 2020

Hello,
When I update to 3.17. i got a parentElement undefined in

    animateOut(dropdownElement) {
      if (!this.animationEnabled) return;
      let parentElement = this.renderInPlace ? dropdownElement.parentElement.parentElement : dropdownElement.parentElement;
      let clone = dropdownElement.cloneNode(true);
      clone.id = `${clone.id}--clone`;
      clone.classList.remove(...this.transitioningInClass.split(' '));
      clone.classList.add(...this.transitioningOutClass.split(' '));
      parentElement.appendChild(clone);
      this.set('animationClass', this.transitioningInClass);
      waitForAnimations(clone, function () {
        parentElement.removeChild(clone);
      });
    }

I use ember-bootstrap-power-select so i don't know yet which one is responsible

@oliverlj
Copy link
Author

oliverlj commented Mar 6, 2020

settings animationEnabled to false does nothing as

    // CPs
    get animationEnabled() {
      if (this._animationEnabled) {
        return this._animationEnabled;
      }

      let config = Ember.getOwner(this).resolveRegistration('config:environment');
      return config.environment !== 'test';
    }

this return true

@it-delta
Copy link

it-delta commented Mar 8, 2020

Hello!
Yes, I see the problem too... Ember 3.16.0

@oliverlj
Copy link
Author

oliverlj commented Mar 9, 2020

no error in the test because animationEnabled is false for test environment

@sinankeskin
Copy link

Same here with v3.17

@aonawale
Copy link

Same here ember v3.17

@RobbieTheWagner
Copy link
Collaborator

Same here with 3.17 as well

@williamweckl
Copy link

Same here with 3.17.

@williamweckl
Copy link

williamweckl commented Mar 17, 2020

no error in the test because animationEnabled is false for test environment

shouldn't be enabled in tests too? This error was silently deployed in production even with a high test coverage 😢

@markusm7
Copy link

Same here with Ember 3.17 and EPS 3.0.6
I see it got tackled in some way in EBD 3.

Any plans to fix this in EPS 3.x.x (or EBD 2.x.x)?

@leondmello
Copy link

I got rid of these errors by updating to https://github.com/cibernox/ember-power-select/releases/tag/v4.0.0-beta.6

@markusm7
Copy link

markusm7 commented Mar 23, 2020

@leondmello yeah as i said it got tackled in ember basic dropdown 3, which is the one EPS 4-beta is using. The problem is im using different addons which rely on EBD 2.

@leondmello
Copy link

@leondmello yeah as i said it got tackled in ember basic dropdown 3, which is the one EPS 4-beta is using. The problem is im using different addons which rely on EBD 2.

Yes, I wrote the suggestion for all the other people for whom it might get fixed by updating to the BETA.

@cibernox
Copy link
Owner

I plan to make the beta final today, after I merge a couple PRs. Hopefully that will invite people using ember octane to update

@markusm7
Copy link

updated all the dependencies today already to ebd 3 and eps 4-beta. glad i moved partially already to octane, seems to continue working like before!

@cibernox
Copy link
Owner

I released 4.0.0 final, so everyone having problems in the very latest version of Ember should be able to update and have them fixed. There is barely any breaking changes, the component uses octane idioms, glimmer components, typescript, @Tracked and all that jazz.

@cibernox
Copy link
Owner

I think this can be closed now, this is fixed in 4.0.0

kevinansfield added a commit to TryGhost/Admin that referenced this issue Mar 26, 2020
closes TryGhost/Ghost#11685

- `ember-source` 3.17.x has an incompatibility with `ember-power-select`
    - see cibernox/ember-power-select#1338
- add `ember-source` to renovate ignore list to avoid unintentional bumps
erikap added a commit to erikap/frontend-backstage that referenced this issue Apr 28, 2020
ember-power-select-with-create depends on ember-power-select 3.x which
gives an error in Ember v3.17.
See cibernox/ember-power-select#1338
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

No branches or pull requests

9 participants