Ubuntu
tomboy package

Merge lp://staging/~er-abhinav-upadhyay/ubuntu/natty/tomboy/bugfix-757635 into lp://staging/ubuntu/natty/tomboy

  1. Natty (11.04)
  2. bugfix-757635
  3. Merge into natty
Proposed by Abhinav Upadhyay
Status: Merged
Merge reported by: Sebastien Bacher
Merged at revision: not available
Proposed branch: lp://staging/~er-abhinav-upadhyay/ubuntu/natty/tomboy/bugfix-757635
Merge into: lp://staging/ubuntu/natty/tomboy
Diff against target: 53 lines (+21/-0)
2 files modified
Tomboy/RecentChanges.cs (+11/-0)
debian/changelog (+10/-0)
To merge this branch: bzr merge lp://staging/~er-abhinav-upadhyay/ubuntu/natty/tomboy/bugfix-757635
Reviewer Review Type Date Requested Status
Ubuntu branches Pending
Review via email: mp+57282@code.staging.launchpad.net

Description of the change

Here is a brief outline of what I have done:

I have added an event handler to the Search box for the FocusInEvent. So now
when the user moves the focus to the Search combo box, the Delete option in the
menu bar is disabled.

This required another case to be handled: When the user clicks back on one of
the notes in the tree widget, the Delete option in the menubar (and in the
right click context menu) should be re-enabled, so I added the relevant code in
the ButtonPress event handler of the TreeView widget.

Comments are there in the patch to explain this :-).

I have also submitted the patch upstream and its due to be reviewed.

Thanks

To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

setting to merge since the fix has been uploaded it seems

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