Skip to content

Помощь друга#16

Merged
keksobot merged 8 commits into
htmlacademy-javascript:masterfrom
Alexandra0107:module9-task2
May 25, 2026
Merged

Помощь друга#16
keksobot merged 8 commits into
htmlacademy-javascript:masterfrom
Alexandra0107:module9-task2

Conversation

@Alexandra0107

@Alexandra0107 Alexandra0107 commented May 13, 2026

Copy link
Copy Markdown
Contributor

@keksobot keksobot changed the title Module9 task2 Помощь друга May 13, 2026
@keksobot

Copy link
Copy Markdown
Contributor

♻️ Я собрал ваш пулреквест. Посмотреть можно здесь.

keksobot pushed a commit that referenced this pull request May 13, 2026
@keksobot

Copy link
Copy Markdown
Contributor

♻️ Я собрал ваш пулреквест. Посмотреть можно здесь.

keksobot pushed a commit that referenced this pull request May 25, 2026
@keksobot

Copy link
Copy Markdown
Contributor

♻️ Я собрал ваш пулреквест. Посмотреть можно здесь.

keksobot pushed a commit that referenced this pull request May 25, 2026
Comment thread js/data.js
@@ -21,5 +21,61 @@ const MESSAGES = [
'Лица у людей на фотке перекошены, как будто их избивают. Как можно было поймать такой неудачный момент?!'
];

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Обратите внимание, что при переключении фильтра, уровень эффекта должен сразу сбрасываться до начального состояния, т. е. логика по определению уровня насыщенности должна срабатывать не только при «перемещении» слайдера, но и при переключении фильтров.

смотри, у тебя не сбрасывается как по описанию задания
надо что бы при загрузке фотки новой было значение по умолчанию
также при переключении эффе4кта - ставилось на макс значение изначально

Comment thread js/dom.js
return bigPicture?.querySelector('.big-picture__img img') || null;
};
export {getBigPicture,getBody, getFileInput,getOverlay,getPreviewImage,getCancelButton,getHashtagsInput,getDescriptionInput,getUploadForm,getCommentsContainer,getCommentCountElement,getCommentsLoaderElement,getShownCommentCountElement,getTotalCommentCountElement,getLikesCountElement,getCaptionElement,getImageElement};
export {getBigPicture,getBody, getFileInput,getOverlay,getPreviewImage,getCancelButton,getHashtagsInput,getDescriptionInput,getUploadForm,getCommentsContainer,getCommentCountElement,getCommentsLoaderElement,getShownCommentCountElement,getTotalCommentCountElement,getLikesCountElement,getCaptionElement,getImageElement,getScaleSmallerButton,getScaleBiggerButton,getScaleValueField,getEffectsList,getEffectLevelSlider,getEffectLevelValue,getEffectLevelContainer};

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

список экспортов тут вполне моно столбиком

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

поправила

Comment thread js/image-processor.js
// Скрываем контейнер слайдера по умолчанию
container.classList.add('hidden');

noUiSlider.create(slider, {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

функция initEffectSlider выходит большой, попробуй то что внутри типа создания слайдера и добавления слушателя например в отдельные функции

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

поправила

@keksobot keksobot merged commit 2a94bcd into htmlacademy-javascript:master May 25, 2026
1 check passed
keksobot pushed a commit that referenced this pull request Jun 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants