search

htmlacademy / html2-basic-template

Gulp сборка: html, sass, stack, webp.
36 stars 222 forks source link

Улучшение обработки стилей #77

Closed firefoxic closed 6 months ago

firefoxic commented 6 months ago

  1. Связка autoprefixer+postcss-csso заменена на postcss-lightningcss, потому что он с обеими задачами справляется уже лучше тех двоих (например csso до сих пор ничего не знает про range-media, а autoprefixer почему-то не смотрит на поддержку браузерную и до сих пор ругается на start/end во флексовом контексте, которые там уже давно работают).

  2. Теперь студентам нет нужды писать пути до фрагментов стека — postcss-url сам заменит рабочий для исходников путь до файла иконки на рабочий для билда путь до фрагмента стека. Причём вложенные в подпапки иконки тоже поддерживаются — фрагмент будет ссылаться на верный айдишник:

    image

  3. Добавлены примеры подключения медиа-ресурсов.

nikolai-shabalin commented 6 months ago

Пока есть сомнения. Чуть позже вчитаюсь в твои предложения и отвечу

nikolai-shabalin commented 6 months ago

Сейчас меня смущает @use, которого фактически нет на курсах

nikolai-shabalin commented 6 months ago

Можно просто копипастить

Копипастить что? Без понимания того, что ты копипастишь всегда будет присутствовать чувство "я всё сделал неправильно" или "а что я вообще сделал".

Но с ней зато не выйдет так, что sass отрубит импорты совсем, а мы — ой, не готовый.

Даже если отрубит, то ничего страшного, так как мы просто не обновим версию SASS. Так что у нас есть время нормально перейти на @use и не выкручивать самим себе руки.

nikolai-shabalin commented 6 months ago

Был же скринкаст Андрея.

"Бывает @use, а бывает @import" - ты про этот? А потом весь курс на импортах.

nikolai-shabalin commented 6 months ago

На таком сравнении студентам даже проще будет понять их разницу.

Мы куда-то не туда пошли, Серёжа. Я не против добавления @use в программу =) Просто этого не сделать прямо сейчас, поэтому и сборку не перевести на @use.

Дело в том, что sass теперь не видит импорта файла с переменными

Из этого сообщения я понял, что файл с переменными можно подключить только с помощью @use. Пускай оно так и работает, но не в этой версии курса, в котором нет ничего про @use. Сейчас проблема в том, что нет времени добавить прямо сейчас.

firefoxic commented 6 months ago

Удалил почти всё, осталось только улучшение исправления путей. И добавил Resolves #64 @nikolai-shabalin посмотри.