search

dudeonthehorse / kilogram

Email Development Framework
504 stars 94 forks source link

Added browserSync #26

Closed maksim-shabanski closed 8 years ago

dudeonthehorse commented 8 years ago

Принято. Но я уже настолько привык к CMD+R после обработки вотчера, что переучиваться будет проблематично :)

dudeonthehorse commented 8 years ago

Единственное, что мне не очень нравится - наличие: <script id="__bs_script__">//<![CDATA[ document.write("<script async src='/browser-sync/browser-sync-client.2.12.10.js'><\/script>".replace("HOST", location.hostname)); //]]></script> В исходнике на локалхосте.

maksim-shabanski commented 8 years ago

К хорошему легко привыкнуть ) Скрипт добавляется только в браузер, в файл он не записывается.

dudeonthehorse commented 8 years ago

Это понятно. Но я зачастую беру код из исходника в браузере, а эта строчка мешает :)

maksim-shabanski commented 8 years ago

Брать из браузера код не очень хорошо, по моему мнению, т.к часто многие плагины встраивают на страницу свои скрипты и стили.

Мне тут пришла идея: Я например всегда отключаю сжатие html и инлайн сталей, оно мне очень сильно мешает. Было бы круто добавлять json-файл с настройками в каждую папку проекта, где будут вкл./выкл. некоторые функции, т.е. по умолчанию включать только нужные плагины, а в файле включать дополнительные настройки.

dudeonthehorse commented 8 years ago

Это дополнительный геморрой. Куда проще закомментить нужные строки в тасках. Нет смысла так все унифицировать. Каждый дрочит как хочет. Пока что.

Если есть желание повозиться, есть ряд задач, которые неплохо бы решить:

  1. gulp-concat вырезает link теги из head. А это плохо в случае когда мы подключает гуглофонты. Подключать их через import в css - не вариант.
  2. html-minifier не минифицирует условные комментарии для аутлука. у разработчика тикет висел давно, но он так его и не решил.

Сегодня-завтра выложу новый шаблон.

maksim-shabanski commented 8 years ago

Исправил html-minifire. https://github.com/dudeonthehorse/kilogram/pull/27

dudeonthehorse commented 8 years ago

Ну огонь же :)

dudeonthehorse commented 8 years ago

.pipe(inlineCss({ removeStyleTags: false }))

Вот эта шляпа не работает. Все рано вырезает линк-теги. Сможешь выяснить причину?

maksim-shabanski commented 8 years ago

Посмотрю, но уже не сегодня. И там что-то лишнее подабовлялось, когда принял пул-реквест.

dudeonthehorse commented 8 years ago

Не горит. Спасибо. Гит с ума немного сходит. Вроде все в порядок привел. Если что заметишь, пиши.

maksim-shabanski commented 8 years ago

Расскажи, зачем оставлять в шапке, после инлайна, все стили в style? Ради медиа запросов?

dudeonthehorse commented 8 years ago

В свое врем решил оставить так, когда экспериментировал с ховерами и прочими перделками. По уму надо бы убирать. Оставляя только медиазапросы, да.

maksim-shabanski commented 8 years ago

Ок, понял. Тогда я нашел решение, сейчас вышлю реквест. Он оставляет только медиа-запросы и не удаляет линки.