{ "id": "125989", "key": "TIMOB-16534", "fields": { "issuetype": { "id": "1", "description": "A problem which impairs or prevents the functions of the product.", "name": "Bug", "subtask": false }, "project": { "id": "10153", "key": "TIMOB", "name": "Titanium SDK/CLI", "projectCategory": { "id": "10100", "description": "Titanium and related SDKs used in application development", "name": "Client" } }, "fixVersions": [ { "id": "15935", "description": "2014 Sprint 06", "name": "2014 Sprint 06", "archived": true, "released": true, "releaseDate": "2014-03-28" }, { "id": "15938", "description": "2014 Sprint 06 Tooling", "name": "2014 Sprint 06 Tooling", "archived": true, "released": true, "releaseDate": "2014-03-28" }, { "id": "15422", "description": "Release 3.3.0", "name": "Release 3.3.0", "archived": false, "released": true, "releaseDate": "2014-07-16" } ], "resolution": { "id": "1", "description": "A fix for this issue is checked into the tree and tested.", "name": "Fixed" }, "resolutiondate": "2014-03-24T01:38:18.000+0000", "created": "2014-02-06T19:05:15.000+0000", "priority": { "name": "High", "id": "2" }, "labels": [ "mobileweb", "qe-testadded" ], "versions": [ { "id": "15856", "description": "Release 3.2.1", "name": "Release 3.2.1", "archived": false, "released": true, "releaseDate": "2014-02-10" } ], "issuelinks": [], "assignee": { "name": "cbarber", "key": "cbarber", "displayName": "Chris Barber", "active": true, "timeZone": "America/Chicago" }, "updated": "2014-06-19T12:42:34.000+0000", "status": { "description": "The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.", "name": "Closed", "id": "6", "statusCategory": { "id": 3, "key": "done", "colorName": "green", "name": "Done" } }, "components": [ { "id": "11500", "name": "MobileWeb", "description": "Mobile Web (HTML) Platform" } ], "description": "Mousewheel code is developed to only support chrome and safari. Does not contain the correct code for Firefox or IE.\r\n\r\nFixed in this pull request:\r\n\r\nhttps://github.com/appcelerator/titanium_mobile/pull/4968", "attachment": [], "flagged": false, "summary": "MobileWeb: MouseWheel does not work on MobileWeb, Firefox and IE", "creator": { "name": "andrew@gstreetmedia.com", "key": "andrew@gstreetmedia.com", "displayName": "Andrew Greenstreet", "active": true, "timeZone": "America/Los_Angeles" }, "subtasks": [], "reporter": { "name": "andrew@gstreetmedia.com", "key": "andrew@gstreetmedia.com", "displayName": "Andrew Greenstreet", "active": true, "timeZone": "America/Los_Angeles" }, "environment": "3.2.1+", "comment": { "comments": [ { "id": "291870", "author": { "name": "ragrawal", "key": "ragrawal", "displayName": "Ritu Agrawal", "active": true, "timeZone": "America/Los_Angeles" }, "body": "Thank you for submitting this pull request. It appears that this pull request includes multiple improvements and fixes. Please submit one pull request for one issue. We would also need a test case that we can use to validate your issue and the fix.\n\nhttp://docs.appcelerator.com/titanium/latest/#!/guide/Pull_Request_Guide\n", "updateAuthor": { "name": "ragrawal", "key": "ragrawal", "displayName": "Ritu Agrawal", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2014-02-07T06:56:56.000+0000", "updated": "2014-02-07T06:56:56.000+0000" }, { "id": "293394", "author": { "name": "ragrawal", "key": "ragrawal", "displayName": "Ritu Agrawal", "active": true, "timeZone": "America/Los_Angeles" }, "body": "Please let us know if you plan to submit a single pull request for this change.", "updateAuthor": { "name": "ragrawal", "key": "ragrawal", "displayName": "Ritu Agrawal", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2014-02-17T07:18:50.000+0000", "updated": "2014-02-17T07:18:50.000+0000" }, { "id": "294306", "author": { "name": "ngupta", "key": "ngupta", "displayName": "Neeraj Gupta", "active": true, "timeZone": "America/Los_Angeles" }, "body": "Our Mobile Web platform targets mobile browsers and is neither supported nor optimized for Desktop browsers. Given that MouseWheel functionality is specific to Desktop browsers, we do not plan to add this functionality.\r\n\r\nThanks for your contributions.", "updateAuthor": { "name": "ngupta", "key": "ngupta", "displayName": "Neeraj Gupta", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2014-02-24T05:39:08.000+0000", "updated": "2014-02-24T05:39:08.000+0000" }, { "id": "294361", "author": { "name": "andrew@gstreetmedia.com", "key": "andrew@gstreetmedia.com", "displayName": "Andrew Greenstreet", "active": true, "timeZone": "America/Los_Angeles" }, "body": "That is NOT true. You already have the mouse wheel functionality in there for Safari, & Chrome, this patch just fixed it for Firefox and IE. \n\nPlease look at line 172 of your code and tell me what you see?\n\nhttps://github.com/appcelerator/titanium_mobile/blob/master/mobileweb/titanium/Ti/_/UI/KineticScrollView.js\n\nI see:\n\n{code}\n\t// Handle mouse wheel scrolling\nenableMouseWheel && (this._disconnectMouseWheelEvent = on(self.domNode, \"mousewheel\",function(e) {\nif (self.scrollingEnabled) {\nself._cancelAnimations();\nvar distanceX = contentContainer._measuredWidth - self._measuredWidth,\ndistanceY = contentContainer._measuredHeight - self._measuredHeight,\ncurrentPositionX = -self._currentTranslationX,\ncurrentPositionY = -self._currentTranslationY;\n\nsetMinTranslations();\n\n// Start the scrollbar\nself._startScrollBars({\nx: currentPositionX / distanceX,\ny: currentPositionY / distanceY\n},\n{\nx: self._measuredWidth / contentContainer._measuredWidth,\ny: self._measuredHeight / contentContainer._measuredHeight\n});\n\n// Set the scroll position\nself._setTranslation(Math.min(0, Math.max(self._minTranslationX,-currentPositionX + e.wheelDeltaX)),\nMath.min(0, Math.max(self._minTranslationY,-currentPositionY + e.wheelDeltaY)));\n\nclearTimeout(scrollbarTimeout);\nscrollbarTimeout = setTimeout(function(){\nself._endScrollBars();\n},500);\n\nself._handleMouseWheel && self._handleMouseWheel();\n}\n}));\n{code}\n", "updateAuthor": { "name": "andrew@gstreetmedia.com", "key": "andrew@gstreetmedia.com", "displayName": "Andrew Greenstreet", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2014-02-24T16:08:09.000+0000", "updated": "2014-02-24T16:08:09.000+0000" }, { "id": "294368", "author": { "name": "andrew@gstreetmedia.com", "key": "andrew@gstreetmedia.com", "displayName": "Andrew Greenstreet", "active": true, "timeZone": "America/Los_Angeles" }, "body": "I reopened a clean pull request for this issue. Use it if you would like.\n\nhttps://github.com/appcelerator/titanium_mobile/pull/5369 ", "updateAuthor": { "name": "andrew@gstreetmedia.com", "key": "andrew@gstreetmedia.com", "displayName": "Andrew Greenstreet", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2014-02-24T16:43:17.000+0000", "updated": "2014-02-24T16:43:17.000+0000" }, { "id": "294451", "author": { "name": "ngupta", "key": "ngupta", "displayName": "Neeraj Gupta", "active": true, "timeZone": "America/Los_Angeles" }, "body": "Reopening this ticket as we do support MouseWheel for Chrome and Safari, but not Firefox because they invented their own API a long time ago. While Appcelerator does NOT officially support desktop browsers including Firefox, it is recognized that desktop browsers are used for developing Mobile Web apps. Because of this, we would reevaluate this PR.", "updateAuthor": { "name": "ngupta", "key": "ngupta", "displayName": "Neeraj Gupta", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2014-02-24T20:17:20.000+0000", "updated": "2014-02-24T20:17:20.000+0000" }, { "id": "294455", "author": { "name": "andrew@gstreetmedia.com", "key": "andrew@gstreetmedia.com", "displayName": "Andrew Greenstreet", "active": true, "timeZone": "America/Los_Angeles" }, "body": "Thanks. I'm not sure I could make it through development without firebug, and firebug is only available for Firefox. Incidentally, even though you don't \"officially\" support desktop, with the exception of a mousewheel, I can't really see what is different. \n\nWe are currently building the Android & Web (yes, web desktop) versions of FindingRover.com using Titanium (ios is currently native and on the app store). With the exception of bugs that exist across all browsers (mobile and desktop) we really haven't encountered anything that is desktop specific. The only desktop specific things I can think of were mouse pointers (which were easily fixed) and mousewheel.\n\nSo far Ti MobileWeb is working great for a our desktop version. \n\nAnother benefit of the web version, is I can build a piece of functionality, compile in about 7 seconds and test. Nine times out ten, that code ports right back into the Android version. I can't begin to tell you just how valuable the web version of TI is; I'm saving literally 2 hours a day not having to compile to Android to test everything. ", "updateAuthor": { "name": "andrew@gstreetmedia.com", "key": "andrew@gstreetmedia.com", "displayName": "Andrew Greenstreet", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2014-02-24T20:34:15.000+0000", "updated": "2014-02-24T20:34:15.000+0000" }, { "id": "294458", "author": { "name": "ngupta", "key": "ngupta", "displayName": "Neeraj Gupta", "active": true, "timeZone": "America/Los_Angeles" }, "body": "Good to hear. You have submitted a number of tickets and pull requests over last few weeks and we will make sure that they get due attention. Thanks for your help.", "updateAuthor": { "name": "ngupta", "key": "ngupta", "displayName": "Neeraj Gupta", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2014-02-24T20:59:31.000+0000", "updated": "2014-02-24T20:59:31.000+0000" }, { "id": "295207", "author": { "name": "ragrawal", "key": "ragrawal", "displayName": "Ritu Agrawal", "active": true, "timeZone": "America/Los_Angeles" }, "body": "Moving this ticket to engineering to evaluate this PR in the context of this ticket.", "updateAuthor": { "name": "ragrawal", "key": "ragrawal", "displayName": "Ritu Agrawal", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2014-03-03T02:44:49.000+0000", "updated": "2014-03-03T02:44:49.000+0000" }, { "id": "304066", "author": { "name": "emerriman", "key": "emerriman", "displayName": "Eric Merriman ", "active": true, "timeZone": "America/Los_Angeles" }, "body": "What is the PR for this? Can't seem to track down if anything was merged.", "updateAuthor": { "name": "emerriman", "key": "emerriman", "displayName": "Eric Merriman ", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2014-05-08T18:26:51.000+0000", "updated": "2014-05-08T18:26:51.000+0000" }, { "id": "304073", "author": { "name": "cbarber", "key": "cbarber", "displayName": "Chris Barber", "active": true, "timeZone": "America/Chicago" }, "body": "[~emerriman] Read a few comments up and you'll see the PR is https://github.com/appcelerator/titanium_mobile/pull/5369.", "updateAuthor": { "name": "cbarber", "key": "cbarber", "displayName": "Chris Barber", "active": true, "timeZone": "America/Chicago" }, "created": "2014-05-08T18:35:48.000+0000", "updated": "2014-05-08T18:35:48.000+0000" }, { "id": "306913", "author": { "name": "oromero", "key": "oromero", "displayName": "Olga Romero", "active": true, "timeZone": "America/Los_Angeles" }, "body": "Tested and verified as fixed using\r\nMac osx 10.9.3 Mavericks\r\nWindows7, Windows8\r\nAppcelerator Studio, build: 3.3.0.201405271647\r\nTitanium SDK, build: 3.3.0.v20140528144113 \r\nNode.JS Version: v0.10.13 \r\nNPM Version: 1.3.2 \r\nacs@1.0.14 \r\nalloy@1.4.0-beta \r\nnpm@1.3.2 \r\ntitanium@3.3.0-beta \r\ntitanium-code-processor@1.1.1\r\n", "updateAuthor": { "name": "oromero", "key": "oromero", "displayName": "Olga Romero", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2014-05-29T21:14:23.000+0000", "updated": "2014-05-29T21:14:23.000+0000" } ], "maxResults": 15, "total": 15, "startAt": 0 } } }