search

performanceArtist / ui-website

Second FSD task
https://performanceartist.github.io/ui/
0 stars 0 forks source link

Именование в js #13

Closed viktoriya-ch closed 5 years ago

viktoriya-ch commented 5 years ago

Имя переменной должно полно и точно описывать сущность и ее предназначение (https://github.com/fullstack-development/front-end-best-practices/blob/master/JS/naming.md)

Не стоит именовать переменные x, y, s, r, sa. ctx, perc... - из их имен совсем не сразу понятно, что они в себе содержат. (например, блоки pie-chart и ripple)

viktoriya-ch commented 5 years ago

В ripple.js остались переменные x,y

viktoriya-ch commented 5 years ago

А gh-pages ты не обновлял?

performanceArtist commented 5 years ago

Нет ещё - как завтра поправлю страницу с ui, добавлю.

viktoriya-ch commented 5 years ago

Просто в локальной версии ripple-эффект на кнопках на данный момент не работает

performanceArtist commented 5 years ago

Да, там опечатка была в имени модификатора - исправил.

viktoriya-ch commented 5 years ago

Есть еще переменные с некорректными именами:

Datepicker:

Снимок экрана в 2019-08-05 16-17-42

selected переводится как "выбранный". Переменные принято именовать существительными.

MessageForm. То же самое:

const valid - валидный.

Order

const current- текущий. const next - следующий