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

Allow style rules subsetting #140

Merged
merged 1 commit into from Jul 5, 2020

Conversation

autarc
Copy link
Contributor

@autarc autarc commented Jul 4, 2020

Pull Request

Currently the output of .createMediaStyle() includes all style rules that are possible based on the breakpoint keys. Even if just some are used in a codebase it always results in the same string. While it might not be much it's an overhead that can be avoided by allowing developers to specify a potential subset.

For better referencing the supported breakpoint keys the entries were defined and exported as an enum.

@damassi
Copy link
Member

damassi commented Jul 5, 2020

Really nice PR! thanks @autarc 👍

@damassi damassi merged commit 89e6efc into artsy:master Jul 5, 2020
@artsyit
Copy link
Contributor

artsyit commented Jul 5, 2020

🚀 PR was released in v1.2.0 🚀

@alloy
Copy link
Collaborator

alloy commented Jul 8, 2020

Nice one, thanks! ✨

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants