expo/ex-navigation

Dont apply scale on floatVertical

Open

#37 ouverte le 16 août 2016

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)JavaScript (1 002 stars) (207 forks)batch import
enhancementhelp wanted

Description

as discussed on exponent slack with @brentvatne, we shouldn't do scale on floatVertical. That's any the default animation anyway atleast on ios.

Refer the thread here: https://exponentjs.slack.com/files/naveenkonduru/F2190U1J8/pasted_image_at_2016_08_14_05_52_pm.png

Guide contributeur

Stack technique
react nativejavascript
Domaine
mobile
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
Aucun
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
First, locate the file that defines the floatVertical transition animation, typically in a transitions directory or within navigation configuration. Look for a scale property being applied to the floatVertical transition and remove or disable it. The Slack thread mentioned may provide additional context, but is not publicly accessible. The change is minimal and localized.