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

{passive: false}を明示的に指定。 #108

Merged
merged 1 commit into from Feb 24, 2024
Merged

Conversation

satopian

Collaborator

Screen-2024-02-23_13-15-34

Chromeのコンソールに、36件の

[Violation] Added non-passive event listener to a scroll-blocking

の警告が表示されるため、{passive: false}を明示的に指定しました。
Chromeの警告の内容は、{passive: true}として、Defaultの動作をキャンセルしない事をブラウザにあらかじめ伝えると高速に処理できるというものなので、Defaultの動作をキャンセルしていたり、キャンセルする可能性のある箇所に{passive: false}を指定しても処理速度が向上するわけでは無い筈なのですが、この指定によりChromeの警告を消す事はできました。
もとからイベントリスナーに入っていたtrue/falseの引数は、captureであるため、


  { passive: false,
	capture: false }

のようにまとめ、元の記述がtrueの場合はcapture: trueとして、同じ指定になるようにしました。

@satopian satopian merged commit ff724c1 into funige:master Feb 24, 2024

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

1 participant
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.

{passive: false}を明示的に指定。 by satopian · Pull Request #108 · funige/neo