search

nicothin / NTH-start-project

Startkit for HTML / CSS / JS pages layout.
https://nicothin.pro/NTH-start-project/blocks-demo.html
Do What The F*ck You Want To Public License
616 stars 214 forks source link

fixed write to entry.js #87

Closed Hrommi closed 5 years ago

Hrommi commented 5 years ago

Исправлен баг с записью элементов в entry.js. В текущем виде у меня добавлялись только элементы из обязательного блока alwaysAddBlocks.

nicothin commented 5 years ago

Все несколько сложнее. Кажется, в вашем варианте в сборку не будут взяты js-файлы из блоков alwaysAddBlocks.

Я написал раздельное добавления для alwaysAddBlocks и для использованных в разметке Блоков: https://github.com/nicothin/NTH-start-project/commit/6beaa5cedc2c5c451a8c9d8acb7cc9fab6cbe575

Пожалуйста, попробуйте собрать свой проект с этим моим изменением (просто замените ф-ю writeJsRequiresFile в галпфайле на эту мою новую: https://github.com/nicothin/NTH-start-project/blob/6beaa5cedc2c5c451a8c9d8acb7cc9fab6cbe575/gulpfile.js#L279-L308

И напишите решило ли это проблему.

Hrommi commented 5 years ago

Ваше решение тоже исправило проблему, но разве nth.blocksFromHtml уже не содержит элементы из alwaysAddBlocks? https://github.com/nicothin/NTH-start-project/blob/6beaa5cedc2c5c451a8c9d8acb7cc9fab6cbe575/gulpfile.js#L49

nicothin commented 5 years ago

И правда, вполне себе содержит. Однако, по совершенно иррациональной причине мне хочется, чтобы require блоков из секции alwaysAddBlocks обязательно шли до блоков, написанных в разметке :D

Hrommi commented 5 years ago

Пусть будет так :) Спасибо за помощь.