Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convert to WebGL #49

Closed
wants to merge 4 commits into from
Closed

Convert to WebGL #49

wants to merge 4 commits into from

Conversation

Rudxain
Copy link
Owner

@Rudxain Rudxain commented Mar 8, 2024

@Rudxain Rudxain added the perf Performance problem label Mar 8, 2024
@@ -223,13 +223,16 @@
And set font-size accordingly
*/
const resize = () => {
// to-do: detect if the final dimensions

Check failure

Code scanning / ESLint

Disallow specified warning terms in comments Error

Unexpected 'to-do' comment: 'to-do: detect if the final dimensions'.
const { clientWidth, clientHeight } = body
canv.style.width = clientWidth + "px"
canv.style.height = clientHeight + "px"
const scale = devicePixelRatio
const save = ctx.getImageData(0, 0, w, h)
w = canv.width = clientWidth * scale >>> 0
h = canv.height = clientHeight * scale >>> 0
// to-do: call only if the canvas grows AND is white

Check failure

Code scanning / ESLint

Disallow specified warning terms in comments Error

Unexpected 'to-do' comment: 'to-do: call only if the canvas grows AND...'.
@Rudxain
Copy link
Owner Author

Rudxain commented Mar 9, 2024

This would make the code less maintainable. I'll RIIR before doing this

@Rudxain Rudxain closed this Mar 9, 2024
@Rudxain Rudxain reopened this Jul 17, 2024
@Rudxain Rudxain closed this Jul 17, 2024
@Rudxain Rudxain deleted the gl branch July 17, 2024 02:51
@Rudxain Rudxain mentioned this pull request Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
perf Performance problem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant