#8 JS11(foop): 6?

Open
opened 1 year ago by gitgod · 0 comments
gitgod commented 1 year ago

http://gitlab.a-level.com.ua/eugeniy.ho/javascript_practice/src/master/hw10/loginForm.js#L35, http://gitlab.a-level.com.ua/eugeniy.ho/javascript_practice/src/master/hw10/loginForm.js#L22 - onOpenChange задается снаружи (по аналогии с onChange), поэтому надо еще проверять его на наличие.

http://gitlab.a-level.com.ua/eugeniy.ho/javascript_practice/src/master/hw10/loginFormConstructor.js#L38-L62 - тут должен быть пароль (new Password), запуск onLogin по логину, getLogin (который есть под названием getValue), getPassword (который просто берет пароль из объекта Password)

http://gitlab.a-level.com.ua/eugeniy.ho/javascript_practice/src/master/hw10/loginFormConstructor.js#L68-L84 - это должно быть внутри конструктора;

http://gitlab.a-level.com.ua/eugeniy.ho/javascript_practice/src/master/hw10/passwordVerify.js#L65

не надо использовать querySelector, если есть переменная с этим дом-элементом; не надо заносить в Password кнопку, ведь пароль - это просто пароль;

http://gitlab.a-level.com.ua/eugeniy.ho/javascript_practice/src/master/hw10/loginForm.js#L35, http://gitlab.a-level.com.ua/eugeniy.ho/javascript_practice/src/master/hw10/loginForm.js#L22 - `onOpenChange` задается снаружи (по аналогии с `onChange`), поэтому надо еще проверять его на наличие. http://gitlab.a-level.com.ua/eugeniy.ho/javascript_practice/src/master/hw10/loginFormConstructor.js#L38-L62 - тут должен быть пароль (`new Password`), запуск onLogin по логину, getLogin (который есть под названием getValue), getPassword (который просто берет пароль из объекта Password) http://gitlab.a-level.com.ua/eugeniy.ho/javascript_practice/src/master/hw10/loginFormConstructor.js#L68-L84 - это должно быть внутри конструктора; http://gitlab.a-level.com.ua/eugeniy.ho/javascript_practice/src/master/hw10/passwordVerify.js#L65 не надо использовать `querySelector`, если есть переменная с этим дом-элементом; не надо заносить в `Password` кнопку, ведь пароль - это просто пароль;
Sign in to join this conversation.
No Label
No Milestone
No assignee
1 Participants
Loading...
Cancel
Save
There is no content yet.