[MAGNOLIA-3013] Moving paragraphs using edit bars not working properly in Internet Explorer (IE7 / IE8) Created: 18/Jan/10 Updated: 09/Oct/12 Resolved: 08/Jul/10 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | gui |
| Affects Version/s: | 4.2.3 |
| Fix Version/s: | 4.2.5, 4.3.3 |
| Type: | Bug | Priority: | Critical |
| Reporter: | Hay Kranen | Assignee: | Ondrej Chytil |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Internet Explorer 7.0 and 8.0 under Windows. |
||
| Attachments: |
|
||||||||
| Issue Links: |
|
||||||||
| Template: |
|
||||||||
| Acceptance criteria: |
Empty
|
||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
||||||||
| Date of First Response: | |||||||||
| Description |
|
When moving paragraphs using the 'move' button a bug occurs in both IE7 and IE8. The green bar is floating approximately 150 pixels higher than were the mouse cursor is, in some cases it's completely invisible. In Firefox / Safari the bar is located where you expect it to be, right next to the mouse cursor. |
| Comments |
| Comment by Magnolia International [ 08/Mar/10 ] |
|
Hay, sorry for not seeing this issue earlier - is this happening with custom templates or with STK, or ... ? A custom css maybe ? |
| Comment by Nickolaus Wing [ 11/Jun/10 ] |
|
We ran into this problem as well. It only happens when the window has been scrolled down a bit, and for us only seems to happen for paragraphs in our sidebar. There may be some kind of style difference that helps trigger it, but I couldn't tell you which. To fix it I changed getMousePos() to use the same method that the Prototype library uses to get the mouse position from the event. It worked. See the patch file I've included. |
| Comment by Nickolaus Wing [ 11/Jun/10 ] |
|
Actually I just confirmed that this is always broken for us in IE8 (which makes sense, considering the fix). The magnitude of the problem is directly proportional to how much you've scrolled down your window. |
| Comment by Nickolaus Wing [ 30/Jun/10 ] |
|
Here's a second patch to apply - I noticed that it was sometimes possible to move the mouse in IE7 before all the necessary objects were available, causing an error to be generated. |
| Comment by Daniel Lipp [ 13/Mar/12 ] |
|
This issue had been closed for 4.2 & 4.3-branch as well as on trunk - which at that time was for 4.4 |