Size: a a a

2017 August 17

VN

Viktor Nikolaev in 5minreact
источник

VN

Viktor Nikolaev in 5minreact
tl;dr; Верстальщик просто надобавлял своих тегов. Это не knockout/angular/polymer компоненты
источник

Д

Дмитрий in 5minreact
А чем тебе плохо от этого?
источник

VN

Viktor Nikolaev in 5minreact
Да я вот и пытаюсь понять, плох ли такой подход.
Пока такие доводы у меня:

1. Нужно знать кастомные теги
2.  Подобные "классы" не выставишь на другие теги
3. Есть вероятность перехода на ангулар, и там придётся синхронизироваться между "компонентами" верстальщика и ангулар-компонентами
4. Ну и это как-то странно. Непонятно что будет, если в спецификацию попадут теги с таким же названием.
5. Да и само именование слишком генерализованное (ну тут к самому верстальщику претензии) - как видно из css, на тег columns навешан border. Как по мне, так это тоже самое что и всем div'aм бордеры поставить
источник

Д

Дмитрий in 5minreact
1. Иначе нужно знать кастомные классы
источник

Д

Дмитрий in 5minreact
2. Возможно это плюс. Для глобальной стилизации тэги. Я бы не хотел, чтобы мой класс button-green, навешали не на кнопку. Это будет тэг.
А Которые нужно переиспользовать, напр, 'red-color' - классы
источник

VN

Viktor Nikolaev in 5minreact
ну иметь даже css класс  c названием button-green уже моветон, не говоря о целом теге
источник

Д

Дмитрий in 5minreact
4. Согласен
источник

PH

Pavel Horoshilov in 5minreact
Что только не придумают, лишь бы по БЭМ не писать классы
источник

VN

Viktor Nikolaev in 5minreact
Я стараюсь не заикаться тут про БЭМ, это слишком сложно и вызывает взрывы
источник

Д

Дмитрий in 5minreact
5. Тэг columns - очень громко сказано, да. Но если попробовать защитить верстальщика - я думаю он знает что это за блок и этот блок отображён на разных страницах и должен быть везде одинаков с бордюром. Можно переименовать тэг, чтобы новеньким было понятно для чего он.
источник

Д

Дмитрий in 5minreact
Вообщем ответ на твой вопрос - п.4
источник

Д

Дмитрий in 5minreact
Viktor Nikolaev
ну иметь даже css класс  c названием button-green уже моветон, не говоря о целом теге
Окей, с кнопкой плохой пример. Заменим его на тэг "advertising"
источник

VN

Viktor Nikolaev in 5minreact
Да, спасибо :)
Пойду бороться
источник

Д

Дмитрий in 5minreact
А если использовать less, то в  button-green нет ничего страшного
источник

Д

Дмитрий in 5minreact
источник

Д

Дмитрий in 5minreact
Но подход интересный. Особо минусов я не вижу, чтобы обойти п.4 можно использовать префиксы к тэгам
источник

Д

Дмитрий in 5minreact
Я тут где-то недавно увидел нейминг классов на русском, вот это жесть. class="Красная-кнопка" 😂
источник

VN

Viktor Nikolaev in 5minreact
Подход интересный. Называется custom tags. Только в моём случае это очень урезанный кастом тагс)
https://stackoverflow.com/a/9845124/2118133
источник

Д

Дмитрий in 5minreact
Тайпскрипт ругается, каждый такой тег нужнo описать.
А у тэгов с дефисом вообще className реакт не парсит в class
источник