-
Notifications
You must be signed in to change notification settings - Fork 236
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Заменить sass-плагин для gulp на такой для postcss #89
Comments
Уже обсуждали. Пока не можем внедрить из-за работы с |
Для обсуждения ишью и завёл, чтобы то обсуждение не пропало в закрытом пулреквесте.
Так и не будет резкого перехода. Вообще перехода не будет. Импорты так и останутся для склейки файлов. А юзы только для использования чего-то сассовского в одном файле из другого. И знакомство с этим можно размазать на постепенное, потому что если не использовать в файле переменных/миксинов из другого файла, то юзы не нужны, а как только захотелось, что-то заиспользовать — добавили. Юзов то не так много будет, если не упарываться по сасс-фичам. |
Плюс к мотивацииСейчас сорсмапы как-то странно работают: На это студенты внимание обратили. А у моего студента (в проекте которого уже есть предлагаемое улучшение сборки), сорсмапы корректно работают: Не знаю, как это чинить в текущей сборке, да и знать не хочу, ибо зачем, если есть не костыль к чему-то падающеу, а замена ему на новое хорошо стоящее на ногах (что конечно не помогает студентам текущего потока, но уж как есть). |
Плюсы:
Бонус (не минус!):
@use
, что несложно совсем, и может даже не потребует добавлений в материалы курса, зато в полном соответствии с рекомендациями авторов сасса.The text was updated successfully, but these errors were encountered: