search

manGoweb / MangoPress

(WordPress + Nette)²
35 stars 4 forks source link

Dependencies: bourbon lock #103

Closed FilipChalupa closed 6 years ago

ViliamKopecky commented 6 years ago

Nechceme spis prejit na 5? :)

Myslim ze by stacilo opravit cestu k libce a opravit jedno spatny pouziti size mixinu

@import '../../node_modules/bourbon/core/bourbon'

Jsem taky chystal pull request, a chtel jsem spolecny veci presunout do _common.sass, ale zda se mi ze import souboru s hvezdickou funguje jenom v entry souboru (jako index.sass, print.sass, ...) co se kompiluje a treba v _common.sass uz hvezdicka nefunguje.

To nejak silene hackujeme, co @enzy? :)

vasimi commented 6 years ago

@ViliamKopecky Nerozbije nahodou prechod na 5 build vsech existujicich projektu, ktere pouzivaji size vic nez jedenkrat/jine funkce z bourbonu, ktere uz nefunguji nebo funguji jinak?

FilipChalupa commented 6 years ago

@ViliamKopecky Nejsem proti, ale řeším teď něco jiného. Lock na 4 šel v pohodě udělat i z browseru.

FilipChalupa commented 6 years ago

@MekliCZ Staré projekty to nerozbije, protože ty mají starý kód. Aktuální MangoPress na ně nemá vliv. (Rozbité ale už jsou, když nejsou locknuté na kompatibilní verzi.)

enzy commented 6 years ago

Bourbon v5 už v sobě skoro nic nemá, takže pozbývá smyslu :D ... takže klidně zrušit úplně a nahradit pouze pár funkcí, které používáme (position možná jen?)