Sahar12j14 / lw4

0 stars 0 forks source link

Замечания по lw4 #1

Open ivklgn opened 3 years ago

ivklgn commented 3 years ago

в js файле убито форматирование

ivklgn commented 3 years ago
  element.style['boxShadow'] = 'inset 0 -1px 0 red';

вместо применения стилей css - создайте класс и управляйте добавлением/удалением:

// add
element.classList.add("my-class");

// delete
element.classList.remove("my-class");
ivklgn commented 3 years ago

fieldValidationName -> nameValidation

ValidationName - не в camelCase (с маленькой буквы)

ivklgn commented 3 years ago
    let name = document.querySelector('#name');
    let lastName = document.querySelector('#last-name');
    let email = document.querySelector('#email');
    let password = document.querySelector('#password');

const, почитайте в чем разнича

ivklgn commented 3 years ago
    if(fieldValidationName(name) === 0) {
    document.getElementById('form-registration__item__error-name__empty').style.visibility = "hidden";
    document.getElementById('form-registration__item__error-name__input').style.visibility = "hidden";
    }
    else if(fieldValidationName(name) === 1) {
      document.getElementById('form-registration__item__error-name__empty').style.visibility = "hidden";
      document.getElementById('form-registration__item__error-name__input').style.visibility = "visible";
    }
    else{
      document.getElementById('form-registration__item__error-name__empty').style.visibility = "visible"; 
      document.getElementById('form-registration__item__error-name__input').style.visibility = "hidden";     
    }

перепишите функцию fieldValidationName таким образом, чтобы она возвращала не код ошибки а текст или объект ошибки:

return {
  errCode: -1,
  errMessage: 'Поле обязательно для заполнения',
};

обойдитесь вместо 3 else-if конструкций одним if/else.

код связанный с паролем и email - то же самое