bumbu/svg-pan-zoom

Method for only disabling the click listener's preventDefault()

Open

#156 opened on 2015年10月13日

GitHub で見る
 (7 comments) (0 reactions) (0 assignees)JavaScript (1,585 stars) (383 forks)batch import
enhancementhelp wanted

説明

Thank you for #123.

But having the new option affect all events goes a little far for my use case. Now if I set preventMouseEventsDefault: false, I scroll the screen when trying to pan via touch or zoom with the mousewheel.

Is there a way to prevent this?

コントリビューターガイド

技術スタック
javascript
領域
frontend
Issue 種別
feature
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
half day
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
mostly clear
前提条件
Basic JavaScriptUnderstanding of event listeners
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
40
調査方針
Investigate the current implementation of event handling in the library, specifically how options like `preventMouseEventsDefault` are applied. The goal is to add a separate option to disable `preventDefault()` only for click listeners without affecting scroll/pan behavior. Review the codebase for event listener registration (likely in `src/svg pan zoom.js` or similar). Consider adding a new boolean option, e.g., `preventClickDefault`, and modify the click event listener to conditionally call `preventDefault()`. Ensure backward compatibility with existing options. Reference the existing PR #123 for context on the original change.