-
Notifications
You must be signed in to change notification settings - Fork 715
[css-borders-4] Rename border-clip-<side>
to border-<side>-clip
?
#9252
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
Comments
Renaming them makes sense to me. As that's probably a quick decision, let's add this to the agenda. Sebastian |
|
So far it isn't and so isn't Discussing whether the new properties should be affected or even be part of the related shorthands, is a whole different discussion. @Loirooriol Can you please open a seperate issue for that? Sebastian |
I'm just going to mark this as accepted Editor Discretion. This feature was only partially drafted in the first place, and this is an obvious fix to that editor's draft. The whole feature needs CSSWG review once it's properly drafted anyway. |
I don't think the relationship with the shorthand is an unrelated discussion. I prefer |
I am not aware of the discussions that have taken place about Assuming this syntax would be valid in the future...: .box {
border: {
color: green;
clip: 10px 1fr;
top: {
clip: normal;
color: red;
}
}
} ... I do not have a clear idea on how shorthand expansion would work, but the order of the components in |
I am not sure there is a pressing need to stabilize and implement
border-clip
but the naming of its longhands seems inconsistent with otherborder
properties:border-top-color
border-top-radius
border-top-style
border-top-width
border-clip-top
The text was updated successfully, but these errors were encountered: