rename .opacity mixin - Githubissues

pmsipilot / ui

A themable UI library based on Bootstrap

http://pmsipilot.github.io/ui/

MIT License

8 stars 5 forks source link

rename .opacity mixin #6

Open iamdey opened 9 years ago

iamdey commented 9 years ago

this mixin is in conflict w/ twitter bootstrap opacity

several solutions :

rename the mixin (eg. .ui-opacity())
add twitter bootstrap as a deps
mark this as incompatible w/ twitter bootstrap

jubianchi commented 9 years ago

~~ça vaudrait pas le coup de prefixer toutes les mixins par ui- pour éviter les conflits avec bootstrap mais également d'autres framework CSS ?~~

Should we use a prefix (for example ui-) for every mixins to avoid conflict with others CSS frameworks, not only bootstrap ?

youknowriad commented 9 years ago

bonne idée @jubianchi et autre chose, essayer de passer par les mixins twitter bootstrap en priorité lorsque ceux ci sont disponibles et les supprimer dans ce cas là de ui