deleting a task from the task editor does not close it

Bug #529919 reported by Luca Invernizzi on 2010-03-01
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Getting Things GNOME!
Critical
Luca Invernizzi

Bug Description

Deleting a task from the taskeditor does delete the task but doesn't close the editor itself.

Related branches

Changed in gtg:
status: Triaged → Fix Committed

This is a very interesting usecase. Your fix is good but, if we want to be really perfect, we should do it otherwise.

In fact, the TaskEditor should catch the "task-deleted" signal from the requster and react by closing itself.

This has several advantages : TaskEditor is independant from the ViewManager and can be replaced by any TaskEditor module while, with your fix, we create a strong dependancy that the TaskEditor should always have a destroy function that does what it's expected it should be doing (and this is not always clear).

By reaction to the signal, you are taking your own responsibility, just like the browser is doing right now.

This is, in essence, the new philosophy I had when doing the refactoring. (I hope it will work but, to be honnest, I don't believe that connecting to the signal is harder than your fix)

Changed in gtg:
milestone: 0.3 → 0.2.9
Izidor Matušov (izidor) on 2012-02-13
Changed in gtg:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers