По умолчанию не все поддерживают спрайт символов из внешнего файла (легко полифилится, svg4everybody), а в остальном нет проблем. Вот стек да, не везде поддерживается без полифилов, но я его добавил для примера, так как это будущее SVG.
А как же спрайт символов? А как же стек?
При этом обоих этих подходах можно модифицировать цвета. + есть плагины для работы с этим через CLI или Gulp/Grunt/что-то еще.
А я в очередной раз порекомендую реализацю в TARS. Из коробки доступна работа со спрайтами и с svg-symbols с тремя вариантами загрузки этих самых symbols + хелперы для Jade и Handlebars, которые генерируют код для вставки symbol в html. Даже если не будете использовать TARS, можно просто взглянуть на реализацию всего этого добра, может что-то для себя подчерпнете.
Вот как раз в этом месте сначала логичнее указать .foo, а ниже уже .foo.bar
/* плохо */
if (x === undefined || x === null) { /*… */ }
/* хорошо */
if (x == undefined) { /*… */ }
Опять же, иногда нужно явное сравнение с undefined. Это же JS! Вообще == и === нужно использовать только там, где это нужно. Хотите показать, что вы именно с undefined сравниваете, используйте ===, это не будет плохим кодом.
Очень странно написано про каррирование. Из текста кажется, что это просто антипаттерн для всего.
Понимаю, что перевод, но многие рекомендации из статьи обсуждаемы все же.
Ох уж эти правки README.md Даже удивительно, что те, кому нравится ваш продукт не хотят сами в него вложить ничего. Найти бы какой-либо способ замотивировать слать стоящие PR.
Чтобы избавиться от проблем с глубокой вложенностью в node_modules на Windows можно использовать npm 3 версии. Он все пакеты ставит в один node_modules, а не в каждый пакет отдельно.
При этом обоих этих подходах можно модифицировать цвета. + есть плагины для работы с этим через CLI или Gulp/Grunt/что-то еще.
Вот как раз в этом месте сначала логичнее указать .foo, а ниже уже .foo.bar
Опять же, иногда нужно явное сравнение с undefined. Это же JS! Вообще == и === нужно использовать только там, где это нужно. Хотите показать, что вы именно с undefined сравниваете, используйте ===, это не будет плохим кодом.
Очень странно написано про каррирование. Из текста кажется, что это просто антипаттерн для всего.
Понимаю, что перевод, но многие рекомендации из статьи обсуждаемы все же.
Это сфера viewModel, model и т.д. как бы это еще не назвалось.