It would be nice if we could configure other options on the formatter such as the scrollWheelZoom. I will work on a patch.


briand44’s picture

Status:Active» Needs review
new1.66 KB

Here is the patch. It just adds a checkbox to disable the scrollWheelZoom.

rbp’s picture

new2.13 KB

Tested and worked great but re-rolled to reverse the logic of the setting.

rbp’s picture

new2.11 KB

Removed a dpm() statement.

briand44’s picture

Status:Needs review» Reviewed & tested by the community

Patch in #3 worked for me.

briand44’s picture

Status:Reviewed & tested by the community» Needs review
new3.38 KB

I created a new patch which adds the ability to disable the scrollWheelZoom in views as well.

danon1981’s picture

Trying the patch in #5 failed

patching file leaflet_views/
Hunk #1 FAILED at 33.
Hunk #2 FAILED at 154.
Hunk #3 FAILED at 365.

Has someone else applied the patch in #5 successfully?

2pha’s picture

the patch at: gives the ability to set all map options, including scrollWhelZoom.

gcb’s picture

Issue summary:View changes
Status:Needs review» Closed (duplicate)

Marking duplicate: if we are going to add this option we'll add the rest of the config options as in #2089209. Thanks for the patch though!

briand44’s picture

Status:Closed (duplicate)» Needs review
new1.98 KB

I am opening this issue back up because of I have re-created the patch to apply against the latest dev version.

bneil’s picture

new494 bytes
new2 KB

The functionality in #9 works as advertised. I did run into the issue identified in #2185767: Leaflet field formatter errors when trying to edit geocoded field in view or panel which is a separate issue and had to comment out #empty_options so I could test the patch.

I've modified only the #description to standardize it with the language used in the Leaflet API regarding the scroll wheel.

fullerja’s picture

Status:Needs review» Reviewed & tested by the community

Patch in #10 works for me. I also had to comment out #empty_options to test leaflet_views