GTG

Merge lp:~gtg-user/gtg/bugfix-493335 into lp:~gtg/gtg/old-trunk

Proposed by Alan Gomes Alvino
Status: Rejected
Rejected by: Izidor Matušov
Proposed branch: lp:~gtg-user/gtg/bugfix-493335
Merge into: lp:~gtg/gtg/old-trunk
Diff against target: 15 lines (+5/-0)
1 file modified
GTG/gtk/editor/taskview.py (+5/-0)
To merge this branch: bzr merge lp:~gtg-user/gtg/bugfix-493335
Reviewer Review Type Date Requested Status
Izidor Matušov Disapprove
Bertrand Rousseau (community) Needs Information
Review via email: mp+113499@code.launchpad.net

Description of the change

The bug is because the delete_range is call after the delete_selection, and the delete_seletcion doesn't remove the arrow. What i do was after the delete_range i remove the arrow.

To post a comment you must log in.
Revision history for this message
Bertrand Rousseau (bertrand-rousseau) wrote :

Do you need to both disconnect the delete-range signal and ask the buffer to stop the emission of this signal? Isn't that enough to just stop the emission of the signal?

review: Needs Information
Revision history for this message
Izidor Matušov (izidor) wrote :

Closing because of inactivity

review: Disapprove

Unmerged revisions

1200. By Alan Gomes <email address hidden>

Fixed delete bug

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'GTG/gtk/editor/taskview.py'
--- GTG/gtk/editor/taskview.py 2012-05-20 19:05:20 +0000
+++ GTG/gtk/editor/taskview.py 2012-07-05 07:10:24 +0000
@@ -797,6 +797,11 @@
797 it.forward_char()797 it.forward_char()
798 #now we really delete the selected stuffs798 #now we really delete the selected stuffs
799 selec = self.buff.get_selection_bounds()799 selec = self.buff.get_selection_bounds()
800 # Delete what not being deleted by delete_selection, the ideal is run _delete_range before the delete_selection
801 self.buff.disconnect(self.delete_sigid)
802 self.buff.stop_emission("delete-range")
803 self.buff.delete(start, end)
804 self.delete_sigid = self.buff.connect("delete-range", self._delete_range)
800# if selec:805# if selec:
801# print "deleted text is ##%s##" %self.buff.get_text(selec[0], selec[1])#(start, end)806# print "deleted text is ##%s##" %self.buff.get_text(selec[0], selec[1])#(start, end)
802# self.buff.disconnect(self.delete_sigid)807# self.buff.disconnect(self.delete_sigid)

Subscribers

People subscribed via source and target branches

to status/vote changes: