Changeset 51

Timestamp:

10/28/07 00:40:42 (6 years ago)

Author:

Lantash

Message:

Action symbol that are shown when the side menu isn't visible are observed in a much more efficient way. Introduced a single event handler that is attached to the whole table that handles this. [Closes #27]

File:

• trunk/javascript/klassenbuch.tasks.js (modified) (5 diffs)

trunk/javascript/klassenbuch.tasks.js

@@ -490 +490 @@
                 );
                 
-                this._taskTable.addColumn("Aktionen", (function() {
+                this._taskTable.addColumn("Aktionen", (function(task) {
                                 /**
                                  * Der HTML-Text für die Aktionssymbole, die angezeigt werden, wenn das Seitenmenü ausgeblendet ist.
@@ -499 +499 @@
                                  * @memberof TaskManagement.View
                                 */
-                                this._getContent5Cache = this._getContent5Cache || "<div class=\"taskTableActions\">" +
+                                this._getContent5Cache = this._getContent5Cache ||
                                         new Sprite("smallIcons", 0).toHTML("actionIcon iconShowComments") + ((User.signedIn) ? 
                                         new Sprite("smallIcons", 2).toHTML("actionIcon iconEditTask") +
                                         new Sprite("smallIcons", 4).toHTML("actionIcon iconDeleteTask") :
                                         new Sprite("smallIcons", 3).toHTML() +
-                                        new Sprite("smallIcons", 5).toHTML()) + "</div>";
-                                
-                                return this._getContent5Cache;
+                                        new Sprite("smallIcons", 5).toHTML());
+                                
+                                return "<div class=\"taskTableActions\" name=\"" + task.id + "\">" + this._getContent5Cache + "</div>";
                         }).bind(this), {
                                 width: "80px",
@@ -518 +518 @@
         this._taskTable.on("sort", this._onSort, this);
                 this._taskTable.on("highlightRow", this._onHighlightTask, this);
-                this._taskTable.on("refresh", this._onRefreshTable, this);
+                
+                this._taskTable.on("refresh", function() {
+                        this._getContent5Cache = "";
+                }, this);
                 
                 // Wird auf eine Aufgabe doppelt geklickt, wird das Kommentarfenster zu dieser Aufgabe geöffnet.
@@ -524 +527 @@
                         this.reportNavigation(task.id + "/kommentare");
                 }, this);
+                
+                this._taskTable.element.observe("click", (function(event) {
+                        if (!this._sideMenu.visible()) {
+                                var element = event.element();
+                                
+                                if (element.hasClassName("actionIcon")) {
+                                        var taskId = $(element.parentNode).readAttribute("name");
+                                        
+                                        if (taskId) {
+                                                if (element.hasClassName("iconShowComments")) {
+                                                        this.reportNavigation(taskId + "/kommentare");
+                                                } else if (element.hasClassName("iconEditTask")) {
+                                                        this.reportNavigation(taskId + "/bearbeiten")
+                                                } else if (element.hasClassName("iconDeleteTask")) {
+                                                        this.removeTask(TaskManagement.Tasks.find(function(task) {
+                                                                return task.id == taskId;
+                                                        }));
+                                                }
+                                        }
+                                }
+                        }
+                }).bindAsEventListener(this));
                 
                 // Zeichnet die Tabelle
@@ -681 +706 @@
                         this._sideMenu.items[1].enable();
                         this._sideMenu.items[2].enable();
-                }
-        },
-
-        /**
-         * Wird aufgerufen, wenn die Tabelle mit den Aufgaben neu "gezeichnet" wird, sei dies nun weil der Benutzer eine
-         * andere Sortierung gewählt hat oder weil die Aufgaben aktualisiert worden sind usw.<br /><br />Einerseits ist die
-         * Methode dafür verantwortlich, dass für den Fall, dass das Seitenmenü ausgeblendet ist, die Aktionssymbole in der
-         * Tabelle mit den entsprechenden Funktionen verknüpft werden. Andererseits erkennt die Funktion, wenn gar keine
-         * Aufgaben darzustellen sind, versteckt in diesem Fall die verbliebenen Spaltenüberschriften und zeigt einen Hinweis
-         * an, dass keine Aufgaben vorhanden sind.
-         * @memberof TaskManagement.View
-        */              
-        _onRefreshTable: function() {
-                this._getContent5Cache = "";
-                
-                // Aktionssymbole werden nur sichtbar, wenn das Seitenmenü versteckt ist.
-                if (!this._sideMenu.visible()) {
-                        var icons = this._taskTable.select(".actionIcon");
-                        
-                        // Wie viele Symbole gibt es pro Aufgabe
-                        var f = icons.length / this._taskTable.sortedRows.length;
-                        
-                        icons.each(function(icon, i) {
-                                var task = this._taskTable.rows.get(this._taskTable.sortedRows[Math.floor(i / f)]);
-                                
-                                if (icon.hasClassName("iconShowComments")) {
-                                        icon.observe("click", this.reportNavigation.bind(this, task.id + "/kommentare"));
-                                } else if (icon.hasClassName("iconEditTask")) {
-                                        icon.observe("click", this.reportNavigation.bind(this, task.id + "/bearbeiten"));
-                                } else if (icon.hasClassName("iconDeleteTask")) {
-                                        icon.observe("click", this.removeTask.bind(this, task));
-                                }
-                        }, this);
                 }
         },