global-var support for less #33
Conversation
| Thank you for your contribution. I like the feature that you propose and I'd like to include it. But I'd prefer not add hashes to compiled file names to keep the names clean. I understand that you do that do force file recompilation when global vars are changed. But I think it's a rare use case, and you can just remove all compiled CSS files to force recompilation. Could you remove the Also, I don't understand the purpose of If would be great if you could add tests for this new feature. However, I can add it myself. |
| Прошу прощения за русский, но так будет проще донести мысль :)
Проблема возникает на мультисайтовости: разные сайты генерируют разные конфиги с разными глобальными переменными. Сходу получилось придумать только вот такое решение. Теоретически можно добавить в static_precompiler.settings флаг, указывающий на использование нескольких сайтов, и разделять генерируемые css по SITE_ID.
Генерация зависимостей подавилась переменными в импорте, а ради них всё и затевалось. Если переменная из LESS_GLOBAL_VARS, то значение можно подставить, а вот как быть с остальными переменными — непонятно.
Тесты добавлю. |
| Может быть разнести разные сайты по разным папкам? Чтобы у каждого был свой STATIC_ROOT и своя статика? Еще вариант: если есть возможность отказаться от @import и просто подключить этот файл как отдельную таблицу стилей, то тогда в шаблоне можно сделать Либо сделать templatag для этого, который будет подключать нужные стили в зависимости от SITE_ID. В нём можно использовать функцию |
| Практически вся статика и стили идентичны и вычисляются на основе небольшого набора параметров, подключаемых в основной less-файл через Через link тоже не вариант, так как переменные не подцепятся к основному файлу. Может задавать в конфиге необязательную лямбду для генерации source_filename? Тогда можно будет гибко раскладывать генерируемые файлики по нужным папкам. |
| Даже не для source_filename, а для source_dir |
| Может тебе тогда сделать свой компилятор на базе |
| Рискну предложить версию без хэшей :) |
| Да, кстати, юниттесты по sass падают из-за отсутствия в репозитории следующих папок: |
| Я всё-таки считаю, что фунциональность, связанную с многосайтовостью, лучше вынести в кастомный компилятор. Насчёт тестов, эти папки вроде бы не требуются, тесты прогоняются автоматически и в данный момент не падают. |
| А чем мешает многосайтовость, если без параметра в конфиге она никак не проявляется? |
| Это дополнительный функционал, который нужно поддерживать (код, тесты, документация). При этом, мне кажется, что юзкейс достаточно редкий, чтобы вносить добавлять эту функцию в основной код. Именно для таких редких юзкейсов я и сделал возможность использовать кастомные компиляторы. |
andreyfedoseev force-pushed the master branch from 3409726 to 03736be Compare | I'm going to add support for |
3 participants
No description provided.