Index: includes/common.inc
===================================================================
RCS file: /cvs/drupal/drupal/includes/common.inc,v
retrieving revision 1.704
diff -u -p -r1.704 common.inc
--- includes/common.inc	19 Oct 2007 10:30:54 -0000	1.704
+++ includes/common.inc	29 Oct 2007 08:08:32 -0000
@@ -1915,6 +1915,25 @@ function drupal_get_js($scope = 'header'
   return $output;
 }
 
+function drupal_add_tabledrag($table_id, $group, $action, $relationship, $target, $source = NULL, $hidden = TRUE) {
+  static $js_added = FALSE;
+  if (!$js_added) {
+    drupal_add_js('misc/tabledrag.js', 'core');
+    $js_added = TRUE;
+  }
+
+  // If source isn't set, assume it is the same as the target.
+  $source = isset($source) ? $source : $target;
+  $settings['tabledrag'][$table_id][$group][] = array(
+    'target' => $target,
+    'source' => $source,
+    'relationship' => $relationship,
+    'action' => $action,
+    'hidden' => $hidden,
+  );
+  drupal_add_js($settings, 'setting');
+}
+
 /**
  * Aggregate JS files, putting them in the files directory.
  *
===================================================================
RCS file: misc/tabledrag.js
diff -N misc/tabledrag.js
--- /dev/null	1 Jan 1970 00:00:00 -0000
+++ misc/tabledrag.js	29 Oct 2007 08:08:33 -0000
@@ -0,0 +1,925 @@
+// $Id $
+
+Drupal.behaviors.tableDrag = function(context) {
+  for (var base in Drupal.settings.tabledrag) {
+    if (!$('#' + base + '.tabledrag-processed').size(), context) {
+      var tableSettings = Drupal.settings.tabledrag[base];
+
+      $('#' + base + ':not(.tabledrag-processed)').each(function() {
+        // Create the new tabledrag instance. Save in the Drupal variable
+        // to allow other scripts access to the object.
+        Drupal.tabledrag[base] = new Drupal.tabledrag(this, tableSettings);
+      });
+
+      $('#' + base).addClass('tabledrag-processed');
+    }
+  }
+};
+
+Drupal.tabledrag = function(table, tableSettings) {
+  var self = this;
+
+  // Required object variables.
+  this.table = table;
+  this.tableSettings = tableSettings;
+  this.dragObject = null; // Used to hold information about a current drag operation.
+  this.rowObject = null; // Provides operations for row manipulation.
+  this.oldRowElement = null; // Remember the previous element.
+  this.oldY = 0; // Used to determine up or down direction from last mouse move.
+
+  // Configure the scroll settings.
+  this.scrollSettings = { amount: 4, interval: 50, trigger: 70 };
+  this.scrollInterval = null;
+  this.scrollY = 0;
+  this.windowHeight = 0;
+
+  // Check if this table contains indentations.
+  var indents = $('div.indentation', table);
+  this.indentEnabled = indents.size() > 0 ? true : false;
+  if (this.indentEnabled) {
+    var indentSize = indents.css('width');
+    this.oldX = 0;
+    this.indentCount = 1; // Total width of indents, set in makeDraggable.
+    this.indentIncrement = indentSize.replace(/[0-9\.]*/, '');
+    this.indentAmount = parseInt(indentSize);
+  }
+
+  // Make each applicable row draggable.
+  $('tr.draggable', table).each(function() { self.makeDraggable(this); });
+
+  // Hide columns containing affected form elements.
+  this.hideColumns();
+
+  // Add mouse bindings to the document. The self variable is passed along
+  // as event handlers do not have direct access to the tabledrag object.
+  $(document).bind('mousemove', { tabledrag: self }, self.dragRow);
+  $(document).bind('mouseup', { tabledrag: self }, self.dropRow);
+};
+
+/**
+ * Hide the columns containing form elements according to the settings for
+ * this tabledrag instance.
+ */
+Drupal.tabledrag.prototype.hideColumns = function() {
+  for (var group in this.tableSettings) {
+    var hidden = this.tableSettings[group][0]['hidden'];
+    var field = $('.' + this.tableSettings[group][0]['target'] + ':first', this.table);
+    var cell = field.parents('td:first, th:first');
+    if (hidden && cell[0] && cell.css('display') != 'none') {
+      var columnIndex = $('td', field.parents('tr:first')).index(cell.get(0));
+      var headerIndex = $('td:not(:hidden)', field.parents('tr:first')).index(cell.get(0));
+      columnIndex++; // nth-child selector is 1 based, not 0 based.
+      headerIndex++;
+      $('tbody tr', this.table).each(function() {
+        // Find and hide the cell in the table body.
+        var cell = $('td:nth-child('+ columnIndex +')', this);
+        if (cell.size()) {
+          cell.css('display', 'none');
+        }
+        // We might be dealing with a row spanning the entire table.
+        // Reduce the colspan on the first cell to prevent the cell from
+        // overshooting the table.
+        else {
+          cell = $('td:first', this);
+          cell.attr('colspan', cell.attr('colspan') - 1);
+        }
+      });
+      $('thead tr', this.table).each(function() {
+        // Remove table header cells entirely (Safari doesn't hide properly).
+        var th = $('th:nth-child('+ headerIndex +')', this);
+        if (th.size()) {
+          th.remove();
+        }
+      });
+    }
+  }
+};
+
+/**
+ * Find the target used within a particular row and group.
+ */
+Drupal.tabledrag.prototype.rowSettings = function(group, row) {
+  var field = $('.' + group, row);
+  for (delta in this.tableSettings[group]) {
+    var targetClass = this.tableSettings[group][delta]['target'];
+    if (field.is('.' + targetClass)) {
+      // Return a copy of the row settings.
+      var rowSettings = new Object();
+      for (var n in this.tableSettings[group][delta]) {
+        rowSettings[n] = this.tableSettings[group][delta][n];
+      }
+      return rowSettings;
+    }
+  }
+};
+
+/**
+ * Take an item and add an onmousedown method so that we can make it draggable.
+ */
+Drupal.tabledrag.prototype.makeDraggable = function(item) {
+  if (!item) return;
+  var self = this; // Keep the context of the tabledrag object.
+  // Create the handle.
+  var handle = $('<a href="#" title="Drag to re-order" class="tabledrag-handle"><div class="handle">&nbsp;</div></a>');
+  // Insert the handle after indentations (if any).
+  if ($('td:first .indentation:last', item).after(handle).size()) {
+    // Update the total width of indentation in this entire table.
+    self.indentCount = Math.max($('.indentation', item).size(), self.indentCount);
+  }
+  else {
+    $('td:first', item).prepend(handle);
+  }
+
+  // Add hover action for the handle.
+  handle.hover(function() {
+    self.dragObject == null ? $(this).addClass('tabledrag-handle-hover') : null;
+  }, function() {
+    self.dragObject == null ? $(this).removeClass('tabledrag-handle-hover') : null;
+  });
+
+  // Add the mousedown action for the handle.
+  handle.mousedown(function(event) {
+    // Create a new dragObject recording the event information.
+    self.dragObject = new Object();
+    self.dragObject.initMouseOffset = self.getMouseOffset(item, event);
+    self.dragObject.initMouseCoords = self.mouseCoords(event);
+    if (self.indentEnabled) {
+      self.dragObject.indentMousePos = self.dragObject.initMouseCoords;
+    }
+
+    // If there's a lingering row object from the keyboard, remove it's focus.
+    if (self.rowObject) {
+      $('a.tabledrag-handle', self.rowObject.element).blur();
+    }
+    // Create a new rowObject for manipulation of this row.
+    self.rowObject = new self.row(item, true);
+
+    // Save the position of the table.
+    self.table.topY = self.getPosition(self.table).y;
+    self.table.bottomY = self.table.topY + self.table.offsetHeight;
+
+    // Add classes to the handle and row.
+    $(this).addClass('tabledrag-handle-hover');
+    $(item).addClass('drag');
+    if (self.oldRowElement) {
+      $(self.oldRowElement).removeClass('highlight');
+    }
+
+    // Set the document to use the move cursor during drag.
+    $('body').addClass('drag');
+
+    // Hack for IE6 that flickers uncontrollably if select lists are moved.
+    if (navigator.userAgent.indexOf('MSIE 6.') != -1) {
+      $('select', this.table).css('display', 'none');
+    }
+    return false;
+  });
+
+  // Prevent the anchor tag from jumping us to the top of the page.
+  handle.click(function() {
+    return false;
+  });
+
+  // Similar to the .hover method, add a class when the handle is focused.
+  handle.focus(function() {
+    $(this).addClass('tabledrag-handle-hover');
+    self.safeBlur = true;
+  });
+
+  // Add arrow-key support to the handle.  
+  handle.keydown(function(event) {
+    // If a rowObject doesn't yet exist and this isn't the tab key.
+    if (event.keyCode != 9 && !self.rowObject) {
+      self.rowObject = new self.row(item, true);
+    }
+
+    switch (event.keyCode) {
+      case 37: // Left arrow.
+      case 63234: // Safari left arrow.
+        self.rowObject.indent(-1);
+        break;
+      case 38: // Up arrow.
+      case 63232: // Safari up arrow.
+        var previousRow = $(self.rowObject.group).filter(':first').prev('tr').get(0);
+        if (previousRow) {
+          self.safeBlur = false; // Do not allow the onBlur cleanup.
+          if (!self.rowObject.isValidSwap(previousRow, 'up', 0)) {
+            self.rowObject.swap('before', previousRow);
+            self.rowObject.indent(-1);
+          }
+          else {
+            self.rowObject.swap('before', previousRow);
+          }
+          handle.focus(); // Regain focus after the DOM manipulation.
+          window.scrollBy(0, -parseInt(item.offsetHeight));
+        }
+        break;
+      case 39: // Right arrow.
+      case 63235: // Safari right arrow.
+        self.rowObject.indent(1);
+        break;
+      case 40: // Down arrow.
+      case 63233: // Safari down arrow.
+        var nextRow = $(self.rowObject.group).filter(':last').next('tr').get(0);
+        if (nextRow) {
+          self.safeBlur = false; // Do not allow the onBlur cleanup.
+          if (!self.rowObject.isValidSwap(nextRow, 'down', 0)) {
+            self.rowObject.swap('after', nextRow);
+            self.rowObject.indent(-1);
+          }
+          else {
+            self.rowObject.swap('after', nextRow);
+          }
+          handle.focus(); // Regain focus after the DOM manipulation.
+          window.scrollBy(0, parseInt(item.offsetHeight));
+        }
+        break;
+    }
+
+    if (self.rowObject && self.rowObject.changed == true) {
+      $(item).addClass('drag');
+      if (self.oldRowElement) {
+        $(self.oldRowElement).removeClass('highlight');
+      }
+      self.oldRowElement = item;
+      self.updateFields(self.rowObject.element);
+      self.rowObject.markChanged();
+      self.restripeTable();
+    }
+  });
+
+  // Cleanup keyboard support on link blur.
+  handle.blur(function(event) {
+    if (self.rowObject && self.safeBlur) {
+      self.rowObject.removeIndentClasses();
+      if (self.rowObject.changed) {
+        $(item).removeClass('drag').addClass('highlight');
+      }
+      self.rowObject = null;
+    }
+    $(this).removeClass('tabledrag-handle-hover');
+  });
+
+  // Compatibility addition, return false on keypress to prevent unwanted scrolling.
+  handle.keypress(function(event) {
+    switch (event.keyCode) {
+      case 37: // Left arrow.
+      case 63234: // Safari left arrow.
+      case 38: // Up arrow.
+      case 63232: // Safari up arrow.
+      case 39: // Right arrow.
+      case 63235: // Safari right arrow.
+      case 40: // Down arrow.
+      case 63233: // Safari down arrow.
+        return false;
+    }
+  });
+};
+
+/**
+ * Mousemove event handler, bound to document.
+ */
+Drupal.tabledrag.prototype.dragRow = function(event) {
+  var self = event.data.tabledrag;
+  if (self.dragObject) {
+    self.currentMouseCoords = self.mouseCoords(event);
+
+    var y = self.currentMouseCoords.y - self.dragObject.initMouseOffset.y;
+    var x = self.currentMouseCoords.x - self.dragObject.initMouseOffset.x;
+
+    // Check for row swapping and vertical scrolling.
+    if (y != self.oldY) {
+      var dragDirection = y > self.oldY ? 'down' : 'up';
+      self.oldY = y; // Update the old value.
+ 
+      // Check if the window should be scrolled (and how fast).
+      var scrollAmount = self.checkScroll(self.currentMouseCoords.y);
+      // Stop any current scrolling.
+      clearInterval(self.scrollInterval);
+      // Continue scrolling if the mouse has moved in the scroll direction.
+      if (scrollAmount > 0 && dragDirection == 'down' || scrollAmount < 0 && dragDirection == 'up') {
+        self.setScroll(scrollAmount);
+      }
+
+      // If we have a valid target, perform the swap and restripe the table.
+      var currentRow = self.findDropTargetRow(x, y, dragDirection);
+      if (currentRow) {
+        if (dragDirection == 'down') {
+          self.rowObject.swap('after', currentRow, self);
+        }
+        else {
+          self.rowObject.swap('before', currentRow, self);
+        }
+        self.restripeTable();
+      }
+    }
+
+    // Similar to row swapping, handle indentations.
+    if (self.indentEnabled && x != self.oldX) {
+      self.oldX = x;
+      var xDiff = self.currentMouseCoords.x - self.dragObject.indentMousePos.x;
+      // Set the number of indentations the mouse has been moved left or right.
+      var indentDiff = parseInt(xDiff / self.indentAmount);
+      // Limit the indentation to no less than the left edge of the table and no
+      // more than the total amount of indentation in the table.
+      indentDiff = indentDiff > 0 ? Math.min(indentDiff, self.indentCount - self.rowObject.indents + 1) : Math.max(indentDiff, -self.rowObject.indents);
+      if (indentDiff != 0) {
+        // Indent the row with our estimated diff, which may be further
+        // restricted according to the rows around this row.
+        var indentChange = self.rowObject.indent(indentDiff);
+
+        // Update table and mouse indentations.
+        self.dragObject.indentMousePos.x += self.indentAmount * indentChange;
+        self.indentCount = Math.max(self.indentCount, self.rowObject.indents);
+      }
+    }
+
+    return false;
+  }
+};
+
+/**
+ * Mouseup event handler, bound to document.
+ */
+Drupal.tabledrag.prototype.dropRow = function(event) {
+  var self = event.data.tabledrag;
+  if (self.dragObject != null) {
+    var droppedRow = self.rowObject.element;
+    // The row is already in the right place so we just release it.
+    $(droppedRow).removeClass('drag').addClass('highlight');
+    $('.tabledrag-handle', droppedRow).removeClass('tabledrag-handle-hover');
+    if (self.rowObject.changed == true) {
+      self.updateFields(droppedRow);
+      self.rowObject.markChanged();
+    }
+    if (self.indentEnabled) {
+      self.rowObject.removeIndentClasses();
+    }
+    self.onDrop();
+    self.oldRowElement = droppedRow;
+    self.dragObject = null;
+    self.rowObject = null;
+    $('body').removeClass('drag');
+    clearInterval(self.scrollInterval);
+
+    // Hack for IE6 that flickers uncontrollably if select lists are moved.
+    if (navigator.userAgent.indexOf('MSIE 6.') != -1) {
+      $('select', this.table).css('display', 'block');
+    }
+  }
+};
+
+/**
+ * Get the position of an element by going up the DOM tree and adding up all
+ * the offsets.
+ */
+Drupal.tabledrag.prototype.getPosition = function(element){
+  var left = 0;
+  var top  = 0;
+  // Because Safari doesn't report offsetHeight on table rows, but does on table
+  // cells, grab the firstChild of the row and use that instead.
+  // http://jacob.peargrove.com/blog/2006/technical/table-row-offsettop-bug-in-safari
+  if (element.offsetHeight == 0) {
+    element = element.firstChild; // a table cell
+  }
+
+  while (element.offsetParent){
+    left   += element.offsetLeft;
+    top    += element.offsetTop;
+    element = element.offsetParent;
+  }
+
+  left += element.offsetLeft;
+  top  += element.offsetTop;
+
+  return {x:left, y:top};
+};
+
+/**
+ * Get the mouse coordinates from the event (allowing for browser differences).
+ */
+Drupal.tabledrag.prototype.mouseCoords = function(event){
+  if (event.pageX || event.pageY) {
+    return {x:event.pageX, y:event.pageY};
+  }
+  return {
+    x:event.clientX + document.body.scrollLeft - document.body.clientLeft,
+    y:event.clientY + document.body.scrollTop  - document.body.clientTop
+  };
+};
+
+/**
+ * Given a target element and a mouse event, get the mouse offset from that
+ * element. To do this we need the element's position and the mouse position.
+ */
+Drupal.tabledrag.prototype.getMouseOffset = function(target, event) {
+  var docPos   = this.getPosition(target);
+  var mousePos = this.mouseCoords(event);
+  return {x:mousePos.x - docPos.x, y:mousePos.y - docPos.y};
+};
+
+/**
+ * Find the row the mouse is currently over. This row is then taken and swapped
+ * with the one being dragged.
+ * 
+ * @param x
+ *   The x coordinate of the mouse on the page (not the screen).
+ * @param y
+ *   The y coordinate of the mouse on the page (not the screen).
+ * @param dragDirection
+ *   The y coordinate of the mouse on the page (not the screen).
+ */
+Drupal.tabledrag.prototype.findDropTargetRow = function(x, y, dragDirection) {
+  var rows = this.table.tBodies[0].rows;
+  for (var n=0; n<rows.length; n++) {
+    var row = rows[n];
+    var rowY = this.getPosition(row).y;
+    var rowHeight = parseInt(row.offsetHeight)/2;
+    if (row.offsetHeight == 0) {
+      rowY = this.getPosition(row.firstChild).y;
+      rowHeight = parseInt(row.firstChild.offsetHeight)/2;
+    }
+    // Because we always insert before, we need to offset the height a bit.
+    if ((y > (rowY - rowHeight)) && (y < (rowY + rowHeight))) {
+      if (this.indentEnabled) {
+        // Check that this row is not a child of the row being dragged.
+        for (n in this.rowObject.group) {
+          if (this.rowObject.group[n] == row) {
+            return null;
+          }
+        }
+        // Check that moving this row will not cause invalid relationships. 
+        var xDiff = this.currentMouseCoords.x - this.dragObject.initMouseCoords.x;
+        var indentDiff = parseInt(xDiff / this.indentAmount);
+        if (!this.rowObject.isValidSwap(row, dragDirection, indentDiff)) {
+          return null;
+        }
+      }
+
+      // We've may have found the row the mouse just passed over, but it doesn't
+      // take into account hidden rows. Skip backwards until we find a draggable
+      // row.
+      while ($(row).is(':hidden') && $(row).prev('tr').is(':hidden')) {
+        row = $(row).prev('tr').get(0);
+      }
+      return row;
+    }
+  }
+  return null;
+};
+
+/**
+ * After the row is dropped, update the table fields according to the settings
+ * set for this table.
+ * 
+ * @param changedRow
+ *   DOM object for the row that was just dropped.
+ */
+Drupal.tabledrag.prototype.updateFields = function(changedRow) {
+  for (var group in this.tableSettings) {
+    // Each group may have a different setting for relationship, so we find
+    // the source rows for each seperately.
+    var rowSettings = this.rowSettings(group, changedRow);
+
+    // Siblings are easy, check previous and next rows.
+    if (rowSettings.relationship == 'sibling') {
+      var previousRow = $(changedRow).prev('tr').get(0);
+      var nextRow = $(changedRow).next('tr').get(0);
+      var sourceRow = changedRow;
+      if ($(previousRow).is('.draggable') && $('.' + group, previousRow).length) {
+        if (this.indentEnabled) {
+          if ($('.indentations', previousRow).size() == $('.indentations', changedRow)) {
+            sourceRow = previousRow;
+          }
+        }
+        else {
+          sourceRow = previousRow;
+        }
+      }
+      else if ($(nextRow).is('.draggable') && $('.' + group, nextRow).length) {
+        if (this.indentEnabled) {
+          if ($('.indentations', nextRow).size() == $('.indentations', changedRow)) {
+            sourceRow = nextRow;
+          }
+        }
+        else {
+          sourceRow = nextRow;
+        }
+      }
+    }
+    // Parents, look up the tree until we find a field not in this group.
+    // Go up as many parents as indentations in the changed row.
+    else if (rowSettings.relationship == 'parent') {
+      var previousRow = $(changedRow).prev('tr');
+      while (previousRow.length && $('.indentation', previousRow).length >= this.rowObject.indents) {
+        previousRow = previousRow.prev('tr');
+      }
+      // If we found a row.
+      if (previousRow.length) {
+        sourceRow = previousRow[0];
+      }
+      // Otherwise we went all the way to the top of the table.
+      // Assume that the first item has no indentions.
+      else {
+        // Basically copy first item's value.
+        sourceRow = $('tr.draggable:first')[0];
+        var useSibling = true;
+      }
+    }
+
+    // Because we may have moved the row from one category to another,
+    // take a look at our sibling and borrow its sources and targets.
+    this.copyDragClasses(sourceRow, changedRow, group);
+    rowSettings = this.rowSettings(group, changedRow);
+
+    // In the case that we're looking for a parent, but the row is at the top
+    // of the tree, copy our sibling values.
+    if (useSibling) {
+      rowSettings.relationship = 'sibling';
+      rowSettings.source = rowSettings.target;
+    }
+
+    var targetClass = '.' + rowSettings.target;
+    var targetElement = $(targetClass, changedRow).get(0);
+
+    // Check if a target element exists in this row.
+    if (targetElement) {
+      var sourceClass = '.' + rowSettings.source;
+      var sourceElement = $(sourceClass, sourceRow).get(0);
+      switch (rowSettings.action) {
+        case 'match':
+          // Update the value.
+          targetElement.value = sourceElement.value;
+          break;
+        case 'order':
+          var siblings = this.rowObject.findSiblings(rowSettings);
+          if ($(targetElement).is('select')) {
+            // Get a list of acceptable values.
+            var values = new Array();
+            $('option', targetElement).each(function() {
+              values.push(this.value);
+            });
+            // Populate the values in the siblings.
+            $(targetClass, siblings).each(function() {
+              this.value = values.shift();
+            });
+          }
+          else {
+            // Assume a numeric input field.
+            var weight = 0;
+            $(targetClass, siblings).each(function() {
+              this.value = weight;
+              weight++;
+            });
+          }
+          break;
+      }
+    }
+  }
+};
+
+/**
+ * Copy all special tabledrag classes from one row's form elements to a
+ * different one, removing any special classes that the destination row
+ * may have had.
+ */
+Drupal.tabledrag.prototype.copyDragClasses = function(sourceRow, targetRow, group) {
+  var sourceElement = $('.' + group, sourceRow);
+  var targetElement = $('.' + group, targetRow);
+  if (sourceElement.length && targetElement.length) {
+    targetElement[0].className = sourceElement[0].className;
+  }
+};
+
+Drupal.tabledrag.prototype.checkScroll = function(cursorY) {
+  var de  = document.documentElement;
+  var b  = document.body;
+
+  var windowHeight = this.windowHeight = window.innerHeight || (de.clientHeight && de.clientWidth != 0 ? de.clientHeight : b.offsetHeight);
+  var scrollY = this.scrollY = (document.all ? (!de.scrollTop ? b.scrollTop : de.scrollTop) : (window.pageYOffset ? window.pageYOffset : window.scrollY));
+  var trigger = this.scrollSettings.trigger;
+  var delta = 0;
+
+  // Return a scroll speed relative to the edge of the screen.
+  if (cursorY - scrollY > windowHeight - trigger) {
+    delta = trigger / (windowHeight + scrollY - cursorY);
+    delta = (delta > 0 && delta < trigger) ? delta : trigger;
+    return delta * this.scrollSettings.amount;
+  }
+  else if (cursorY - scrollY < trigger) {
+    delta = trigger / (cursorY - scrollY);
+    delta = (delta > 0 && delta < trigger) ? delta : trigger;
+    return -delta * this.scrollSettings.amount;
+  }
+};
+
+Drupal.tabledrag.prototype.setScroll = function(scrollAmount) {
+  var self = this;
+  this.scrollInterval = setInterval(function() {
+    // Update the scroll values stored in the object.
+    self.checkScroll(self.currentMouseCoords.y);
+    var aboveTable = self.scrollY > self.table.topY;
+    var belowTable = self.scrollY + self.windowHeight < self.table.bottomY;
+    if (scrollAmount > 0 && belowTable || scrollAmount < 0 && aboveTable) {
+      window.scrollBy(0, scrollAmount);
+    }
+  }, this.scrollSettings.interval);
+}
+
+Drupal.tabledrag.prototype.restripeTable = function() {
+  // :even and :odd are reversed because jquery counts from 0 and
+  // we count from 1, so we're out of sync.
+  $('tbody tr:not(:hidden)', this.table)
+    .filter(':odd').filter('.odd')
+      .removeClass('odd').addClass('even')
+    .end().end()
+    .filter(':even').filter('.even')
+      .removeClass('even').addClass('odd');
+};
+
+/**
+ * Stub function. Allows a custom handler when a row is dropped.
+ */
+Drupal.tabledrag.prototype.onDrop = function() {
+  return null;
+};
+
+/**
+ * Theme function for the marker to indicate a row has changed. If the tabledrag
+ * class is removed Drupal.tabledrag.markChanged must also be over ridden.
+ */
+Drupal.theme.prototype.tabledragChangedMarker = function () {
+  return '<span class="warning tabledrag-changed">*</span>';
+};
+
+Drupal.theme.prototype.tabledragIndentation = function () {
+  return '<div class="indentation">&nbsp;</div>';
+};
+
+/**
+ * Constructor to make a new object to manipulate a table row.
+ * 
+ * @param tableRow
+ *   The DOM element for the table row we will be manipulating.
+ * @param addClasses
+ *   Whether we want to add classes to this row to indicate child relationships.
+ */
+Drupal.tabledrag.prototype.row = function(tableRow, addClasses) {
+  this.element = tableRow;
+  this.group = new Array(tableRow);
+  this.groupDepth = $('.indentation', tableRow).size();
+  this.changed = false;
+  this.table = $(tableRow).parents('table:first').get(0);
+
+  // Check if this table contains indentations.
+  var indents = $('div.indentation', this.table);
+  this.indentEnabled = indents.size() > 0 ? true : false;
+
+  if (this.indentEnabled) {
+    this.indents = $('.indentation', tableRow).size();
+    this.children = this.findChildren(addClasses);
+    this.group = $.merge(this.group, this.children);
+    // Find the depth of this entire group.
+    for (var n = 0; n < this.group.length; n++) {
+      this.groupDepth = Math.max($('.indentation', this.group[n]).size(), this.groupDepth);
+    }
+  }
+}
+
+/**
+ * Find all children of rowObject by indentation.
+ *
+ * @param addClasses
+ *   Whether we want to add classes to this row to indicate child relationships.
+ */
+Drupal.tabledrag.prototype.row.prototype.findChildren = function(addClasses) {
+  var parentIndentation = this.indents;
+  var currentRow = $(this.element, this.table).next('tr.draggable');
+  var rows = new Array();
+  var child = 0;
+  while (currentRow.length) {
+    var rowIndentation = $('.indentation', currentRow).length;
+    // A greater indentation indicates this is a child.
+    if (rowIndentation > parentIndentation) {
+      child++;
+      rows.push(currentRow[0]);
+      if (addClasses) {
+        $('.indentation', currentRow).each(function(indentNum) {
+          if (child == 1 && (indentNum == parentIndentation)) {
+            $(this).addClass('tree-child-first');
+          }
+          if (indentNum == parentIndentation) {
+            $(this).addClass('tree-child');
+          }
+          else if (indentNum > parentIndentation) {
+            $(this).addClass('tree-child-horizontal');
+          }
+        });
+      }
+    }
+    else {
+      if (addClasses) {
+        $('.indentation:nth-child(' + (parentIndentation + 1) + ')', currentRow.prev('tr.draggable')).addClass('tree-child-last');
+      }
+      break;
+    }
+    currentRow = currentRow.next('tr.draggable');
+  }
+  return rows;
+};
+
+/**
+ * Ensure that two rows are allowed to be swapped. Returns false if swapping
+ * will cause invalid parent relationships.
+ * 
+ * @param row
+ *   The row being considered for swapping.
+ * @param direction
+ *   Which direction the mouse or keyboard is moving.
+ * @param indentDiff
+ *   A horizontal difference in indendations.
+ */
+Drupal.tabledrag.prototype.row.prototype.isValidSwap = function(row, direction, indentDiff) {
+  var rowIndents = $('.indentation', row).size();
+  var prevIndents = $('.indentation', $(row).prev('tr')).size();
+  var nextIndents = $('.indentation', $(row).next('tr')).size();
+
+  if (
+    (direction == 'down') && (
+      // Prevent being able to drag a row downward with 2 indentations from a parent.
+      this.indents > rowIndents + 1 ||
+      // Prevent orphaning children when dragging into a parent.
+      this.indents < nextIndents - indentDiff
+    ) ||
+    (direction == 'up') && (
+      // Prevent being able to drag a row upward with 2 indentations from a parent.
+      this.indents < rowIndents ||
+      // Prevent orphaning children when dragging between a child and parent.
+      this.indents > prevIndents + 1 - indentDiff
+    )
+  ) {
+    return false;
+  }
+
+  // Restriction special cases for the first row in the table.
+  if (this.table.tBodies[0].rows[0] == row) {
+    // Do not let the first row contain indentations
+    // or let an un-draggable first row have anything put before it.
+    if (this.indents > 0 || $(row).is(':not(.draggable)')) {
+      return false;
+    }
+  }
+
+  return true;
+};
+
+/**
+ * Perform the swap between two rows.
+ * 
+ * @param position
+ *   Whether the swap will occur 'before' or 'after' the given row.
+ * @param row
+ *   DOM element what will be swapped with the row group.
+ */
+Drupal.tabledrag.prototype.row.prototype.swap = function(position, row) {
+  $(row)[position](this.group);
+  this.changed = true;
+  this.onSwap(row);
+}
+
+/**
+ * Indent a row within the legal bounds of the table.
+ * 
+ * @param indentDiff
+ *   The number of indentations the row group should receive, can be negative or
+ *   positive.
+ */
+Drupal.tabledrag.prototype.row.prototype.indent = function(indentDiff) {
+  if (indentDiff > 0) {
+    var prevRow = $(this.group).filter(':first').prev('tr').get(0);
+    if (prevRow) {
+      var prevIndent = $('.indentation', $(this.group).filter(':first').prev('tr')).size();
+      indentDiff = Math.min(prevIndent - this.indents + 1, indentDiff);
+    }
+    else {
+      indentDiff = 0; // First row may not contain indents.
+    }
+  }
+  else {
+    var nextIndent = $('.indentation', $(this.group).filter(':last').next('tr')).size();
+    indentDiff = Math.max(nextIndent - this.indents, indentDiff);
+  }
+
+  // Never allow indentation greater than 8 parents (menu system limit).
+  if (indentDiff + this.groupDepth > 8) {
+    indentDiff = 0;
+  }
+
+  for (var n = 1; n <= Math.abs(indentDiff); n++) {
+    // Add or remove indentations.
+    if (indentDiff < 0) {
+      $('.indentation:first', this.group).remove();
+      this.indents--;
+    }
+    else {
+      $('td:first', this.group).prepend(Drupal.theme('tabledragIndentation'));
+      this.indents++;
+    }
+  }
+  if (indentDiff) {
+    this.changed = true;
+  }
+
+  // Update indentation for this row.
+  this.groupDepth += indentDiff;
+  this.onIndent();
+
+  return indentDiff;
+}
+
+/**
+ * Find all siblings for a row, either according to its subgroup or indentation.
+ * Note that the passed in row is included in the list of siblings.
+ * 
+ * @param settings
+ *   The field settings we're using to identify what constitutes a sibling.
+ */
+Drupal.tabledrag.prototype.row.prototype.findSiblings = function(rowSettings) {
+  var siblings = new Array();
+  var directions = new Array('prev', 'next');
+  var rowIndentation = this.indents;
+  for (var d in directions) {
+    var checkRow = $(this.element)[directions[d]]();
+    while (checkRow.length) {
+      // Check that the sibling contains a similar target field.
+      if ($('.' + rowSettings.target, checkRow)) {
+        // Either add immediately if this is a flat table, or check to ensure
+        // that this row has the same level of indentaiton.
+        if (this.indentEnabled) {
+          var checkRowIndentation = $('.indentation', checkRow).length
+        }
+
+        if (!(this.indentEnabled) || (checkRowIndentation == rowIndentation)) {
+          siblings.push(checkRow[0]);
+        }
+        else if (checkRowIndentation < rowIndentation) {
+          // No need to keep looking for siblings when we get to a parent.
+          break;
+        }
+      }
+      else {
+        break;
+      }
+      checkRow = $(checkRow)[directions[d]]();
+    }
+    // Since siblings are added in reverse order for previous, reverse the
+    // completed list of previous siblings. Add the current row and continue.
+    if (directions[d] == 'prev') {
+      siblings.reverse();
+      siblings.push(this.element);
+    }
+  }
+  return siblings;
+};
+
+/**
+ * Remove indentation helper classes.
+ * 
+ * @param children
+ *   An array of DOM element table rows. The child classes will be removed from
+ *   all rows in this array.
+ */
+Drupal.tabledrag.prototype.row.prototype.removeIndentClasses = function() {
+  for (n in this.children) {
+    $('.indentation', this.children[n])
+      .removeClass('tree-child')
+      .removeClass('tree-child-first')
+      .removeClass('tree-child-last')
+      .removeClass('tree-child-horizontal');
+  }
+};
+
+/**
+ * Add an asterisk or other marker to the changed row.
+ */
+Drupal.tabledrag.prototype.row.prototype.markChanged = function() {
+  var marker = Drupal.theme('tabledragChangedMarker');
+  var cell = $('td:first', this.element);
+  if ($('span.tabledrag-changed', cell).length == 0) {
+    cell.append(marker);
+  }
+};
+
+/**
+ * Stub function. Allows a custom handler when a row is indented.
+ */
+Drupal.tabledrag.prototype.row.prototype.onIndent = function() {
+  return null;
+};
+
+/**
+ * Stub function. Allows a custom handler when a row is swapped.
+ */
+Drupal.tabledrag.prototype.row.prototype.onSwap = function(swappedRow) {
+  return null;
+};
Index: modules/block/block-admin-display-form.tpl.php
===================================================================
RCS file: /cvs/drupal/drupal/modules/block/block-admin-display-form.tpl.php,v
retrieving revision 1.1
diff -u -p -r1.1 block-admin-display-form.tpl.php
--- modules/block/block-admin-display-form.tpl.php	5 Oct 2007 09:36:52 -0000	1.1
+++ modules/block/block-admin-display-form.tpl.php	29 Oct 2007 08:08:33 -0000
@@ -6,13 +6,15 @@
  * Default theme implementation to configure blocks.
  *
  * Available variables:
- * - $block_listing: An array of block controls within regions.
+ * - $block_regions: An array of regions. Keyed by name with the title as value.
+ * - $block_listing: An array of blocks keyed by region and then delta.
+ * - $empty_text: The text displayed in a region header when it is empty.
  * - $form_submit: Form submit button.
  * - $throttle: TRUE or FALSE depending on throttle module being enabled.
  *
- * Each $data in $block_listing contains:
- * - $data->is_region_first: TRUE or FALSE depending on the listed blocks
- *   positioning. Used here to insert a region header.
+ * Each $block_listing[$region] contains an array of blocks for that region.
+ *
+ * Each $data in $block_listing[$region] contains:
  * - $data->region_title: Region title for the listed block.
  * - $data->block_title: Block title.
  * - $data->region_select: Drop-down menu for assigning a region.
@@ -25,9 +27,16 @@
  * @see theme_block_admin_display()
  */
 ?>
-<?php drupal_add_js('misc/tableheader.js'); ?>
-<?php print $messages; ?>
-
+<?php
+  // Add table javascript.
+  drupal_add_js('misc/tableheader.js');
+  drupal_add_js(drupal_get_path('module', 'block') .'/block.js');
+  drupal_add_js(array('block' => array('emptyText' => $empty_text)), 'setting');
+  foreach ($block_regions as $region => $title) {
+    drupal_add_tabledrag('blocks', 'block-region-select', 'match', 'sibling', 'block-region-'. $region, NULL, FALSE);
+    drupal_add_tabledrag('blocks', 'block-weight', 'order', 'sibling', 'block-weight-'. $region);
+  }
+?>
 <table id="blocks">
   <thead>
     <tr>
@@ -42,15 +51,14 @@
   </thead>
   <tbody>
     <?php $row = 0; ?>
-    <?php foreach ($block_listing as $data): ?>
-      <?php if ($data->is_region_first): ?>
-      <tr class="<?php print $row % 2 == 0 ? 'odd' : 'even'; ?>">
-        <td colspan="<?php print $throttle ? '7' : '6'; ?>" class="region"><?php print $data->region_title; ?></td>
+    <?php foreach ($block_regions as $region => $title): ?>
+      <tr class="region region-<?php print $region?> <?php print $row % 2 == 0 ? 'odd' : 'even'; print empty($block_listing[$region]) ? ' region-empty' : ''; ?>">
+        <td colspan="<?php print $throttle ? '6' : '5'; ?>" class="region"><?php print $title; print empty($block_listing[$region]) ? $empty_text : ''; ?></td>
       </tr>
-      <?php $row++; ?>
-      <?php endif; ?>
-      <tr class="<?php print $row % 2 == 0 ? 'odd' : 'even'; ?><?php print $data->row_class ? ' '. $data->row_class : ''; ?>">
-        <td class="block"><?php print $data->block_title; ?><?php print $data->block_modified ? '<span class="warning">*</span>' : ''; ?></td>
+      <?php $row++ ?>
+      <?php foreach ($block_listing[$region] as $delta => $data): ?>
+      <tr class="draggable <?php print $row % 2 == 0 ? 'odd' : 'even'; ?><?php print $data->row_class ? ' '. $data->row_class : ''; ?>">
+        <td class="block"><?php print $data->block_title; ?></td>
         <td><?php print $data->region_select; ?></td>
         <td><?php print $data->weight_select; ?></td>
         <?php if ($throttle): ?>
@@ -59,7 +67,8 @@
         <td><?php print $data->configure_link; ?></td>
         <td><?php print $data->delete_link; ?></td>
       </tr>
-      <?php $row++; ?>
+      <?php $row++ ?>
+      <?php endforeach; ?>
     <?php endforeach; ?>
   </tbody>
 </table>
Index: modules/block/block-rtl.css
===================================================================
RCS file: modules/block/block-rtl.css
diff -N modules/block/block-rtl.css
--- modules/block/block-rtl.css	10 Oct 2007 10:24:25 -0000	1.3
+++ /dev/null	1 Jan 1970 00:00:00 -0000
@@ -1,15 +0,0 @@
-/* $Id: block-rtl.css,v 1.3 2007/10/10 10:24:25 goba Exp $ */
-
-#blocks td.block {
-  padding-left: inherit;
-  padding-right: 1.5em;
-}
-#blocks select {
-  margin-left: 24px;
-}
-#blocks select.progress-disabled {
-  margin-left: 0px;
-}
-#blocks .progress .bar {
-  float: right;
-}
Index: modules/block/block.admin.inc
===================================================================
RCS file: /cvs/drupal/drupal/modules/block/block.admin.inc,v
retrieving revision 1.9
diff -u -p -r1.9 block.admin.inc
--- modules/block/block.admin.inc	5 Oct 2007 09:35:09 -0000	1.9
+++ modules/block/block.admin.inc	29 Oct 2007 08:08:33 -0000
@@ -36,15 +36,12 @@ function block_admin_display_form(&$form
   init_theme();
 
   $throttle = module_exists('throttle');
-  $block_regions = array(BLOCK_REGION_NONE => '<'. t('none') .'>') + system_region_list($theme_key);
+  $block_regions = system_region_list($theme_key) + array(BLOCK_REGION_NONE => '<'. t('none') .'>');
 
   // Build form tree
   $form = array(
     '#action' => arg(3) ? url('admin/build/block/list/'. $theme_key) : url('admin/build/block'),
     '#tree' => TRUE,
-    '#cache' => TRUE,
-    '#prefix' => '<div id="block-admin-display-form-wrapper">',
-    '#suffix' => '</div>',
   );
   foreach ($blocks as $i => $block) {
     $key = $block['module'] .'_'. $block['delta'];
@@ -69,7 +66,7 @@ function block_admin_display_form(&$form
     );
     $form[$key]['region'] = array(
       '#type' => 'select',
-      '#default_value' => $block['status'] ? (isset($block['region']) ? $block['region'] : system_default_region($theme_key)) : BLOCK_REGION_NONE,
+      '#default_value' => $block['region'],
       '#options' => $block_regions,
     );
 
@@ -82,20 +79,9 @@ function block_admin_display_form(&$form
     }
   }
 
-  // Attach the AHAH events to the submit button. Set the AHAH selector to every
-  // select element in the form. The AHAH event could be attached to every select
-  // element individually, but using the selector is more efficient, especially
-  // on a page where hundreds of AHAH enabled elements may be present.
   $form['submit'] = array(
     '#type' => 'submit',
     '#value' => t('Save blocks'),
-    '#ahah' => array(
-      'path' => 'admin/build/block/list/js/'. $theme_key,
-      'selector' => '#block-admin-display-form-wrapper select',
-      'wrapper' => 'block-admin-display-form-wrapper',
-      'event' => 'change',
-      'effect' => 'fade',
-    ),
   );
 
   return $form;
@@ -115,104 +101,28 @@ function block_admin_display_form_submit
 }
 
 /**
- * Javascript callback for AHAH replacement. Re-generate the form with the
- * updated values and return necessary html.
- */
-function block_admin_display_js($theme = NULL) {
-  // Load the cached form.
-  $form_cache = cache_get('form_'. $_POST['form_build_id'], 'cache_form');
-
-  // Set the new weights and regions for each block.
-  $blocks = array();
-  foreach (element_children($form_cache->data) as $key) {
-    $field = $form_cache->data[$key];
-    if (isset($field['info'])) {
-      $block = array(
-        'module'   => $field['module']['#value'],
-        'delta'    => $field['delta']['#value'],
-        'info'     => html_entity_decode($field['info']['#value'], ENT_QUOTES),
-        'region'   => $_POST[$key]['region'],
-        'weight'   => $_POST[$key]['weight'],
-        'status'   => $_POST[$key]['region'] == BLOCK_REGION_NONE ? 0 : 1,
-      );
-
-      $throttle = module_exists('throttle');
-      if ($throttle) {
-        $block['throttle'] = $_POST[$key]['throttle'];
-      }
-
-      if ($block['weight'] != $form_cache->data[$key]['weight']['#default_value'] || $block['region'] != $form_cache->data[$key]['region']['#default_value']) {
-        $changed_block = $block['module'] .'_'. $block['delta'];
-      }
-
-      $blocks[] = $block;
-    }
-  }
-
-  // Resort the blocks with the new weights.
-  usort($blocks, '_block_compare');
-
-  // Create a form in the new order.
-  $form_state = array('submitted' => FALSE);
-  $form = block_admin_display_form($form_state, $blocks, $theme);
-
-  // Maintain classes set on individual blocks.
-  foreach (element_children($form_cache->data) as $key) {
-    if (isset($form_cache->data[$key]['#attributes'])) {
-      $form[$key]['#attributes'] = $form_cache->data[$key]['#attributes'];
-    }
-  }
-
-  // Preserve the order of the new form while merging the previous data.
-  $form_order = array_flip(array_keys($form)); // Save the form order.
-  $form = array_merge($form_cache->data, $form);  // Merge the data.
-  $form = array_merge($form_order, $form); // Put back into the correct order.
-
-  // Add a permanent class to the changed block.
-  $form[$changed_block]['#attributes']['class'] = 'block-modified';
-
-  cache_set('form_'. $_POST['form_build_id'], $form, 'cache_form', $form_cache->expire);
-
-  // Add a temporary class to mark the new AHAH content.
-  $form[$changed_block]['#attributes']['class'] = empty($form[$changed_block]['#attributes']['class']) ? 'ahah-new-content' : $form[$changed_block]['#attributes']['class'] .' ahah-new-content';
-  $form['js_modified'] = array(
-    '#type' => 'value',
-    '#value' => TRUE,
-  );
-
-  $form['#post'] = $_POST;
-  $form['#theme'] = 'block_admin_display_form';
-
-  // Add messages to our output.
-  drupal_set_message(t('Your settings will not be saved until you click the <em>Save blocks</em> button.'), 'warning');
-
-  // Render the form.
-  drupal_alter('form', $form, array(), 'block_admin_display_form');
-  $form = form_builder('block_admin_display_form', $form, $form_state);
-
-  // Remove the wrapper from the form to prevent duplicate div IDs.
-  unset($form['#prefix'], $form['#suffix']);
-
-  $output = drupal_render($form);
-
-  // Return the output in JSON format.
-  drupal_json(array('status' => TRUE, 'data' => $output));
-}
-
-/**
  * Helper function for sorting blocks on admin/build/block.
  *
  * Active blocks are sorted by region, then by weight.
  * Disabled blocks are sorted by name.
  */
 function _block_compare($a, $b) {
-  $status = $b['status'] - $a['status'];
+  global $theme_key;
+  static $regions;
+
+  // We need the region list to correctly order by region.
+  if (!isset($regions)) {
+    $regions = array_flip(array_keys(system_region_list($theme_key)));
+    $regions[BLOCK_REGION_NONE] = count($regions);
+  }
+
   // Separate enabled from disabled.
+  $status = $b['status'] - $a['status'];
   if ($status) {
     return $status;
   }
-  // Sort by region.
-  $place = strcmp($a['region'], $b['region']);
+  // Sort by region (in the order defined by theme .info file).
+  $place = $regions[$a['region']] - $regions[$b['region']];
   if ($place) {
     return $place;
   }
@@ -442,14 +352,21 @@ function block_box_delete_submit($form, 
 function template_preprocess_block_admin_display_form(&$variables) {
   global $theme_key;
 
-  $variables['throttle'] = module_exists('throttle');
   $block_regions = system_region_list($theme_key);
+  $variables['throttle'] = module_exists('throttle');
+  $variables['block_regions'] = $block_regions + array(BLOCK_REGION_NONE => t('Disabled'));
+  $variables['empty_text'] = ' <span class="region-empty-text">('. t('empty') .')</span>';
 
-  // Highlight regions on page to provide visual reference.
   foreach ($block_regions as $key => $value) {
+    // Highlight regions on page to provide visual reference.
     drupal_set_content($key, '<div class="block-region">'. $value .'</div>');
+    // Initialize an empty array for the region.
+    $variables['block_listing'][$key] = array();
   }
 
+  // Initialize disabled blocks array.
+  $variables['block_listing'][BLOCK_REGION_NONE] = array();
+
   // Setup to track previous region in loop.
   $last_region = '';
   foreach (element_children($variables['form']) as $i) {
@@ -460,34 +377,23 @@ function template_preprocess_block_admin
       // Fetch region for current block.
       $region = $block['region']['#default_value'];
 
-      // Track first block listing to insert region header inside block_admin_display.tpl.php.
-      $is_region_first = FALSE;
-      if ($last_region != $region) {
-        $is_region_first = TRUE;
-        // Set region title. Block regions already translated.
-        if ($region != BLOCK_REGION_NONE) {
-          $region_title = drupal_ucfirst($block_regions[$region]);
-        }
-        else {
-          $region_title = t('Disabled');
-        }
-      }
-
-      $variables['block_listing'][$i]->is_region_first = $is_region_first;
-      $variables['block_listing'][$i]->row_class = isset($block['#attributes']['class']) ? $block['#attributes']['class'] : '';
-      $variables['block_listing'][$i]->block_modified = isset($block['#attributes']['class']) && strpos($block['#attributes']['class'], 'block-modified') !== FALSE ? TRUE : FALSE;
-      $variables['block_listing'][$i]->region_title = $region_title;
-      $variables['block_listing'][$i]->block_title =  drupal_render($block['info']);
-      $variables['block_listing'][$i]->region_select = drupal_render($block['region']) . drupal_render($block['theme']);
-      $variables['block_listing'][$i]->weight_select = drupal_render($block['weight']);
-      $variables['block_listing'][$i]->throttle_check = $variables['throttle'] ? drupal_render($block['throttle']) : '';
-      $variables['block_listing'][$i]->configure_link = drupal_render($block['configure']);
-      $variables['block_listing'][$i]->delete_link = !empty($block['delete']) ? drupal_render($block['delete']) : '';
+      // Set special classes needed for table drag and drop.
+      $variables['form'][$i]['region']['#attributes']['class'] = 'block-region-select block-region-'. $region;
+      $variables['form'][$i]['weight']['#attributes']['class'] = 'block-weight block-weight-'. $region;
+
+      $variables['block_listing'][$region][$i]->row_class = isset($block['#attributes']['class']) ? $block['#attributes']['class'] : '';
+      $variables['block_listing'][$region][$i]->block_modified = isset($block['#attributes']['class']) && strpos($block['#attributes']['class'], 'block-modified') !== FALSE ? TRUE : FALSE;
+      $variables['block_listing'][$region][$i]->block_title =  drupal_render($block['info']);
+      $variables['block_listing'][$region][$i]->region_select = drupal_render($block['region']) . drupal_render($block['theme']);
+      $variables['block_listing'][$region][$i]->weight_select = drupal_render($block['weight']);
+      $variables['block_listing'][$region][$i]->throttle_check = $variables['throttle'] ? drupal_render($block['throttle']) : '';
+      $variables['block_listing'][$region][$i]->configure_link = drupal_render($block['configure']);
+      $variables['block_listing'][$region][$i]->delete_link = !empty($block['delete']) ? drupal_render($block['delete']) : '';
+      $variables['block_listing'][$region][$i]->printed = FALSE;
 
       $last_region = $region;
     }
   }
 
-  $variables['messages'] = isset($variables['form']['js_modified']) ? theme('status_messages') : '';
   $variables['form_submit'] = drupal_render($variables['form']);
 }
Index: modules/block/block.css
===================================================================
RCS file: /cvs/drupal/drupal/modules/block/block.css,v
retrieving revision 1.5
diff -u -p -r1.5 block.css
--- modules/block/block.css	10 Oct 2007 10:24:25 -0000	1.5
+++ modules/block/block.css	29 Oct 2007 08:08:33 -0000
@@ -3,8 +3,10 @@
 #blocks td.region {
   font-weight: bold;
 }
-#blocks td.block {
-  padding-left: 1.5em; /* LTR */
+#blocks span.region-empty-text {
+  font-weight: normal;
+  font-size: 0.9em;
+  color: #999;
 }
 .block-region {
   background-color: #ff6;
@@ -12,12 +14,3 @@
   margin-bottom: 4px;
   padding: 3px;
 }
-#blocks select {
-  margin-right: 24px; /* LTR */
-}
-#blocks select.progress-disabled {
-  margin-right: 0px; /* LTR */
-}
-#blocks tr.ahah-new-content {
-  background-color: #ffd;
-}
Index: modules/block/block.js
===================================================================
RCS file: modules/block/block.js
diff -N modules/block/block.js
--- /dev/null	1 Jan 1970 00:00:00 -0000
+++ modules/block/block.js	29 Oct 2007 08:08:33 -0000
@@ -0,0 +1,84 @@
+// $Id $
+/**
+ * Move a block in the blocks table from one region to another. If the
+ * destination region is currently empty, unhide the region header. Conversely,
+ * if the region becomes empty, hide the region header.
+ * 
+ * This behavior is dependent on the tabledrag behavior, since it uses the
+ * defined classes initiallized in that behavior.
+ */
+Drupal.behaviors.blockRegionSelect = function(context) {
+  var table = $('table#blocks');
+  var tableDrag = Drupal.tabledrag.blocks; // Get the blocks tabledrag object.
+
+  // Add a custom handler for when a row is dropped, update empty regions.
+  tableDrag.row.prototype.onSwap = function() {
+    checkEmptyRegions(table);
+  };
+
+  $('select.block-region-select:not(.blockregionselect-processed)', context).each(function() {
+    $(this).change(function(event) {
+      // Make our new row and select field.
+      var row = $(this).parents('tr:first');
+      var select = $(this);
+
+      // Find the correct region and insert the row as the first in the region.
+      $('tr.region', table).each(function() {
+        if ($(this).is('.region-' + select[0].value)) {
+          // Ensure the region is visible.
+          $(this).removeClass('empty-region');
+          // Add the new row and remove the old one.
+          $(this).after(row);
+          // Manually update weights and restripe.
+          tableDrag.restripeTable(table.get(0));
+          tableDrag.markChanged(row.get(0));
+          tableDrag.updateFields(row.get(0));
+          // Add the highlight.
+          if (tableDrag.oldDragElement) {
+            $(tableDrag.oldDragElement).removeClass('highlight');
+          }
+          tableDrag.oldDragElement = row.get(0);
+          row.addClass('highlight');
+        }
+      });
+
+      // Modify empty regions with added or removed fields.
+      checkEmptyRegions(table);
+      // Remove focus from selectbox.
+      select.blur();
+    });
+    $(this).addClass('blockregionselect-processed');
+  });
+
+  var checkEmptyRegions = function(table) {
+    var emptyText = Drupal.settings.block.emptyText;
+
+    $('tr.region', table).each(function() {
+      if ($(this).next().is(':not(.draggable)') || $(this).next().size() == 0) {
+        // An empty region, add the empty text.
+        if ($(this).is(':not(.region-empty)')) {
+          $(this).addClass('region-empty');
+          $('td', this).append(emptyText);
+        }
+      }
+      else if ($(this).is('.region-empty')) {
+        // A populated region. Ensure the empty text is removed.
+        $(this).removeClass('region-empty');
+        var htmlText = $('<div>' + emptyText + '</div>').get(0).innerHTML;
+        $('td', this).get(0).innerHTML = $('td', this).get(0).innerHTML.replace(htmlText, '');
+
+        // Update the row fields with a value if necessary.
+        var regionField = $('select.block-region-select', $(this).next());
+        var weightField = $('select.block-weight', $(this).next());
+        var newRegionName = this.className.replace(/([^ ]+[ ]+)*region-([^ ]+)([ ]+[^ ]+)*/, '$2');
+        var oldRegionName = weightField[0].className.replace(/([^ ]+[ ]+)*block-weight-([^ ]+)([ ]+[^ ]+)*/, '$2');
+
+        if (!regionField.is('.block-region-'+ newRegionName)) {
+          regionField.removeClass('block-region-' + oldRegionName).addClass('block-region-' + newRegionName);
+          weightField.removeClass('block-weight-' + oldRegionName).addClass('block-weight-' + newRegionName);
+          regionField.val(newRegionName);
+        }
+      }
+    });
+  };
+};
\ No newline at end of file
Index: modules/block/block.module
===================================================================
RCS file: /cvs/drupal/drupal/modules/block/block.module,v
retrieving revision 1.284
diff -u -p -r1.284 block.module
--- modules/block/block.module	21 Oct 2007 18:59:01 -0000	1.284
+++ modules/block/block.module	29 Oct 2007 08:08:33 -0000
@@ -242,7 +242,7 @@ function _block_rehash() {
         if (!empty($old_blocks[$module][$delta])) {
           $block['status'] = $old_blocks[$module][$delta]->status;
           $block['weight'] = $old_blocks[$module][$delta]->weight;
-          $block['region'] = $old_blocks[$module][$delta]->region;
+          $block['region'] = empty($old_blocks[$module][$delta]->region) ? BLOCK_REGION_NONE : $old_blocks[$module][$delta]->region;
           $block['visibility'] = $old_blocks[$module][$delta]->visibility;
           $block['pages'] = $old_blocks[$module][$delta]->pages;
           $block['custom'] = $old_blocks[$module][$delta]->custom;
Index: modules/system/system.css
===================================================================
RCS file: /cvs/drupal/drupal/modules/system/system.css,v
retrieving revision 1.37
diff -u -p -r1.37 system.css
--- modules/system/system.css	10 Oct 2007 10:24:25 -0000	1.37
+++ modules/system/system.css	29 Oct 2007 08:08:35 -0000
@@ -3,6 +3,9 @@
 /*
 ** HTML elements
 */
+body.drag {
+  cursor: move;
+}
 th.active img {
   display: inline;
 }
@@ -11,7 +14,13 @@ tr.even, tr.odd {
   border-bottom: 1px solid #ccc;
   padding: 0.1em 0.6em;
 }
-td.active {
+tr.highlight {
+  background-color: #ffd;
+}
+tr.drag {
+  background-color: #fffff0;
+}
+tr.active, td.active {
   background-color: #ddd;
 }
 tbody {
@@ -32,6 +41,21 @@ thead th {
 .breadcrumb {
   padding-bottom: .5em
 }
+div.indentation {
+  width: 20px;
+  margin: -0.4em 0.2em -0.4em -0.4em;
+  padding: 0.4em 0 0.4em 0.6em;
+  float: left;
+}
+div.tree-child {
+  background: url(../../misc/tree.png) no-repeat 11px center;
+}
+div.tree-child-last {
+  background: url(../../misc/tree-bottom.png) no-repeat 11px center;
+}
+div.tree-child-horizontal {
+  background: url(../../misc/tree.png) no-repeat -11px center;
+}
 .error {
   color: #e55;
 }
@@ -333,6 +357,30 @@ html.js .resizable-textarea textarea {
 }
 
 /*
+** Table drag and drop.
+*/
+a.tabledrag-handle {
+  cursor: move;
+  float: left;
+  height: 1.7em;
+  margin: -0.42em 0 -0.42em -0.5em;
+  padding: 0.42em 1.5em 0.42em 0.5em;
+  text-decoration: none;
+}
+a.tabledrag-handle:hover {
+  text-decoration: none;
+}
+a.tabledrag-handle .handle {
+  margin-top: 0.3em;
+  height: 13px;
+  width: 13px;
+  background: url(../../misc/draggable.png) no-repeat 0 0;
+}
+a.tabledrag-handle-hover .handle {
+  background-position: 0 -20px;
+}
+
+/*
 ** Teaser splitter
 */
 .joined + .grippie {
Index: themes/garland/style.css
===================================================================
RCS file: /cvs/drupal/drupal/themes/garland/style.css,v
retrieving revision 1.27
diff -u -p -r1.27 style.css
--- themes/garland/style.css	11 Oct 2007 09:51:29 -0000	1.27
+++ themes/garland/style.css	29 Oct 2007 08:08:36 -0000
@@ -226,6 +226,14 @@ tr.even {
   background-color: #fff;
 }
 
+tr.highlight {
+  background-color: #ffd;
+}
+
+tr.drag {
+  background-color: #fffff0;
+}
+
 tr.odd td.active {
   background-color: #ddecf5;
 }
