Added drag and drop functionality for table rows

Review Request #147 - Created April 19, 2010 and submitted

Information
Andrei Grigorean
infoarena
Reviewers
hackers
A table can be made 'drag and drop'-able by a class assignment.

Tasks macro support a new parameter to make it support drag and drop.

Row parity needs to be fixed.

Bogdan-Cristian Tătăroiu
Seems ok to me. I'm not sure what our policy is regarding adding external js files though. Anyone else want to step it? :P

Also, maybe you want to create a separate branch for the features you are about to implement which we will merge back into trunk afterwards?
trunk/www/views/header.php (Diff revision 1)
 
 
We need to rethink this soon, the list of js files doesn't even fit on my screen :P
Savin  Tiberiu
We should move external js into a separate folder. About that long list, maybe we could make an array and use some php to write that list of scripts.
Something like:

js_scripts = Array('mochikit, dragndrp, ...);
foreach (script in js_scripts) {
  echo "<script ... />"
}

However this is irrelevant to the change. Great job Wef.
  1. I wasn't referring to how it looked in the code, I was referring to the fact that we load a lot of javascript files on every page that are not always needed and that do slow loading times down for the user. Not that infoarena is blazingly fast. :)
  2. My comment was in response to "We need to rethink this soon, the list of js files doesn't even fit on my screen :P" :). Anyway I get it what you mean, but it seems like an almost impossible change :P
trunk/www/macros/macro_tasks.php (Diff revision 1)
 
 
{ ... }
Loading...