Порядок в модификаторах

[awinogradov]

Сейчас мой код выглядит так:

{
     block : 'github-ribbon',
     mods : { theme : 'darkblue', right : true },
     url : '//github.com/verybigman/bem-content'
},
...
{
     block : 'github-button',
     mods : { type : 'star', count : true },
     user : 'verybigman',
     repo : 'bem-content'
},
{
     block : 'twitter',
     mods : { widget : 'share' },
     url : 'https://github.com/verybigman/bem-content',
     text : 'Отличный набор блоков для верстки статики в терминах БЭМ',
     via : 'verybigman',
     related : 'bem_ru',
     hashtags : 'bem,b_',
     size : 'small',
     lang : 'ru'
}

А мне бы очень хотелось как-то так:

{
     block : 'github-ribbon',
     mods : { theme : 'darkblue', position: 'right' },
     url : '//github.com/verybigman/bem-content'
},
...
{
     block : 'github-button',
     mods : { type : 'star', size : 's', count : true },
     user : 'verybigman',
     repo : 'bem-content'
},
{
     block : 'twitter',
     mods : { widget : 'share', size : 's' },
     url : 'https://github.com/verybigman/bem-content',
     text : 'Отличный набор блоков для верстки статики в терминах БЭМ',
     via : 'verybigman',
     related : 'bem_ru',
     hashtags : 'bem,b_',
     lang : 'ru'
}

Чтобы поддержать нормально размерность. А то размеры задаются по разному. Плюс, мне кажется, было бы лучше сократить количество модификаторов без значения за счет его задания. Как в примере с position для github-ribbon. Это касается всех блоков.

[voischev]

Все по существу! :+1: Я все ждал когда наберется некоторое количество блоков что бы понять какие параметры можно причесать. Хочется добавить еще как минимум несколько виджетов https://github.com/voischev/bem-social/issues/42 https://github.com/voischev/bem-social/issues/39 https://github.com/voischev/bem-social/issues/38 https://github.com/voischev/bem-social/issues/37 https://github.com/voischev/bem-social/issues/36 https://github.com/voischev/bem-social/issues/35 https://github.com/voischev/bem-social/issues/34 https://github.com/voischev/bem-social/issues/33

Тогда и буду наводить порядок. Поддерживаешь идею или порядок нужно было сразу наводить?

[qfox]

@voischev в опенсорсе вангование стоит сверхдорого, как сейчас делаете — дешевле. так что, лучше набрать варианты, а уже потом думать что причесывать.

[voischev]

Склоняюсь к этому же, но давайте узнаем еще мнения) Например от @tadatuta @ilyar

[awinogradov]

Сложно сказать когда лучше. Потом код может начать расти и причесывать будет сложнее. Тут ванговать и не надо. Можно просто поюзать практики из проектов рядом. Я бы сразу сделал. Вроде работы совсем не много.

[qfox]

Как раз тот случай, когда пора ;-)

[voischev]

@verybigman пока планирую к этой версии навести порядок https://github.com/voischev/bem-social/milestones/v0.4.0

[tadatuta]

предложение @verybigman поддерживаю.

а ответа на вопрос, когда начинать причесывать у меня нет. очевидно, что чем раньше — тем лучше, но с другой стороны очевидно, что пока нет достаточного количества кейсов, причесывать нечего.

[voischev]

В общем ничего не подсказали) Буду делать как и планировал.

Вообще планирую к версии 1.0.0 все прилезать, набрать приличное количество виджетов и некоторых других плюшек. Сейчас пользоваться уже можно четко фиксируя версию в проекте.

[qfox]

@voischev Да пора уже ;-)

[ilyar]

:+1: очень может быть что лучшим вариантом буде причесывать перед очередным релизом, когда можно будет обозреть все уже реализованные блоки в текущем релизе.