saikota / google-maps-utility-library-v3

Automatically exported from code.google.com/p/google-maps-utility-library-v3
Apache License 2.0
0 stars 0 forks source link

InfoBox Scrolling wheel event should stop propagation #281

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
I'm using the InfoBox library as a InfoWindow substitute due to it's added 
flexibility. One disturbing difference I found was the scrolling wheel behavior 
which differs from the InfoWindow.
Compare 
https://developers.google.com/maps/documentation/javascript/examples/infowindow-
simple
To 
http://google-maps-utility-library-v3.googlecode.com/svn/trunk/infobox/examples/
infobox-basic.html
Using your scroll wheel inside the infowindow and infobox you get different 
response. The infowindow captures the scroll-event and the map does not zoom. 
Infobox passes the scroll event to the map and the map zooms. Even when you put 
a DIV in the infobox with it's own scroll bars, scrolling wheel events are sent 
to the map rather than the scrolling div.

What steps will reproduce the problem?
1. Put a infobox on a map
2. Put a div on the infobox with fixed height and overflow:auto
3. Add content to that div that makes scrollbars appear
4. Use your scroll wheel inside that div

Expected result:
The scrolling div schould scroll

Actual result:
The scrolling div does not scroll, the map zooms in and out

Version: Revision 457

Browser / Operating System:
Firefox, Chrome, IE9 / Windows

Additional comments:
[Enter any additional comments about the bug here.]

*********************************************************
Tip: Star this issue (next to title) to receive notifications of status
changes against this issue, also used as a gauge for how many people are
interested in seeing it resolved.
*********************************************************

Original issue reported on code.google.com by er...@cdyweb.com on 21 Nov 2013 at 3:30

GoogleCodeExporter commented 9 years ago
Add "mousewheel" and  "DOMMouseScroll" to the events list in Infobox.js .

Original comment by ty...@spacelist.ca on 18 Mar 2014 at 12:33

GoogleCodeExporter commented 9 years ago
I added "mousewheel" and  "DOMMouseScroll" to the events list in infobox.js and 
this has corrected the issue for me, thanks.

Suggest that these events should be included by default in the next version of 
infobox.js.

Original comment by percy.grunwald on 19 Mar 2014 at 12:36

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
Partial fixed; it solved it for me in Chrome, not in Firefox. Have not tried 
all other browsers.
Live example with the proposed fix is here
http://www.quinte-mls.com/listings/centre-hastings?PropertyType[]=res&district[]
=527

Original comment by er...@cdyweb.com on 19 Mar 2014 at 1:03

GoogleCodeExporter commented 9 years ago
Firefox fix (included jquery.mousehweel.js events)

events = ["mousedown", "mouseover", "mouseout", "mouseup",
      "click", "dblclick", "touchstart", "touchend", "touchmove", 'wheel', 'mousewheel', 'DOMMouseScroll', 'MozMousePixelScroll'];

Original comment by tamcs...@gmail.com on 4 Nov 2014 at 11:32