From: Ben Klang Date: Sat, 9 Jan 2010 16:47:04 +0000 (-0500) Subject: Shout: Add spinner on delete destination too X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=e2338cc5bcbac03f47887f79703f31b552e57781;p=horde.git Shout: Add spinner on delete destination too --- diff --git a/shout/templates/extensions/list.inc b/shout/templates/extensions/list.inc index 18cca33b3..91c87d6fd 100644 --- a/shout/templates/extensions/list.inc +++ b/shout/templates/extensions/list.inc @@ -105,6 +105,7 @@ function _resetExtenDest(exten) img.src = "getImageDir() . '/shout.png'; ?>"; text = document.createTextNode(" "+s+" "); del = document.createElement('img'); + del.id = "dest"+s+"X"+exten+"del"; del.src = "getImageDir('horde') . '/delete-small.png'; ?>" del.style.cursor = 'pointer'; del.setAttribute('onclick', 'delDest("'+exten+'", "device", "'+s+'")'); @@ -121,6 +122,7 @@ function _resetExtenDest(exten) img.src = "getImageDir() . '/telephone-pole.png'; ?>"; text = document.createTextNode(" "+s+" "); del = document.createElement('img'); + del.id = "dest"+s+"X"+exten+"del"; del.src = "getImageDir('horde') . '/delete-small.png'; ?>" del.style.cursor = 'pointer'; del.setAttribute('onclick', 'delDest("'+exten+'", "number", "'+s+'")'); @@ -263,6 +265,13 @@ function delDest(exten, type, dest) 'action': 'deleteDestination' }); + // Hide the delete button and replace it with a spinner + $("dest"+dest+"X"+exten+"del").hide(); + spinner = document.createElement('img'); + spinner.src = "getImageDir('horde') . '/loading.gif'; ?>" + parent = $("dest"+dest+"X"+exten+"del").parentNode; + parent.insertBefore(spinner, $("dest"+dest+"X"+exten+"del")); + // FIXME: Better error handling new Ajax.Request('', {