Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

panel: sometimes doesn't appear in 1.1.23 #11946

Closed
Splaktar opened this issue Jun 22, 2020 · 4 comments · Fixed by #11947
Closed

panel: sometimes doesn't appear in 1.1.23 #11946

Splaktar opened this issue Jun 22, 2020 · 4 comments · Fixed by #11947
Assignees
Labels
has: Pull Request A PR has been created to address this issue P1: urgent Urgent issues that should be addressed in the next minor or patch release. resolution: fixed severity: regression This issue is related to a regression type: bug
Milestone

Comments

@Splaktar
Copy link
Member

Bug

Demo and steps to reproduce the issue

This blank StackBlitz demo can be used to create a reproduction that demonstrates your issue.

Demo URL (required)*: https://material.angularjs.org/HEAD/demo/panel#panel-animations

Detailed Reproduction Steps

  1. Open dialog on panel animations demo
  2. Open dialog twice in reuse demo

Explain the expected behavior

Demos should still work

Explain the current behavior

Panel Animations and Reuse demos are broken.

Discuss the use-case or motivation for changing the existing behavior

Correctness

List the affected versions of AngularJS, Material, OS, and browsers

  • AngularJS: 1.8.0
  • AngularJS Material: 1.1.23
  • OS: macOS
  • Browsers: Chrome

Add anything else we should know

Screen Shot 2020-06-22 at 14 00 33
Screen Shot 2020-06-22 at 14 00 19

@Splaktar Splaktar self-assigned this Jun 22, 2020
@Splaktar Splaktar added P1: urgent Urgent issues that should be addressed in the next minor or patch release. severity: regression This issue is related to a regression labels Jun 22, 2020
@Splaktar Splaktar added this to the 1.1.24 milestone Jun 22, 2020
@oliversalzburg
Copy link
Contributor

oliversalzburg commented Jun 22, 2020

It seems like only the Slide and Scale animations are broken. Seems like there are leftover inline styles from the end of the previous animation that cause problems on the second run.

@Splaktar
Copy link
Member Author

Splaktar commented Jun 22, 2020

For the first issue of panel animations breaking, that's caused by PR #11942 needing to also this.innerWrapper.removeClass('_md-panel-offscreen'); in _updatePosition(). I've got this fix ready.

For the second issue of re-use of panels causing the reused panel to not remove all of the inline styles after animating closed, that looks like it was caused by PR #11390. I'm investigating the right way to eliminate these styles being left around.

@Splaktar
Copy link
Member Author

OK, PR #11390 does not appear to be at fault here.

It appears that PR #11856 made panels more re-usable, but it didn't account for the transforms on the md-panel needing to be cleared after the close animation completes.

I've got a solution for this as well and will post a PR to resolve both of these issues soon.

@Splaktar Splaktar added the has: Pull Request A PR has been created to address this issue label Jun 22, 2020
Splaktar added a commit that referenced this issue Jun 22, 2020
- this resulted in animated in panels not displaying on screen

Fixes #11946
Splaktar added a commit that referenced this issue Jun 22, 2020
- so that re-used panels don't accumulate transforms
- improve type safety and JSDoc comments
- remove unused variable and adjust code styles

Fixes #11946
Splaktar added a commit that referenced this issue Jun 23, 2020
- this resulted in animated in panels not displaying on screen

Fixes #11946
Splaktar added a commit that referenced this issue Jun 23, 2020
- so that re-used panels don't accumulate transforms
- improve type safety and JSDoc comments
- remove unused variable and adjust code styles

Fixes #11946
@Splaktar
Copy link
Member Author

This is fixed in master. Please try out the fix (before we release it) via npm install http://github.com/angular/bower-material#master --save or updating your package.json with "angular-material": "git+ssh://[email protected]/angular/bower-material.git#master",.

If you find any issues, please report them via GitHub issues ASAP.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
has: Pull Request A PR has been created to address this issue P1: urgent Urgent issues that should be addressed in the next minor or patch release. resolution: fixed severity: regression This issue is related to a regression type: bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants