Ticket #33

extensions/webmail/main.py

@@ -163,6 +163,12 @@ def move(request):
     mbc.move(",".join(msgset), fdname, request.GET["to"])
     return folder(request, fdname, False)
 
+@login_required
+def delete(request, fdname, mail_id):
+    mbc = IMAPconnector(request)
+    mbc.move(mail_id, fdname, "Trash")
+    return folder(request, fdname, False)
+
 @login_required
 def empty(request, name):
     mbc = IMAPconnector(request)

extensions/webmail/templates/index.html

@@ -160,7 +160,14 @@ tr.selected {
     updatelisting(resp, "hidden");
     $$("a[name=back]").addEvent("click", loadFolder);
     $$("a[name=reply]").addEvent("click", reply);
-    setDivHeight("mailcontent", /*$("table-container").getSize().y*/0, 0);
+    $$("a[name=delete]").addEvent("click", function(event) {
+      var lnk = event.target;
+
+      event.stop();
+      new Request.JSON({url: lnk.get("href"),
+                        onSuccess: get_callback("folder")}).get();
+    });
+    setDivHeight("mailcontent", 5, 0);
   });
 
   compose_callback = function(resp) {

extensions/webmail/templates/viewmail.html

@@ -22,7 +22,7 @@
       }
       #emailheaders td,
       #emailheaders th {
-          height: 20px;
+          height: 22px;
           text-align: left;
           padding-left: 10px;
           border: 1px solid #ccc;

extensions/webmail/templatetags/webextras.py

@@ -42,6 +42,10 @@ def viewm_menu(selection, session, mail_id, perms):
          "url" : "",
          "img" : "/static/pics/alias.png",
          "label" : _("Forward")},
+        {"name" : "delete",
+         "img" : "/static/pics/remove.png",
+         "url" : reverse(webmail.main.delete, args=[session["folder"], mail_id]),
+         "label" : _("Delete")},
         ]
     
     return render_to_string('main/menu.html', 

extensions/webmail/urls.py

@@ -6,6 +6,7 @@ urlpatterns = patterns('mailng.extensions.webmail.main',
                        (r'^move/$', "move"),
                        (r'^empty/(?P<name>.+)/', "empty"),
                        (r'^(?P<folder>.+)/(?P<mail_id>[\w\-\+]+)/part/$', 'getattachment'),
+                       (r'^(?P<fdname>.+)/(?P<mail_id>[\w\-\+]+)/delete/$', 'delete'),
                        (r'^(?P<folder>.+)/(?P<mail_id>[\w\-\+]+)/content/$', 'getmailcontent'),
                        (r'^(?P<folder>.+)/(?P<mail_id>[\w\-\+]+)/reply/$', 'reply'),
                        (r'^(?P<folder>.+)/(?P<mail_id>[\w\-\+]+)/$', 'viewmail'),

static/js/common.js

@@ -210,15 +210,10 @@ confirmation = function(question, action, callback) {
 setDivHeight = function(id, extrah, modulo) {
     var contentsize = $(document.body).getSize().y 
         - $("topmenubar").getSize().y
-        - $("header").getSize().y 
+        - $("header").getSize().y
+        - $("footer").getSize().y
         - extrah;
 
-    if ($("navbar")) {
-        contentsize -= $("navbar").getSize().y;
-    }
-    if ($("thead")) {
-        contentsize -= $("thead").getSize().y;
-    }
     if (modulo) {
         contentsize -= contentsize % modulo;
     }