bem / bem-core

BEM Core Library
https://ru.bem.info/technologies/classic/i-bem/
Other
276 stars 95 forks source link

page.bemhtml head after styles #458

Closed 4ok closed 10 years ago

4ok commented 10 years ago

Сейчас в page.bemhtml есть код отвечающий за отрисовку данных в элементе head:

{ elem : 'head', content : [ { tag : 'meta', attrs : { charset : 'utf-8' } }, { tag : 'title', content : ctx.title }, { block : 'ua' }, ctx.styles, ctx.head, ctx.favicon? { elem : 'favicon', url : ctx.favicon } : '' ] }

тут ctx.styles стоят перед ctx.head и если я в массив head добавляю новый элемент < base url="..." /> то стили игнорируют базовый урл и соответственно не подгружаются, я думаю ctx.head и ctx.styles надо поменять тут местами

4ok commented 10 years ago

Может что-то непонятно из описанного?

narqo commented 10 years ago

Можно пример BEMJSON с которым не работает и пример того, что должно получиться? Пока не очень понятно

tadatuta commented 10 years ago

@narqo если я правильно понял, то речь о том, что при

head: [
        { tag: 'base', attrs: { href: 'http://mysite.ru' } }
    ],
    styles: [
        { elem: 'css', url: '_index.css' }
    ]

путь до _index.css проигнорирует base.

tadatuta commented 10 years ago

https://github.com/bem/bem-core/pull/465