Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upWhitelist rather than blacklist when merging text fragments #3165
Labels
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently we "merge" text fragments a handful a CSS properties have the same value, i.e. we’re blacklisting differences that should not be merged. This causes issue when the list of properties turns out to be incomplete. See #3163 for example.
Instead, this should probably be a whitelist of properties that are allowed to be different when merging, so that the list being incomplete has a performance cost rather than a correctness cost.