[Tickets #1060] moving rules

[bugs@bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=1060
-----------------------------------------------------------------------
 Ticket             | 1060
 Updated By         | risko at risko.hu
 Summary            | moving rules
 Queue              | Ingo
 Version            | 1.0
 State              | Feedback
 Priority           | 1. Low
 Type               | Enhancement
 Owners             | 
+New Attachment     | ingo-move-v2.patch
-----------------------------------------------------------------------


risko at risko.hu (2005-07-25 07:08) wrote:

> Looks fine so far, but I have severaly comments/suggestions:
>
> - Don't use a POST submit, but a document.location redirection.
> - Use the rule_up/rule_down actions, just add an optional offset. You 
> can do the calculation with JavaScript.
> - in ruleUp()/ruleDown() merge the while() test into the for() test.
> - Move the JavaScript to a separate function in filters/header.inc.

Thanks for the feedback, here is the result.

Gergely