Ticket #2529136 (closed defect)
Reporter  Iliyan Peychev |
Opened: 09/20/10 Last modified: 11/19/12 Status: closed Type: defect Resolution: wontfix |
Owner Dav Glass |
Target Release: Priority: P2 (high) |
|---|---|---|---|
| Summary: | Drag&Drop does not work in IE9 (Document mode - IE9 standards) | ||
| Description: | Drag&Drop does not work in IE9 Beta (Document mode - IE9 standards). if (YAHOO.util.Event.isIE && !e.button) { In IE9 (IE9 standards) e.button is 0, function returns and Drag&Drop does not start. Resize and Layout components also don't work, but the issue should be similar or same. |
||
| Type: | defect | Observed in Version: | 2.8.1 |
| Component: | DragDrop | Severity: | S3 (normal) |
| Assigned To: | Dav Glass | Target Release: | |
| Location: | Library Code | Priority: | P2 (high) |
| Tags: | Relates To: | ||
| Browsers: | All | ||
| URL: | |||
| Test Information: | Check any of the examples, related to Drag&Drop at YUI2 site. |
||
Change History
|
Posted: 09/20/10
|
|
Posted: 09/20/10
|
|
Posted: 09/20/10
This patch must be improved. Right now it broke the other browsers, for example Google Chrome. Your patch is: if ((YAHOO.util.Event.isIE < 9) && !e.button) { In Chrome, "isIE" will be 0 (of course) and "e.button" will be 0 also. The whole expression will be true and in fact - Drag&Drop stopped to work this time in Chrome. :) Also, I think "YAHOO.env.ua.ie" is better choice when you check for specific version, instead a method, which starts with "is", for example: var ieVersion = YAHOO.env.ua.ie; |
|
Posted: 09/20/10
Ha, my bad, I pushed the old commit and didn't push the one that fixed that.. Here's the one with the "right" patch: |
|
Posted: 09/21/10
The second patch is incorrect too. :) + if ((YAHOO.env.ua.ie < 9) && !e.button) { The correct code should be: |
|
Posted: 09/21/10
Wow, I really screwed up that set of commits. It's been a while since I looked at 2.x code ;) http://github.com/davglass/yui2/commit/87f00009db28dc511d0c239e9f359894a732bdb4 |
|
Posted: 09/22/10
|
|
Posted: 12/20/10
|
|
Posted: 01/10/11
|
|
Posted: 01/13/11
[Fixes #2529136] - Fixed issue with IE9 beta not reporting e.button properly |
|
Posted: 03/16/11
Ticket #2529307 was marked as a duplicate of this ticket. |
|
Posted: 04/13/11
|
George
bing [bot]
|
Posted: 11/14/12
|
|
Posted: 11/19/12
|
|
Posted: 11/19/12
Hi Dav, if (YAHOO.util.Event.isIE && !e.button) { When we comment this chunk of code, the dragging works but it has issue in dragging out of the drag area. Thanks. |
|
Posted: 11/19/12
We are not fixing issues with YUI 2, the project is in maintenance mode. |
A fix for this has been checked into HEAD and will be available on the next build:
http://github.com/davglass/yui2/commit/0179a9ef177b5cee565f0e3db1f4bb75e9542cf2