Skip to content

добавила таймер pomodoro#8

Open
Threed95 wants to merge 8 commits intoCodegirlSchool:mainfrom
Threed95:main
Open

добавила таймер pomodoro#8
Threed95 wants to merge 8 commits intoCodegirlSchool:mainfrom
Threed95:main

Conversation

@Threed95
Copy link
Copy Markdown

No description provided.

script.js Outdated

function switchTime() {
isBreak = !isBreak;
timeLeft = isBreak ? 25 * 60 : 5 * 60;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

не логично. Если режим break - то устанавливается время 25 . Лучше ввести переменную mode , сначала она "pomodoro", а потом перезаписывать на "break". C boolean часто получается путаница

pomodoroButton.classList.remove('active');
buttonBreak.classList.add('active');
updateTimer();
})
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

А ещё должно быть нажатие на кнопку pomodoro, А она не работает)


})

buttonBreak.addEventListener("click", () => {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Даша, при нажатии на кнопки режимов , таймер должен останавливаться

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.

2 participants