#40 ✓hold
Arjan

Drag drop fix for ie6

Reported by Arjan | July 8th, 2008 @ 04:59 PM

In Ie6, when using sortable.create the elements become invisible. I fixed this by adding the following line:

new Effect.Opacity(element, {duration:0.2, from:element._opacity, to:0.7});

To to bottom of the function:

Draggable.prototype = {

initialize: function(element) {

This trigger IE 6 to show the elements just like firefox allready does. The change doesnt seem to affect the behavior in Firefox :)

I hope this helps someone facing the same trouble I was.

Comments and changes to this ticket

  • underbjerg

    underbjerg November 11th, 2008 @ 12:10 PM

    • Tag set to draggable, ie, scriptaculous, sortable.create, sortables

    I can confirm this problem with Rails 2.1.0 and 2.1.2. In each case I updated to the latest prototype/script.aculo.us versions by running "rake rails:update:javascripts".

    Rails 2.1.0 was using Prototype 1.6.0.1. I don't know which script.aculo.us version is used.

    The fix you mention does work, but I assume this should be fixed in script.aculo.us itself.

  • ronin-15560 (at lighthouseapp)

    ronin-15560 (at lighthouseapp) November 20th, 2008 @ 04:49 PM

    • State changed from “new” to “hold”

    Can you guys add a failing functional test for this? The functional sortable tests work fine for me on IE6, no invisibility for me.

  • rajkovats

    rajkovats April 22nd, 2009 @ 04:10 PM

    Sortable works fine in IE7 - except from sortable TRs, they become invisible when they are dragged. After disabling the opacity-effect in line 259 of dragdrop.js:

    new Effect.Opacity(element, {duration:0.2, from:element._opacity, to:0.7});

    the table row remains visible while being dragged.

    That seems to be exactly the line Arjan mentioned above, but I did not see any negative effect in IE6 or IE7 after inserting

    if(!(element.tagName == 'TR' && Prototype.Browser.IE))

    before that line.

    however, dragging TRs does not work at all in IE6 (but IE8 is coming these days ;-))

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

script.aculo.us is an open-source JavaScript framework for visual effects and interface behaviours.
<br/><b>Source available from github</b>
The Git repository resides at:
<a href="http://github.com/madrobby/scriptaculous">http://github.com/madrobby/scriptaculous</a>
<br/>Check out the current development trunk with:
<code>git clone git://github.com/madrobby/scriptaculous.git</code>
<br/>As <b>script.aculo.us 1.xx is feature-frozen</b>, this development trunk is for <b>bugfixes only</b>.
<br/>New development should happen only for
<b>script.aculo.us 2</b>.
<br/><b>Creating a bug report</b>
When creating a bug report, be sure to include as much relevant information as possible. Post a an example that shows off the problem. Preferably, <b>alter the unit tests</b> and show through either changed or added tests how the expected behavior is not occuring.

People watching this ticket

Attachments

Pages