6.0.0-beta1
▾
Tasks
New Task
Search
Photos
Wiki
▾
Tickets
New Ticket
Search
dev.horde.org
Toggle Alerts Log
Help
7/26/25
H
istory
A
ttachments
C
omment
W
atch
Download
Comment on [#1060] moving rules
*
Your Email Address
*
Spam protection
Enter the letters below:
. .. . .\ /.__. |\ |\ / | >< [__] | \| \/ \__|/ \| |
Comment
>> 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
Attachment
Watch this ticket
N
ew Ticket
M
y Tickets
S
earch
Q
uery Builder
R
eports
Saved Queries
Open Bugs
Bugs waiting for Feedback
Open Bugs in Releases
Open Enhancements
Enhancements waiting for Feedback
Bugs with Patches
Enhancements with Patches
Release Showstoppers
Stalled Tickets
New Tickets
Horde 5 Showstoppers