Looking for some various areas needing of improvement
authorEd Page <epage@laplace.(none)>
Sat, 25 Apr 2009 00:23:12 +0000 (19:23 -0500)
committerEd Page <epage@laplace.(none)>
Sat, 25 Apr 2009 00:23:12 +0000 (19:23 -0500)
src/rtm_view.py

index 8fab412..3270106 100644 (file)
@@ -73,6 +73,7 @@ def get_token(username, apiKey, secret):
 
 
 def get_credentials(credentialsDialog):
+       # @todo Pass in parent window
        username, password = credentialsDialog.request_credentials()
        token = get_token(username, rtm_backend.RtmBackend.API_KEY, rtm_backend.RtmBackend.SECRET)
        return username, password, token
@@ -300,6 +301,7 @@ class ItemListView(object):
                        elif viewColumn is self._dueColumn:
                                due = self._manager.get_task_details(taskId)["dueDate"]
                                if due.is_good():
+                                       # @todo Pass to calendar the parent widget
                                        calendar = gtk_toolbox.PopupCalendar(None, due.get(), "Due Date")
                                        calendar.run()
                        elif viewColumn is self._linkColumn:
@@ -307,6 +309,7 @@ class ItemListView(object):
                        elif viewColumn is self._notesColumn:
                                self._notesDialog.enable()
                                try:
+                                       # @todo Need to pass in parent window
                                        self._notesDialog.run(self._manager, taskId)
                                finally:
                                        self._notesDialog.disable()
@@ -482,10 +485,12 @@ class GtkRtMilk(object):
 
                self._editDialog.enable(self._manager)
                try:
+                       # @todo Need to pass in parent
                        self._editDialog.request_task(self._manager, taskId)
                finally:
                        self._editDialog.disable()
                self._itemView.reset_task_list(projId)
 
        def _on_proj_activate(self, *args):
+               # @todo Need to pass in parent
                self._projDialog.enable(self._manager)