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 CSS property list-style #162

Merged
merged 1 commit into from
Apr 23, 2019

Conversation

jaredbeck
Copy link
Contributor

Hello, first time contributing to loofah. I'm not sure if this should go in ACCEPTABLE_CSS_PROPERTIES or in SHORTHAND_CSS_PROPERTIES but of those two, only the former makes my test pass.

@flavorjones
Copy link
Owner

@jaredbeck Thank you for submitting this pull request! It seems reasonable. Give me a few days to carve out some time and I'll take a closer look.

@flavorjones
Copy link
Owner

@jaredbeck Thank you! Merging, this will be in v2.3.0 when it ships.

@flavorjones flavorjones added this to the v2.3.0 milestone Apr 23, 2019
@flavorjones flavorjones merged commit 8220c70 into flavorjones:master Apr 23, 2019
flavorjones added a commit that referenced this pull request Apr 23, 2019
@jaredbeck jaredbeck deleted the allow_list-style branch April 23, 2019 16:58
@flavorjones
Copy link
Owner

v2.3.0 has been released.

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

Successfully merging this pull request may close these issues.

2 participants