16078 2007-11-21 00:03:03 -0800 Google Maps zooming via the scroll wheel (almost) always zooms IN only (Safari 3/10.4.11) 2007-12-15 01:04:38 -0800 1 1 1 Unclassified WebKit WebCore Misc. 523.x (Safari 3) Mac OS X 10.4 RESOLVED FIXED http://maps.google.com/ GoogleBug P2 Normal --- 1 ark3 ap ap oldest_to_newest 61886 0 ark3 2007-11-21 00:03:03 -0800 Overview Description: On Google Maps, using the scroll wheel in the maps area should allow you to zoom in or out. Scrolling up should zoom in and scrolling down should zoom out (based on my experience in Firefox and Camino). In Safari both scroll directions generally zoom in. A rapid multi-click scroll down often causes both zooming out and in, which is easily visible if the map tile images are already loaded/cached. Steps to Reproduce: 1. Load the URL http://maps.google.com/ 2. Move the pointer over the map region. 3. Scroll up using the mouse scroll wheel, one click at a time. This should zoom in. 4. Scroll down using the mouse scroll wheel, one click at a time. This should zoom out, but usually doesn't. 5. Scroll down rapidly (i.e. many clicks in quick succession). Actual Results: 3. Each click zooms IN one step (based on the scale on the left). This is correct. 4. Each click zooms IN one step (based on the scale on the left). This is WRONG! 5. The result is some amount of zooming in and some amount of zooming out. This is wrong. Expected Results: 3. Each click zooms IN one step (based on the scale on the left). 4. Each click zooms OUT one step (based on the scale on the left). 5. The map view zooms OUT many steps. Build Date & Platform: Safari Version 3.0.4 (523.12) Mac OS X 10.4.11 Build 8S165 (PPC/G4) 61887 1 ap 2007-11-21 00:04:56 -0800 Confirmed with r27628 (also 10.4.11 on PPC). 64173 2 17894 ap 2007-12-14 10:33:53 -0800 Created attachment 17894 proposed fix 64217 3 17894 darin 2007-12-14 15:25:17 -0800 Comment on attachment 17894 proposed fix r=me 64243 4 ap 2007-12-15 01:04:38 -0800 Committed revision 28745. For the record, this was reproducible on Leopard/Intel, too. 17894 2007-12-14 10:33:53 -0800 2007-12-14 15:25:17 -0800 proposed fix 16078r1_patch.txt text/plain 1612 ap SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiAyODcxMikKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTQgQEAKKzIwMDctMTItMTQgIEFsZXhleSBQcm9za3VyeWFrb3YgIDxhcEB3ZWJr aXQub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAg IGh0dHA6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE2MDc4CisgICAgICAgIEdv b2dsZSBNYXBzIHpvb21pbmcgdmlhIHRoZSBzY3JvbGwgd2hlZWwgKGFsbW9zdCkgYWx3YXlzIHpv b21zIElOIG9ubHkgKFNhZmFyaSAzLzEwLjQuMTEpCisKKyAgICAgICAgKiBkb20vV2hlZWxFdmVu dC5jcHA6IChXZWJDb3JlOjpXaGVlbEV2ZW50OjpXaGVlbEV2ZW50KToKKyAgICAgICAgRW5zdXJl IHRoYXQgZGVsdGEgaXMgbmV2ZXIgcm91bmRlZCBkb3duIHRvIHplcm8gLSB3ZSBhcmUgZ2V0dGlu ZyB2YWx1ZXMgbGVzcyB0aGFuIDEgZnJvbSAKKyAgICAgICAgbWFueSBtaWNlIG9uIE9TIFgsIGFu ZCBHb29nbGUgbWFwcyBjb2RlIGFzc3VtZXMgc2Nyb2xsaW5nIGRvd24gaWYgZXZlbnQud2hlZWxE ZWx0YSBpcyB6ZXJvLgorCiAyMDA3LTEyLTEzICBBbHAgVG9rZXIgIDxhbHBAYXRva2VyLmNvbT4K IAogICAgICAgICBjdXJsIGJhY2tlbmQgYnVpbGQgZml4IGZvciBicmVha2FnZSBpbnRyb2R1Y2Vk IGluIHIyODcwOS4KSW5kZXg6IFdlYkNvcmUvZG9tL1doZWVsRXZlbnQuY3BwCj09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0K LS0tIFdlYkNvcmUvZG9tL1doZWVsRXZlbnQuY3BwCShyZXZpc2lvbiAyODcxMSkKKysrIFdlYkNv cmUvZG9tL1doZWVsRXZlbnQuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC00OCw2ICs0OCwxMSBAQCBX aGVlbEV2ZW50OjpXaGVlbEV2ZW50KGZsb2F0IHdoZWVsRGVsdGFYCiAgICAgLCBtX3doZWVsRGVs dGFYKGxyb3VuZGYod2hlZWxEZWx0YVgpICogMTIwKQogICAgICwgbV93aGVlbERlbHRhWShscm91 bmRmKHdoZWVsRGVsdGFZKSAqIDEyMCkgLy8gTm9ybWFsaXplIHRvIHRoZSBXaW5kb3dzIDEyMCBt dWx0aXBsZQogeworICAgIC8vIFJvdW5kaW5nIGRlbHRhIHRvIHplcm8gbWFrZXMgbm8gc2Vuc2Ug YW5kIGJyZWFrcyBHb29nbGUgTWFwcywgPGh0dHA6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcu Y2dpP2lkPTE2MDc4Pi4KKyAgICBpZiAod2hlZWxEZWx0YVggJiYgIW1fd2hlZWxEZWx0YVgpCisg ICAgICAgIG1fd2hlZWxEZWx0YVggPSAod2hlZWxEZWx0YVggPiAwKSA/IDEyMCA6IC0xMjA7Cisg ICAgaWYgKHdoZWVsRGVsdGFZICYmICFtX3doZWVsRGVsdGFZKQorICAgICAgICBtX3doZWVsRGVs dGFZID0gKHdoZWVsRGVsdGFZID4gMCkgPyAxMjAgOiAtMTIwOwogfQogCiB2b2lkIFdoZWVsRXZl bnQ6OmluaXRXaGVlbEV2ZW50KGludCB3aGVlbERlbHRhWCwgaW50IHdoZWVsRGVsdGFZLCBBYnN0 cmFjdFZpZXcqIHZpZXcsCg==