intuit / intuit/postcss-themed

Refactor Legacy and Modern Theme Generation

Open
#20 0 comments 0 reactions 0 assignees View on GitHub
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.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.