DockManager

Code review comment for lp://staging/~erigami/dockmanager/workrave-helper

  1. workrave-helper
  2. Merge into trunk
Revision history for this message
Rico Tzschichholz (ricotz) wrote :

@Erigami: Thanks for cleaning this up.

I haven't tested it since I don't use Workrave, but I think there a some problems.

What is the reason for these two classes WorkraveWatcher and WorkraveItem? Why not merging them together?

review: Needs Information

« Back to merge proposal