Skip to content

6214 Зимина Е.А. Лаб.4#421

Closed
Z1mina wants to merge 8 commits intoitsecd:mainfrom
Z1mina:lab4
Closed

6214 Зимина Е.А. Лаб.4#421
Z1mina wants to merge 8 commits intoitsecd:mainfrom
Z1mina:lab4

Conversation

@Z1mina
Copy link
Copy Markdown

@Z1mina Z1mina commented Dec 17, 2025

В папку lab, кроме предыдущих лабораторных работ, добавлен калькулятор и лабораторная работа 3 с добавлением ссылок на прошлые работы.

Comment thread lab/calculator.js Outdated
Comment on lines +2 to +4
let num1 = Number(document.getElementById("num1").value);
let num2 = Number(document.getElementById("num2").value);
let operation = document.getElementById("operation").value;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

А если какой-то элемент не найден?

Comment thread lab/calculator.html
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

немного нет БЭМ

@Z1mina Z1mina requested a review from AvtoBBus December 17, 2025 11:20
@AvtoBBus
Copy link
Copy Markdown
Collaborator

Принято

@AvtoBBus AvtoBBus closed this Dec 17, 2025
@github-actions github-actions bot added Done and removed In progress labels Dec 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants