Skip to content
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

Отказаться от генерации стека/спрайта #88

Open
firefoxic opened this issue Apr 22, 2024 · 2 comments
Open

Comments

@firefoxic
Copy link
Contributor

Это потребует редактирования программы курса, но немного вроде:

  • минус один критерий про спрайт (просто удаляем)
  • материалы по спрайтам совсем удалять не нужно, вещь в информационном смысле полезная, просто что-то можно в допы перенести
  • немного переписать ДЗ, если в нём есть упоминание спрайта
  • возможно добавить короткий скринкаст с пояснением, почему лучше урл до иконки класть в кастомку и уже эту кастомку подключать в свойства — потому что длинный энкодинг лучше делать один и использовать его многократно короткой кастомкой

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

@nikolai-shabalin
Copy link
Contributor

Ты написал, что нужно поправить или оставить, но совсем забыл про мотивацию. "Отказаться от генерации стека/спрайта, чтобы ..." =)

@firefoxic
Copy link
Contributor Author

firefoxic commented Apr 24, 2024

Мотивации:

  • Хоть чутка упростить сборку.
  • Избавиться от ещё одного gulp-плагина — ещё один шаг к избавлению от привязки к одному сборщику (кстати, vite начиная с 5 версии по умолчанию энкодит svg… хотя я по некоторым причинам это отключаю и использую всё тот же postcss-url).
  • Избавиться от проблем с загрузкой (в ff старый баг с множественной загрузкой большого svg при запросе разных его фрагментов).
  • Более чёткое (для студентов) разделение контентной декоративной графики (не будет никаких юзов иконок из спрайта в разметке).
  • Перенос инфы про спрайты/стеки в допы поможет разгрузить программу курса для добавления и/или расширения освещения более важных тем.
  • Бо́льшая надёжность стилей — с энкодингом не может возникнуть ситуация, когда стили приехали и скрыли визуально текст ссылок/кнопок, а спрайт не доехал/сломался и иконки нет.
  • Переход на стек изначально был промежуточным этапом к переходу на энкодинг. Потому что тогда, несколько лет назад я понял, что перетянуть академию с юзов символьного спрайта сразу на энкодинг нереально, вот и продвигал стек как то, что можно и юзами и масками подключать. Сейчас уже все распробовали подключение через стили и перекраску через маски, да ещё и конвертацию путей допилил — теперь без разницы, во что оно там в билде превращается, главное что работает (а работать с энкодингом будет лучше, чем со стеком).
  • остановите меня…

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants