Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(drawer): Destroy list in destroy method #3474

Merged
merged 3 commits into from
Aug 30, 2018

Conversation

williamernest
Copy link
Contributor

refs: #3471
List was being instantiated in the init but never destroyed.

@mdc-web-bot
Copy link
Collaborator

All 349 screenshot tests passed for commit 89ad4f3 vs. master! 💯🎉

@codecov-io
Copy link

codecov-io commented Aug 29, 2018

Codecov Report

❗ No coverage uploaded for pull request base (master@247f75f). Click here to learn what that means.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #3474   +/-   ##
=========================================
  Coverage          ?   98.42%           
=========================================
  Files             ?      119           
  Lines             ?     5026           
  Branches          ?      617           
=========================================
  Hits              ?     4947           
  Misses            ?       79           
  Partials          ?        0
Impacted Files Coverage Δ
packages/mdc-drawer/index.js 97.26% <100%> (ø)

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 247f75f...535e2cd. Read the comment docs.

@mdc-web-bot
Copy link
Collaborator

All 349 screenshot tests passed for commit 2266cf5 vs. master! 💯🎉

@mdc-web-bot
Copy link
Collaborator

All 353 screenshot tests passed for commit 535e2cd vs. master! 💯🎉

@williamernest williamernest merged commit 325317c into master Aug 30, 2018
@williamernest williamernest deleted the fix/drawer/destroy-list-in-destroy branch August 30, 2018 19:29
adrianschmidt pushed a commit to Lundalogik/material-components-web that referenced this pull request Sep 25, 2018
@jamesmfriedman jamesmfriedman mentioned this pull request Sep 26, 2018
49 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants