feat(switch): Move component specific logic out of foundation

[lynnmercier]

No description provided.

[mdc-web-bot]

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

[codecov-io]

Codecov Report

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

@@            Coverage Diff             @@
##           master    #3342      +/-   ##
==========================================
- Coverage   98.13%   98.12%   -0.01%     
==========================================
  Files         123      123              
  Lines        5194     5185       -9     
  Branches      638      638              
==========================================
- Hits         5097     5088       -9     
  Misses         97       97

Impacted Files	Coverage Δ
packages/mdc-switch/index.js	97.95% <100%> (-0.05%)	⬇️
packages/mdc-switch/foundation.js	100% <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 dd20ea8...9c0081b. Read the comment docs.

[mdc-web-bot]

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

[kfranqueiro]

Removing the init code from the foundation regresses #3147. (The two #init tests you removed are meaningful to this.)

Maybe we can relocate that handleChange call to the component's initialSyncWithDOM, but we would also need to pass in a mock event/target based on the initial state, and that seems maybe a little hacky.

[lynnmercier]

Re: #3147, "Firefox would remember the state of the hidden input element on refresh "

How do I reproduce this bug?

[kfranqueiro]

In Firefox:

• npm run dev
• Go to http://localhost:8080/switch.html
• Toggle the first switch
• Refresh
• Toggle the first switch again

Or, in Chrome, it can be reproduced via browser history:

• Toggle the first switch
• Use the arrow in the MDC Toolbar (not the browser's back button) to navigate to the index page
• Use the browser's back button to go back to the switch page
• Toggle the first switch again

During the last step, the switch doesn't move... because it was actually already toggled, but its UI wasn't in sync with the fact that the browser recalled its state.

[lynnmercier]

Okay i fixed this bug where the checked state can be out of sync with the styling. I moved the logic to the vanilla component in the initialSyncWithDom method

[mdc-web-bot]

All 353 screenshot tests passed for commit 632b550 vs. master! 💯🎉

[kfranqueiro]

FYI I think this needs a BREAKING CHANGE: when merged because it removes 2 foundation APIs and makes handleChange require an event parameter.

[mdc-web-bot]

All 353 screenshot tests passed for commit 69dca96 vs. master! 💯🎉

[mdc-web-bot]

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

[mdc-web-bot]

All 353 screenshot tests passed for commit 62325ac vs. master! 💯🎉

[mdc-web-bot]

All 353 screenshot tests passed for commit 9c0081b vs. master! 💯🎉