Skip to content

Conversation

@pubuzhixing8
Copy link
Contributor

This pull request will implement eraser trail animation effect mentioned in #247.

… to drawing laser effect

This pull request will implement eraser trail animation effect mentioned in #247
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Sep 11, 2025

Deploying drawnix with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3c10174
Status: ✅  Deploy successful!
Preview URL: https://dd06695e.drawnix.pages.dev
Branch Preview URL: https://laser-pointer.drawnix.pages.dev

View logs

@pubuzhixing8
Copy link
Contributor Author

pubuzhixing8 commented Sep 11, 2025

Hi @NaoCoding, I finished the feature, please help me review.

@NaoCoding NaoCoding self-requested a review September 11, 2025 14:40
Copy link
Member

@NaoCoding NaoCoding left a comment

Choose a reason for hiding this comment

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

Generally LGTM, just a small part I think maybe we can improve? Or actually there may be some cases and usage in the future.

@pubuzhixing8
Copy link
Contributor Author

@NaoCoding Done!

  1. Refactor the query logic into the init method and use a class selector instead of an ID.
  2. Replace the mousemove event with pointermove for touch device compatibility.
  3. Scope the event listeners to the component's container instead of the global document

Copy link
Member

@NaoCoding NaoCoding left a comment

Choose a reason for hiding this comment

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

LGTM!

@pubuzhixing8 pubuzhixing8 merged commit 535af0e into develop Sep 12, 2025
2 checks passed
@pubuzhixing8 pubuzhixing8 deleted the laser-pointer branch September 12, 2025 02:01
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