Надо подкачаться #12
Надо подкачаться #12
Conversation
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
js/api.js
Outdated
| @@ -0,0 +1,35 @@ | |||
| //import {showErrorGet} from './response.js'; | |||
js/api.js
Outdated
| } | ||
| throw new Error(`${response.status} - ${response.statusText}`); | ||
| }) | ||
| .then((response) => response.json()) |
There was a problem hiding this comment.
надо объединить с предыдущей секцией. Обсуждали, что код в ней синхронный
js/api.js
Outdated
| .then((response) => { | ||
| if (response.ok) { | ||
| onSuccess(); | ||
| } else { |
There was a problem hiding this comment.
else очень неудобный элемент для чтения и предполагает, что две ветки объединяются в для дальнейшего вычисления. А если вычисления нет - завершаем работу:
if (response.ok) {
onSuccess();
return;
}
onError();
js/img-form-uploader.js
Outdated
| isCommentValid(inputText.value); | ||
| }; | ||
|
|
||
| /* |
There was a problem hiding this comment.
надо убрать код, если он не используется
js/img-form-uploader.js
Outdated
| inputText.addEventListener('input', handleTextChange); | ||
|
|
||
| imgUploadForm.addEventListener('submit', validateFormSubmit); | ||
| //imgUploadForm.addEventListener('submit', validateFormSubmit); |
There was a problem hiding this comment.
Управляем кнопкой через слушателя изменений на форме
form.addEventListener('input', () => {
const isValid = pristine.validate();
submitButton.disabled = !isValid;
});
submitButton.disabled = !pristine.validate();
Строки убираем - они не функциональны:
const handleTagChange = () => {
isHashtagsValid(inputHashtag.value);
};
const handleTextChange = () => {
isCommentValid(inputText.value);
};
inputHashtag.addEventListener('input', handleTagChange);
inputText.addEventListener('input', handleTextChange);
js/main.js
Outdated
| import './img-form-uploader.js'; | ||
| renderPhoto(); | ||
| import { getData } from './api.js'; | ||
| //import {showAlert} from './util.js'; |
| @@ -0,0 +1,87 @@ | |||
| import { isEscapeKey } from './util.js'; | |||
| //import {showAlert} from './util.js'; | |||
|
|
|||
There was a problem hiding this comment.
Название файла не соответствует содержимому - больше подходит alert или notifications
js/response.js
Outdated
|
|
||
| }; | ||
|
|
||
| export const showErrorSend = () => { |
There was a problem hiding this comment.
процесс обычно с ing окончанием:
showErrorGetting
showErrorSending
Полезно проверить со словарем. Сейчас название функции переводится как "показать ошибку получить"
js/response.js
Outdated
|
|
||
| }; | ||
|
|
||
| const TIMEOUT = 5000; |
There was a problem hiding this comment.
Константы модуля ожидаются после импортов
| const errorFragment = document.createDocumentFragment(); | ||
| const errorElement = errorTemplateSend.cloneNode(true); | ||
|
|
||
| errorFragment.append(errorElement); |
There was a problem hiding this comment.
Большая часть функции повторяется в showSuccess - будет удобнее реализовать одну функцию для навешивания событий и передавать ей соответсвующий темплейт
js/util.js
Outdated
| }; | ||
|
|
||
| export {getRandomNumber, getRandomItem, isEscapeKey, selectWordByCount}; | ||
| /*const showAlert = (message) => { |
3 participants
🎓 Надо подкачаться
💥 https://htmlacademy-javascript.github.io/2607777-kekstagram-2/12/