frio: fix dropItem() didn't work for hubzila posts
authorrabuzarus <rabuzarus@t-online.de>
Wed, 7 Sep 2016 01:22:02 +0000 (03:22 +0200)
committerrabuzarus <rabuzarus@t-online.de>
Wed, 7 Sep 2016 01:22:02 +0000 (03:22 +0200)
view/theme/frio/js/textedit.js

index 0b3db05..39957db 100644 (file)
@@ -181,11 +181,19 @@ function confirmDelete() { return confirm(aStr.delitem); }
 
 function dropItem(url, object) {
        var confirm = confirmDelete();
+
+       //if the first character of the object is #, remove it because
+       // we use getElementById which don't need the #
+       // getElementByID selects elements even if there are special characters
+       // in the ID (like %) which won't work with jQuery
+       /// @todo ceck if we can solve this in the template
+       object = object.indexOf('#') == 0 ? object.substring(1) : object;
+
        if(confirm) {
                $('body').css('cursor', 'wait');
-               $(object).fadeTo('fast', 0.33, function () {
+               $(document.getElementById(object)).fadeTo('fast', 0.33, function () {
                        $.get(url).done(function() {
-                               $(object).remove();
+                               $(document.getElementById(object)).remove();
                                $('body').css('cursor', 'auto');
                        });
                });