Skip to content

Better mobile compatibility by using different pointer API when supported - #102

Open
mclaire-dev wants to merge 3 commits into
tholman:masterfrom
mclaire-dev:mobile-tweak-pointer
Open

Better mobile compatibility by using different pointer API when supported#102
mclaire-dev wants to merge 3 commits into
tholman:masterfrom
mclaire-dev:mobile-tweak-pointer

Conversation

@mclaire-dev

Copy link
Copy Markdown

Addresses #48

This is my first pull request and I did use Format Document in VSCode(sorry) so I apologize if it's not the best pull request.
It works in my testing though.

@mclaire-dev
mclaire-dev marked this pull request as draft August 15, 2026 14:47
Comment thread src/antsCursor.js Outdated
}
window.removeEventListener("resize", onWindowResize);
};
window.addEventListener("resize", onWindowResize);

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Still want to be removing here right?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I have been basing this off of the rainbowCursor file and in the rainbowCursor there is an addEventListener method in destroy()

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Ahh, haha, probably my mistake in that one spreading out 👯

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Coming along well though, open out of draft when you're ready. Don't worry too much about formatting, I think I should run a script after this to balance it all out across all files :)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

hi thats awesome. ill start working on it again starting tomorrow.
id like to add that the compatibility check is because pointermove and pointerenter were added 6 years ago in 2020 and I'm not sure how far back you want this essential function of the library to go in compatibility.

@mclaire-dev
mclaire-dev marked this pull request as ready for review August 16, 2026 21:30
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.

2 participants