search

Rqik / Hotel-Toxin

1 stars 1 forks source link

Scripts #54

Closed AkhmadBabaev closed 3 years ago

AkhmadBabaev commented 3 years ago

изображение

Те кто сидит на линуксе и маках работают с LF, а так же большинство программистов на винде так же использует LF, так что ты можешь создать проблемы если твой редактор кода будет автоматически преобразовывать концы строк в CRLF, это всё будет отображаться в git diff, что значительно усложнит ревью.

AkhmadBabaev commented 3 years ago

https://github.com/Rqik/FSD/blob/eb2254f20a718030ee7f9e3735ca22c9c2c37e75/src/components/button-like/button-like.js#L9

Посмотри в сторону данного пакета, удобная штука для привязки методов к классу.

AkhmadBabaev commented 3 years ago

https://github.com/Rqik/FSD/blob/eb2254f20a718030ee7f9e3735ca22c9c2c37e75/src/components/button-like/button-like.js#L20-L24

Как я понял уже читая код функции что e это event, но e так же может означать element. Дай более описательное имя параметру чтобы снизить нагрузку на мозг читателя )

AkhmadBabaev commented 3 years ago

Файлы экспортирующие класс должны так носить имя класса который они экспортируют, вместо button-like.js будет ButtonLike.js и так далее.

AkhmadBabaev commented 3 years ago

https://github.com/Rqik/FSD/blob/434fa03034fe3069f782d6b5aa8c2f2c6fe656f3/src/components/drop-down/drop-down.js#L168

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