Домашнє завдання 1 (HTML + CSS) - #16
Conversation
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 3 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit fd8cc4d. Configure here.
| setTimeout(() => { | ||
| document.getElementById("status").innerText = "Вітаю! Ви зібрали п'ятнашки!"; | ||
| }, 100); | ||
| } |
There was a problem hiding this comment.
Win message stays after moves
Medium Severity
After a win, #status is set to the congratulations text, but later valid moves that leave the board unsolved never clear it. Players can see a victory message while the puzzle is still in progress.
Reviewed by Cursor Bugbot for commit fd8cc4d. Configure here.
| if (checkWin()) { | ||
| setTimeout(() => { | ||
| document.getElementById("status").innerText = "Вітаю! Ви зібрали п'ятнашки!"; | ||
| }, 100); |
There was a problem hiding this comment.
Delayed win after shuffle
Medium Severity
The win message uses a 100ms setTimeout that is never cancelled. If the player hits «Перемішати» right after winning, restartGame clears #status but the pending timer can still show a victory message on the new shuffled board.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit fd8cc4d. Configure here.
| if (event.key === 'Enter') addItem(); | ||
| }); | ||
|
|
||
| loadState(); |
There was a problem hiding this comment.
Script crashes without markup
Medium Severity
New shopping-list logic calls addButton.addEventListener and render() using nodes from fixed selectors and ids without checking they exist. Any page that loads script.js without the full markup throws before the list can run.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit fd8cc4d. Configure here.
1 participant
Note
Low Risk
Static front-end homework with no backend or auth; main caveat is reliance on an external WebDataRocks CDN and orphaned list assets not referenced by index.html.
Overview
Replaces the minimal Hello World
index.htmlwith a self-contained Ukrainian «П'ятнашки» (15-puzzle) game: board state, solvable shuffle, win check, shuffle button, and tile moves via WebDataRocks grid (cellclick+updateData), with styles inlined in the page and scripts loaded from the WebDataRocks CDN.main.cssis rewritten from a few demo rules into a full layout/theme for a two-column shopping UI (CSS variables, responsive.container, list items, button tooltips,.student-badge, print styles). Newscript.jsimplements a buy-list app (add/edit/delete, quantities, bought toggle, stats columns,localStoragepersistence). Newstyle.cssprovides a second, parallel stylesheet for the same shopping-list pattern (badge, print rules).Note: the current
index.htmldoes not linkmain.css,script.js, orstyle.css—only the puzzle page is wired as the entry document; the list CSS/JS look like separate homework pieces unless another HTML file is added later.Reviewed by Cursor Bugbot for commit fd8cc4d. Bugbot is set up for automated code reviews on this repo. Configure here.