intuit / intuit/postcss-themed
Refactor Legacy and Modern Theme Generation
Open
enhancement
hacktoberfest
- Dominant language
- TypeScript
- Stars
- 61
- Forks
- 30
- PR merge metrics
- No merged PRs in 30d
Description
When we started this project we had to support IE11 which influenced what is now the "legacy" implementation of theme generation. When we dropped IE support we switched to the modern style.
These generators currently live in the same file and tend to have large individual functions, where some logic could probably be broken out.
If you're interested, I'd be happy to review a pull request to split out the implementations into separate files and clean up some of the functions and comments.
Make sure all of the tests still pass!
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.