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

Статика (js, img, css) переехала в assets/ #619

Merged
merged 3 commits into from Mar 30, 2015

Conversation

madfriend
Copy link
Member

Перенес всю статику (js, img, css) в папку assets. Это удобно тем, что:

  • в assets/vendor Composer ставит сторонние компоненты: jquery, bootstrap, bootstrap-notify; так проще обновлять крупные внешние зависимости, нежели скачивать каждый раз скрипты с сайта
  • обновление используемой версии jquery - это изменение версии в файле composer.json + composer.php update
  • на всю папку assets можно повесить правила кэширования с помощью .htaccess
  • в дальнейшем, наверное, sacy будет класть в эту папку сжатые+минифицированные версии статических файлов (Минификация + сжатие css, js #616)

Пока что в vendor не переехали, потому что не ставятся с помощью Composer:

  • jquery.flot
  • jquery.mousewheel
  • mousetrap
  • bootstrap-datepicker
  • bootstrap-select
  • excanvas
  • jquery.autocomplete

Эти скрипты лежат, как и раньше, в assets/js.

Возможно, нужно добавить в корневой .htaccess RewriteRule а-ля img/ -> assets/img, если на наши картинки где-то есть внешние ссылки. Все, что было в шаблонах, я поправил.

@grandsbor
Copy link
Member

А смёрджи сам в мастер? :)

madfriend added a commit that referenced this pull request Mar 30, 2015
Статика (js, img, css) переехала в assets
@madfriend madfriend merged commit 042087c into master Mar 30, 2015
@madfriend
Copy link
Member Author

Смерджил, можно выкатить?

@madfriend
Copy link
Member Author

Еще кстати подумал. Наверное, не хорошо в репозитории держать assets/vendor - оно же руками должно ставиться, как и корневой vendor..

@grandsbor
Copy link
Member

Если уверен, выкатывай :)
Да, нехорошо

@madfriend
Copy link
Member Author

Убрал assets/vendor из репозитория. Сейчас выкачу

@madfriend
Copy link
Member Author

Пулл приехал в продакшн. Пришлось, правда, сделать неочевидное - отказаться от использования components/bootstrap в Composer (то есть, не скачивать сам бутстрап), а заменить это своим пакетом opencorpora/bootstrap (живет здесь - https://github.com/opencorpora/bootstrap). Сделано это потому, что наш бутстрап был собран кастомно, с другими цветами, и вообще, похоже, версии 2.1.0 (в отличии от 2.3.2 в components)

@madfriend madfriend deleted the new-assets-location branch April 1, 2015 22:17
@akmetainfo
Copy link
Contributor

У меня на тестовом окружении тоже собирается нормально.

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

Successfully merging this pull request may close these issues.

None yet

3 participants