search

ionic-team / ionic-framework

A powerful cross-platform UI toolkit for building native-quality iOS, Android, and Progressive Web Apps with HTML, CSS, and JavaScript.
https://ionicframework.com
MIT License
51.01k stars 13.51k forks source link

bug: large title/back-button animation is broken when had menu-button #21207

Open theopavan opened 4 years ago

theopavan commented 4 years ago

20796 Bug Report

Ionic version:

[x] 5.x

Current behavior: When navigating from page A to page B, both with a large title and menu button, the back button of the entering page is displayed too early and overlaps with the menu button, then jumps into position. If no menu-button is present it is working as intended.

bug_ionic

Expected behavior: Animations smooth as butter :)

Steps to reproduce: Just navigate to a page with menu button and large title.

Related code: A sample application via GitHub https://github.com/theopavan/project

Other information:

Ionic info:

Ionic:

   Ionic CLI                     : 6.6.0 (/Users/theo/.nvm/versions/node/v10.15.0/lib/node_modules/@ionic/cli)
   Ionic Framework               : @ionic/angular 5.1.0
   @angular-devkit/build-angular : 0.803.26
   @angular-devkit/schematics    : 8.3.26
   @angular/cli                  : 8.3.26
   @ionic/angular-toolkit        : 2.2.0

Cordova:

   Cordova CLI       : 9.0.0 (cordova-lib@9.0.1)
   Cordova Platforms : android 8.1.0, ios 5.1.1
   Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 4.1.3, (and 6 other plugins)

Utility:

   cordova-res (update available: 0.14.0) : 0.6.0
   native-run (update available: 1.0.0)   : 0.2.8

System:

   Android SDK Tools : 26.1.1 (/Users/theo/Library/Android/sdk)
   ios-deploy        : 1.9.4
   ios-sim           : 8.0.2
   NodeJS            : v10.15.0 (/Users/theo/.nvm/versions/node/v10.15.0/bin/node)
   npm               : 6.4.1
   OS                : macOS Catalina
   Xcode             : Xcode 11.4.1 Build version 11E503a
ionitron-bot[bot] commented 4 years ago

Thanks for the issue! This issue has been labeled as needs reproduction. This label is added to issues that need a code reproduction.

Please provide a reproduction with the minimum amount of code required to reproduce the issue. Without a reliable code reproduction, it is unlikely we will be able to resolve the issue, leading to it being closed.

For a guide on how to create a good reproduction, see our Contributing Guide.

theopavan commented 4 years ago

@liamdebeasi Project with reproduction created.

https://github.com/theopavan/project

theopavan commented 4 years ago

@liamdebeasi the correction #21270 don't works? Will be added to some milestone/version?