Conversation
There was a problem hiding this comment.
Pull request overview
Updates welcome/instructions content across multiple games (closes #71) to make gameplay guidance clearer and adjust a few UI details.
Changes:
- Refines “How to Play” copy in several game
interface.htmlfragments. - Adds/adjusts instruction visuals and related CSS (e.g., Fast Piggie target image; Otter Stop feedback images).
- Tweaks Sound Sweep difficulty level configuration by adding an additional hardest level.
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
| app/games/sound-sweep/interface.html | Minor instruction text and markup formatting updates |
| app/games/sound-sweep/game.js | Extends LEVELS to include an additional hardest speed level |
| app/games/otter-stop/style.css | Adds layout/styles for instruction feedback images |
| app/games/otter-stop/interface.html | Updates instructions and adds feedback image examples |
| app/games/orbit-sprite-memory/interface.html | Rewrites instruction wording for clarity |
| app/games/object-track/interface.html | Rewrites instruction wording for clarity |
| app/games/high-speed-memory/interface.html | Rewrites instruction wording and collapses some markup |
| app/games/field-of-view/interface.html | Rewrites instruction wording and minor markup formatting |
| app/games/fast-piggie/style.css | Adds styling for a target image in the instructions panel |
| app/games/fast-piggie/interface.html | Adds target image and updates instruction wording |
| app/games/directional-processing/interface.html | Updates instruction wording and minor markup formatting |
| @@ -19,8 +19,20 @@ <h3 class="os-instructions__subheading">How to Play</h3> | |||
| <strong>must</strong> respond or it counts as a miss! | |||
| </li> | |||
| <li>When the sneaky <strong>fish</strong> appears, do <strong>nothing</strong> — freeze!</li> | |||
| <li>The round ends when you see the success or failure images and hear the corresponding sound.</li> | |||
| <li>The game speeds up as your accuracy improves. Good luck!</li> | |||
There was a problem hiding this comment.
The welcome instructions now refer to a "sneaky fish", but the game logic/UI still labels the no-go stimulus as a "No-go otter" (see otter-stop/index.js setting _stimulusImg.alt = 'No-go otter') and the in-game reminder text still says "no-go otter". This mismatch will confuse players—please align the instructions with the actual stimulus naming (either update the text back to otter/no-go, or update the game labeling/assets to fish consistently).
There was a problem hiding this comment.
The no go image is a fish. All references to the no go image should be updated to be a fish.
There was a problem hiding this comment.
Implemented in commit 83763ff.
Updated all no-go references in Otter Stop to fish:
- stimulus alt text is now "No-go fish"
- reminder text now says no-go fish
- manifest description now says no-go fish
- test expectation updated accordingly
Screenshot: 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
@copilot apply changes based on the comments in this thread which are not already marked as resolved. |
Agent-Logs-Url: https://github.com/acrosman/BrainSpeedExercises/sessions/748e4726-9d91-4e75-896a-ce37a25e29a4 Co-authored-by: acrosman <2972053+acrosman@users.noreply.github.com>
Done in commit Applied all unresolved review-thread items:
Screenshot: |
Closes #71