microsoft / fast

The adaptive interface system for modern web experiences.
https://www.fast.design
Other
9.25k stars 590 forks source link

chore: remove 'applyMixins' function #6774

Closed nicholasrice closed 1 year ago

nicholasrice commented 1 year ago

Pull Request

📖 Description

This PR removes the 'applyMixins' function from exported features. 'applyMixins' is an implementation detail of fast-foundation components and doesn't relate to the package's purpose. Additionally, there are better mixin patterns that don't rely on copying property descriptors manually, such as defining mixin classes as functions accepting a base class.

🎫 Issues

👩‍💻 Reviewer Notes

📑 Test Plan

✅ Checklist

General

Component-specific

⏭ Next Steps