search

youseedk / dna

CSS framework for yousee.dk
ISC License
6 stars 6 forks source link

Add helper-classes/utilities-for-text #260

Closed Nomi97 closed 4 years ago

Nomi97 commented 4 years ago

I want utilities classes added for text like text-center, text-bold or text-semi-bold

grandorf commented 4 years ago

I am not sure we want this. There are soo many helper classes in Bootstrap. YouSee DNA is not Bootstrap. We are only using the grid. If you want to utilise Bootstrap more comprehensively I think you need to add Boootstrap as a dependency in your site as you have now. But let us hear what others think. If many others share your wish then of cause we should consider it!

grandorf commented 4 years ago

By the way. We do have this in DNA: https://dna.yousee.dk/docs/components/helper-classes/utilities-for-layout

If we have a handful of helper classes we keep using all the time then we can consider to add these 🧬

Nomi97 commented 4 years ago

From what i can see Bootstrap has Components, Layout and Utilities and DNA covers 2 of these. It would be nice if we could make bootstrap obsolete and only rely on DNA for all our styles.

Nomi97 commented 4 years ago

And i'm aware that DNA is using bootstrap utilities like https://dna.yousee.dk/docs/components/helper-classes/media-queries.html and https://dna.yousee.dk/docs/components/helper-classes/utilities-for-layout.

I just want a few more of them added.

Like having a helper-classes/utilities-for-text helper-classes/utilities-for-spacing

Nomi97 commented 4 years ago

I figured adding all the utilities in one PR might be to much so changing this issue to utilities-for-text specifically