Conversation
| <script src="./js/EventsModule.js"></script> | ||
| <script src ="./js/ViewModule.js"></script> | ||
| <script src="./js/ApplicationModule.js"></script> | ||
| <script src="./js/scrypt.js"></script> |
| <select class="filter-blocks" name="Популярные хэштеги" title="Популярные хэштеги" size="1" id = "selectFilter"> | ||
| <option disabled> Выберите популярный хэштег</option> | ||
| <option value =""></option> | ||
| <option>Закат</option> |
There was a problem hiding this comment.
Лишние option почистить если не используются
Здесь только дефолт.
| </header> | ||
| <!--block of filters--> | ||
| <div class="filters"> | ||
| <div class="filter-blocks"><label>Логин: <input class="filter-input" type="text" name="firstname" id="login"> |
There was a problem hiding this comment.
name="firstname" в инпутах либо убрать совсем, либо использовать форму
| </label></div> | ||
| <div class="filter-blocks"><label>Дата: <input class="filter-input" type="text" name="firstname" id="date"> | ||
| </label></div> | ||
| <select class="filter-blocks" name="Популярные хэштеги" title="Популярные хэштеги" size="1" id = "selectFilter"> |
There was a problem hiding this comment.
name="Популярные хэштеги" -
name - технический атрибут со фполне четкими вариантами использования, никакой кирилицы!!! Называем аккуратно как и переменные.
| getPhotoPosts: function (skip, top, filterConfing) { | ||
| let result = ApplicationModule.photoPosts; | ||
|
|
||
| if(!result) |
| let img = document.createElement('img'); | ||
| img.src = reader.result; | ||
| gallery.appendChild(img); | ||
| photoLnk = reader.result; |
There was a problem hiding this comment.
Side effect?
Не лучший вариант, сильно и не очень ожидаемо менять контекст.
| }, | ||
|
|
||
| editPost: function(e) { | ||
| inputDescriptionEdit.value = ''; |
There was a problem hiding this comment.
А точно одну форму ни как не переюзать?
Ну вообще я знаю чтоответ нет неправильный, но коль уж времени мало на это можно забить.
| if (index !== -1) { | ||
| if (ApplicationModule.users[index].password === pass) { | ||
| ViewModule.loginView(name); | ||
| document.getElementById('exitBtn').addEventListener('click', EventsModule.unAuthorization); |
There was a problem hiding this comment.
Просто заметка: Динамически добавлять листенеры плохая идея в реальной ситуации, если мы не уверены что онибудут откреплены в нужный момент.
| initializeLikeButtons: function(){ | ||
| let likeButtons = document.querySelectorAll('.likePost'); | ||
| for(let i = 0; i < likeButtons.length; i++){ | ||
| likeButtons[i].addEventListener('click', EventsModule.pressLike); |
There was a problem hiding this comment.
No No No!!!!
ни в коем разе не добавлять листенеры на элементы постов напрямую
Плиз читай "делегация событий".
| let editButtons = document.querySelectorAll('.editPost'); | ||
| let deleteButtons = document.querySelectorAll('.deletePost'); | ||
| for(let i = 0; i < editButtons.length; i++){ | ||
| editButtons[i].addEventListener('click', EventsModule.editPost); |
2 participants
No description provided.