GTG

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

Proposed by Alan Gomes Alvino
Status: Rejected
Rejected by: Izidor Matušov
Proposed branch: lp://staging/~gtg-user/gtg/bugfix-493335
Merge into: lp://staging/~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://staging/~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.staging.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
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: