Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

Two fixes #47

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Two fixes #47

wants to merge 2 commits into from

Conversation

petersim-nz
Copy link

@petersim-nz petersim-nz commented Feb 20, 2018

Firstly, my apologies that my editor (VS Code) took it upon itself to remove trailing spaces in levels.js:

#42
The only substantive change is in line 2329 which fixes the problem with the left and right arrows not lining up in the popup at the end of each level.

In about.js there was a problem for me with the bottom ray going through the last line of text. I've moved it down so that it is now between the two ships.

@@ -2325,8 +2325,8 @@ function drawLevelFinal()
drawRoundedRectangle(midx-120*(width/1220), midy, 100*(width/1220), 60*(width/1220), colorCheck, 4);
gameArena.shadowBlur = 0;
gameArena.font = "50px Zorque";
gameArena.fillStyle = "#E6FFFF";
gameArena.fillText("<",midx-143+70,midy-45+92);
gameArena.fillStyle = "#E6FFFF";
Copy link
Owner

Choose a reason for hiding this comment

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

You changed the indentation here. Kindly revert it back

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants