Skip to content

Port HoverCard animation fix to 5.0#5504

Merged
ThomasMichon merged 3 commits intomicrosoft:5.0from
ThomasMichon:ship/5.0/hover-card
Jul 10, 2018
Merged

Port HoverCard animation fix to 5.0#5504
ThomasMichon merged 3 commits intomicrosoft:5.0from
ThomasMichon:ship/5.0/hover-card

Conversation

@ThomasMichon
Copy link
Copy Markdown
Member

@ThomasMichon ThomasMichon commented Jul 10, 2018

Bringing over a bugfix for an animation issue in HoverCard impacting IE 11.
Also adding lib-commonjs to .gitignore to make it easier to switch between branches.

Microsoft Reviewers: Open in CodeFlow

@ThomasMichon
Copy link
Copy Markdown
Member Author

@Jahnp and @atneik, we need this to resolve a support issue. If you don't get back by tomorrow I'll override!

@Jahnp
Copy link
Copy Markdown
Member

Jahnp commented Jul 10, 2018

@ThomasMichon ack--this was already done in master right?

@atneik
Copy link
Copy Markdown
Contributor

atneik commented Jul 10, 2018

@Jahnp Yes I got to know yesterday that it is fixed in v6, but not in v5

lib
lib-amd
lib-es2015
lib-commonjs
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

ThomasMichon and others added 3 commits July 10, 2018 15:16
* Removes an animation class causing a visual bug in IE.

* Rush change log.
* claudia's test

* add appveyor.yml

* revert README.md back

* update appveyor.yml
@ThomasMichon ThomasMichon force-pushed the ship/5.0/hover-card branch from 7e4fd0e to d00eef5 Compare July 10, 2018 22:16
@ThomasMichon ThomasMichon merged commit a73ab2e into microsoft:5.0 Jul 10, 2018
@microsoft microsoft locked as resolved and limited conversation to collaborators Aug 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants