mailru / fest

javascript templates
Other
128 stars 28 forks source link

Предлагаю замену select’у #63

Closed eprev closed 11 years ago

eprev commented 11 years ago

Что если убрать select атрибут, там где нужны выражения в значениях у фестовых тегов, заменить его на { expr } (как для XML тегов):

<f:template>
    <f:get select="json.block" />
    vs.
    <f:get name="{json.block}" />
</f:template>
<f:template>
    <f:element select="json.block" />
    vs.
    <f:element name="{json.block}" />
</f:template>

Получим большую гибкость, например динамические имена атрибутов:

<f:template>
    <div>
        <f:attr name="data-{json.blockName}" value="{json.blockValue}" />
    </div>
</f:template>

Для предыдущего примера, необходимо было бы заводить select-name и select-value атрибуты.

Я постоянно вижу как те, кто только начал использовать фест, пытаются использовать <fest:get name={params.block}/> вместо select.

latrekc commented 11 years ago

+1

RubaXa commented 11 years ago

:+1:

P.S. Для тикетов нужна голосовалка!

AndrewSumin commented 11 years ago

Мы медленно но верно даем писать медленные шаблоны. Но я не против, звучит логично.

RubaXa commented 11 years ago

И где тут медленно? Мы просто убираем лишнюю запись.

P.S. Прошу заметить, что пока Антон не занялся оптимизацией fest, он был не таким уж и быстрым, только после оптимизации, результаты стали такими: https://github.com/RubaXa/AsyncTpl/tree/master/benchmark#benchmark И ничего, работало и работает морде, мы пока не в одном месте не замедлились.