clay icon indicating copy to clipboard operation
clay copied to clipboard

Published 20 hours ago • verified publisher icon sebastiaanvisser

`separate` function (related to `visibility`) should be removed

Open nsaunders opened this issue 2 years ago • 3 comments

The separate function declared here should be removed.

image

nsaunders avatar Mar 02 '22 12:03 nsaunders

Added here: https://github.com/sebastiaanvisser/clay/commit/4221200c53599f81d8f86c99f3b7c5c9fe4556f3

ddssff avatar Mar 02 '22 13:03 ddssff

Ah, so it seems the type signature borderCollapse :: Visibility -> Css is actually wrong? There is a comment here https://github.com/sebastiaanvisser/clay/commit/4221200c53599f81d8f86c99f3b7c5c9fe4556f3#diff-a82ec625bccfadbae37e6579b4a63ecadcd99a3090a43478674483df463b4a24R160-R170 describing a cleaner but lengthier alternative. I'd prefer that alternative as well, but it will trigger a major version bump.

turion avatar Mar 02 '22 14:03 turion

Good find @turion. As you know, naming conflicts have been resolved with "namespacing" before, so that would certainly be an option. On the other hand, could this be resolved with, say, a Collapse type class (similar to global keywords)? If so, I suppose it would still technically break compatibility, but (if I'm thinking about it correctly) existing code should still compile... What do you think?

nsaunders avatar Mar 02 '22 14:03 nsaunders