Skip to content
This repository has been archived by the owner on Feb 28, 2023. It is now read-only.

MODUL-1149: Ameliorer slide transition #1088

Merged
merged 5 commits into from
Jul 31, 2019

Conversation

raphpare
Copy link
Member

@raphpare raphpare commented Jul 22, 2019

@ulaval/modul-components

PR Checklist

  • Provide a small description of the changes introduced by this PR
  • Renommer la prop transition:boolean par disabled:boolean du composant m-accordion-transition
  • Lorsque la prop transition = false ne pas effectuer le this.$scrollTo de la function transitionEnter()
  • Include this section in the release notes
    Rename the prop transition: boolean by disabled: boolean for component m-accordion-transition

Copy link
Contributor

@jpguilmette jpguilmette left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai juste un questionnement sur le changement du nom de la prop.

Si je comprend bien, la prop sert à empêcher la transition sur la composante slide-transition et bloquer un scrollTop. Il me semble que disabled est plus clair que transition. Et en bonus on évite le breaking change

@raphpare
Copy link
Member Author

J'ai juste un questionnement sur le changement du nom de la prop.

Si je comprend bien, la prop sert à empêcher la transition sur la composante slide-transition et bloquer un scrollTop. Il me semble que disabled est plus clair que transition. Et en bonus on évite le breaking change

J'ai changé le nom de la prop pour être constant avec le composant accordion-transition qui possède la même prop. Si nous conservons la prop disabled dans le composant slide-transition, je propose de remplacer la prop transition du accordion-transition par disabled pour conserver une cohérence.

@jipigi jipigi modified the milestones: 1.0.0-beta.115, 1.0.0-beta.116 Jul 23, 2019
@chuckmah chuckmah merged commit e9e5c31 into develop Jul 31, 2019
@chuckmah chuckmah deleted the bugfix/MODUL-1149_ameliorer-slide-transition branch July 31, 2019 19:37
chuckmah added a commit that referenced this pull request Aug 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants