Sochettra / dynatree

Automatically exported from code.google.com/p/dynatree
0 stars 0 forks source link

Drag'n'drop errors when container div tag has scrollbars #211

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
(Reported by Robin Doer)
Hallo Martin!

Ich konnte noch einen Bug triggern. Ich weiß, wie man ihn repoduzieren kann, 
der Grund ist mir schleierhaft.

<script type="text/javascript">
  $(document).ready(function() {
    $("#tree").dynatree({
      dnd: {
        preventVoidMoves: true,
        onDragStart: function(node) {
          return true;
        },
        onDragEnter: function(node, sourceNode) {
          return true;
        },
        onDrop: function(node, sourceNode, hitMode, ui, draggable) {
          sourceNode.move(node, hitMode);
        }
      },
    });
  });
</script>

<style type="text/css">
  .dynatree-container {
    width: 100px;
  };
</style>

<div id="tree">
  <ul>
    <li><a href="#">X</a>
      <ul>
        <li><a href="#">X</a>
          <ul>
            <li><a href="#">X</a>
              <ul>
                <li><a href="#">X</a>
                  <ul>
                    <li><a href="#">X</a></li>
                  </ul>
                </li>
              </ul>
            </li>
          </ul>
        </li>
      </ul>
    </li>
  </ul>
</div>

Der Tree mit der Id "tree" erzeugt einfach nur einen Dummy-Tree mit einer paar 
Schachtelungen. Das CSS sorgt dafür, dass der Tree nur 100px breit ist. Wenn 
man nun den Tree aufklappt, dann erscheint durch 
die "overflow:auto"-Eigenschaft ein horizontaler Scrollbar. Verschiebst man 
den Scrollbar nun nach rechts, dann kriege ich einen Haufen Fehler.

sourceNode is null
jquery.dynatree.js (Zeile 3028)

Das kann man vermeiden, indem man die CSS-overflow-Eigenschaft in der 
dynatree-container-Klasse auf zum Beispiel "visible" setzt. Dann gibt es 
nämlich keinen Scrollbar, und das Problem tritt nicht auf.

Original issue reported on code.google.com by moo...@wwwendt.de on 1 Jul 2011 at 7:33

GoogleCodeExporter commented 8 years ago
I think I had same problem, and I fix them.
In my case attached file is helpfull.

Sorry for my English.

Original comment by il.shamg...@gmail.com on 14 Oct 2011 at 5:35

Attachments:

GoogleCodeExporter commented 8 years ago
I have this Bug in Google Chrome 14.
In Fire Fox 3.6 no Bug, and with this fix drag not start.

Original comment by il.shamg...@gmail.com on 17 Oct 2011 at 9:52

GoogleCodeExporter commented 8 years ago
Issue 234 has been merged into this issue.

Original comment by moo...@wwwendt.de on 17 Oct 2011 at 11:08

GoogleCodeExporter commented 8 years ago

Original comment by moo...@wwwendt.de on 17 Oct 2011 at 11:09

GoogleCodeExporter commented 8 years ago
This issue was closed by revision r542.

Original comment by moo...@wwwendt.de on 26 Dec 2011 at 10:25

GoogleCodeExporter commented 8 years ago
Fixed regression issue 268

Original comment by moo...@wwwendt.de on 7 Jan 2012 at 9:29

GoogleCodeExporter commented 8 years ago
see issue 278

Original comment by moo...@wwwendt.de on 9 Jun 2012 at 7:10