Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[HTML-гайд] Перенос атрибутов #22

Open
x1unix opened this issue Sep 1, 2017 · 6 comments
Open

[HTML-гайд] Перенос атрибутов #22

x1unix opened this issue Sep 1, 2017 · 6 comments
Assignees

Comments

@x1unix
Copy link

x1unix commented Sep 1, 2017

Иногда возникает ситуация, когда у HTML-элемента появляется большое количество аттрибутов, или значение у аттрибутов очень длинное и приходится переносить аттрибуты.

Этот пункт упускают очень многие style-гайды (гугловый в том числе)

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

<div
  id="myComponentItem32"
  class="my-component__item my-component__item--dashed"
  data-ng-click="loadSomething();"
  data-ng-class="{'text-center': isCentered}"
  title="Lorem Ipsum Dolor...."
>
  Lorem Ipsum Dolor Sit....
</div>

Пример выше часто используется в React-комьюнити

@x1unix x1unix changed the title [HTML] Перенос атрибутов [HTML-гайд] Перенос атрибутов Sep 1, 2017
@juwain juwain self-assigned this Sep 4, 2017
@installero
Copy link

Гугл предлагает ещё такой перенос:

<md-progress-circular md-mode="indeterminate" class="md-accent"
    ng-show="ctrl.loading" md-diameter="35">
</md-progress-circular>

Отступ 4 пробела — сомнителен, но в целом, идея не тратить по целой строке на каждый атрибут выглядит здравой.

@x1unix
Copy link
Author

x1unix commented Jun 3, 2018

@installero, можно ссылку на пункт?

@installero
Copy link

@x1unix
Copy link
Author

x1unix commented Jun 4, 2018

@installero, спасибо, сразу не заметил :)

Там представленны все три стиля переноса, включая представленный мной выше. Однако они все три - разные и соотсветственно тут надо выбирать один из них чтобы поддерживать единый стиль кода.

@installero
Copy link

installero commented Jun 4, 2018

Почему? Во многих стайл-гайдах допускается использование одного из нескольких вариантов. Например, в стандартном гайде отступ 2 пробела при вложенности в контейнер — опционален.

It is not necessary to indent every element

@nikolai-shabalin
Copy link
Contributor

Тут бы понять "много" это сколько? Поделитесь своими мыслями. Я остановился на цифре 5, но кажется, что и 4 много если атрибуты строковые. Например, src="...." или class="class1 class2 class3"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants