Skip to content

Conversation

@opax7
Copy link
Contributor

@opax7 opax7 commented Aug 8, 2025

Added starfield background for the splash page.

@opax7 opax7 requested a review from zicklag August 8, 2025 01:18
Copy link
Member

@zicklag zicklag left a comment

Choose a reason for hiding this comment

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

I left a couple hints on the CI errors.

}

function setup() {
rafId > 0 && cancelAnimationFrame(rafId);
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
rafId > 0 && cancelAnimationFrame(rafId);
if (rafId > 0) cancelAnimationFrame(rafId);

console.warn("Canvas element is not available.");
}

let container = document.querySelector("#starfield") as HTMLElement;
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
let container = document.querySelector("#starfield") as HTMLElement;
const container = document.querySelector("#starfield") as HTMLElement;

package.json Outdated
"fix:prettier": "prettier -w ."
},
"dependencies": {
"@astrojs/react": "^4.3.0",
Copy link
Member

Choose a reason for hiding this comment

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

This probably isn't needed anymore.

@opax7 opax7 merged commit 1781aa3 into main Aug 8, 2025
4 checks passed
@opax7 opax7 deleted the starfield-bg branch August 8, 2025 12:24
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.

3 participants