Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

feat(tab): Move event registration to component #3331

Merged
merged 5 commits into from
Aug 13, 2018
Merged

Conversation

kfranqueiro
Copy link
Contributor

@kfranqueiro kfranqueiro commented Aug 10, 2018

Also removes unnecessary animating classes and transitionend handler.

Refs #2813 for Tab.
Fixes #3316.

BREAKING CHANGE: Removes handleTransitionEnd foundation API. Removes [de]registerEventHandler adapter API. Event registration is now the component's responsibility.

@mdc-web-bot
Copy link
Collaborator

All 344 screenshot tests passed for commit d83a862 vs. master! 💯🎉

Copy link
Contributor

@patrickrodee patrickrodee left a comment

Choose a reason for hiding this comment

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

💯

@codecov-io
Copy link

codecov-io commented Aug 13, 2018

Codecov Report

Merging #3331 into master will decrease coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3331      +/-   ##
==========================================
- Coverage   98.06%   98.05%   -0.01%     
==========================================
  Files         120      120              
  Lines        5155     5143      -12     
  Branches      644      643       -1     
==========================================
- Hits         5055     5043      -12     
  Misses        100      100
Impacted Files Coverage Δ
packages/mdc-tab/constants.js 100% <ø> (ø) ⬆️
packages/mdc-tab/foundation.js 100% <ø> (ø) ⬆️
packages/mdc-tab/index.js 96.29% <100%> (+0.21%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1a7f6e7...e91c56f. Read the comment docs.

@mdc-web-bot
Copy link
Collaborator

All 337 screenshot tests passed for commit ce234e3 vs. master! 💯🎉

@mdc-web-bot
Copy link
Collaborator

All 337 screenshot tests passed for commit e91c56f vs. master! 💯🎉

@kfranqueiro kfranqueiro merged commit f2ac793 into master Aug 13, 2018
@kfranqueiro kfranqueiro deleted the feat/tab-events branch August 13, 2018 15:58
@jamesmfriedman jamesmfriedman mentioned this pull request Aug 23, 2018
48 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants