I sometimes need a delayed redirection (such as when displaying a short text in modal box after completing a form), so I've created this patch to add this command.

CommentFileSizeAuthor
#5 ctools-ajax-redirect-with-delayed-optional-check-for-positive-delay.patch1.1 KBdaniboy
#3 ctools-ajax-redirect-with-delayed-optional.patch1.01 KBdaniboy
#1 ctools-ajax-delayed-redirect.patch1.17 KBdaniboy

Comments

daniboy’s picture

daniboy commented
StatusFileSize
new ctools-ajax-delayed-redirect.patch1.17 KB

patch

daniboy’s picture

daniboy commented
Status: Active » Needs review
daniboy’s picture

daniboy commented
StatusFileSize
new ctools-ajax-redirect-with-delayed-optional.patch1.01 KB

I've rewritten the patch, this time it doesn't bloat the API. Please ignore the first patch.

merlinofchaos’s picture

merlinofchaos commented
Status: Needs review » Needs work

I did some asking around, and using setTimeout with a delay of 0 has different effects on different browsers. We should check and not use setTimeout in that case.

daniboy’s picture

daniboy commented
StatusFileSize
new ctools-ajax-redirect-with-delayed-optional-check-for-positive-delay.patch1.1 KB

Now with check for delay

merlinofchaos’s picture

merlinofchaos commented
Status: Needs work » Needs review

Setting back to nr so it gets seen.

merlinofchaos’s picture

merlinofchaos commented
Status: Needs review » Fixed

Committed. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

vasike’s picture

vasike
Location Ramnicu Valcea
commented

same thing for reload?