-
Notifications
You must be signed in to change notification settings - Fork 0
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
Rename "trumps" layer to "utilities" #24
Comments
I really like this and I am personally comfortable with the name. But there is a lot of confusion for other people, and the second pro for "utilities" you (@csshugs) raised is something that makes me think we should probably rename it. cc @csswizardry |
Yep. That makes a lot of sense. And I ran into some trouble myself explaining the u-namespace for the trumps level. |
I fully agree with @nenadjelovac Beyond that, I think the main problem is the maintenance of the framework. There's much point to validate and implement (namespaces, placeholder/silent classes, documentation, ...). Update : it isn't as important, but I think @csswizardry, has also changed |
Yup, I stumbled upon some examples from him somewhere. |
Will make more sense to use the u- namespace indeed. |
Closing this, since we won't introduce such a huge change in the pre-alpha version. Also, this is already implemented in the beta version. |
@aaronstezycki came up with this question here.
I think it's better to create a proper issue here for discussion.
The idea is to remane the "trumps" layer to "utilities".
What I like about "trumps" is that the naming hints at its function, i.e. to trump everything else because you definitely want to take it effect, whatever the class is doing. Since every declaration in this layer has to contain an
!important
, the correlation in naming is more obvious (to me) than it would be with "utilities".On the other hand, naming this layer "utilities" would align it with other frameworks like SUIT CSS and Foundation (naming wise) and probably would clear any confusion to inuit newbies, don't familiar with the trumps layer.
An even more obvious pro for "utilities" is the soon coming u-namespace. The
t-
namespace is reserved for "theme" so it seems like we have to go with theu-
namespace for the trumps layer. Confusing right? I think this is an important point to consider. When introducing these namespaces to the framework, renaming the layer to "utilities" would definitely prevent any confusion.Any thoughts @nenadjelovac @florianbouvot ?
The text was updated successfully, but these errors were encountered: