Skip to content
You have no unread notifications
funige  /   neo  /   Pull requests #99  /  
Open in github.dev Open in a new github.dev tab Open in codespace
Code

オリジナルのPaintBBSと同じように、描画中に円カーソルを表示したままにする #99

Merged
merged 4 commits into from Dec 25, 2023

Conversation

satopian

Collaborator

PaintBBS.mp4


オリジナルのPaintBBSの円カーソルの動作。
オリジナルのPaintBBSをひさしぶりに使ってみたところ、オリジナルのPaintBBSは描画中に円カーソルが表示されたままだとわかりました。
オリジナルの動作に近づけるために、
freeHandMoveHandlerfreeHandUpMoveHandlerに円カーソルを表示する
this.drawCursor(oe);
を追加しました。
アプリの円カーソルをペンのタッチ時にあえて消しているHTML5のアプリもありますので、funigeさんがあえて、消えるようにしていたのかもしれません。
しかしながら、オリジナルに近づけたい、どんな些細な事でも情報が欲しいという当初の話もありましたので、プルリクエストさせていただきました。
マージするかどうかの判断はおまかせします。
また、一通りの動作確認はしましたが、
this.drawCursor(oe);
の追加位置が適切かどうか等、動作上の問題があるようでしたらご指摘いただければ幸いです。

@funige

Owner

funige commented Dec 23, 2023

Choose a reason for hiding this comment

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

お世話になってます。
もうぜんぜん記憶がないので、どうしてこういう仕様にしたのかわからないのですが、
うーん、何か理由があったような気がするのですが……何も思い出せないです。
土日に少しテストしてみて、問題出ないようでしたらマージする方向で考えたいと思います。

@funige funige merged commit 2a2d981 into funige:master Dec 25, 2023

Update branch attempt failed

Oops, something went wrong.

@funige

Add a comment

Select a reply

Slash commands
Beta
Give feedback
Slash commands
Beta
Give feedback

An unexpected error has occurred

Remember, contributions to this repository should follow our GitHub Community Guidelines.
ProTip! Add .patch or .diff to the end of URLs for Git’s plaintext views.
Labels
Apply labels to this pull request
None yet
Projects
Projects
None yet
Development
Link an issue from this repository

Successfully merging this pull request may close these issues.

None yet

2 participants
Lock conversation

Lock conversation on this pull request

  • Other users can’t add new comments to this pull request.
  • You and other collaborators with access to this repository can still leave comments that others can see.
  • You can always unlock this pull request again in the future.

Optionally, choose a reason for locking that others can see. Learn more about when it’s appropriate to lock conversations.

オリジナルのPaintBBSと同じように、描画中に円カーソルを表示したままにする by satopian · Pull Request #99 · funige/neo