Merge lp:~gary-lasker/oneconf/wire-app-action into lp:oneconf
Status: | Merged |
---|---|
Merged at revision: | 73 |
Proposed branch: | lp:~gary-lasker/oneconf/wire-app-action |
Merge into: | lp:oneconf |
Diff against target: |
11 lines (+1/-0) 1 file modified
oneconf/usc_plugin.py (+1/-0) |
To merge this branch: | bzr merge lp:~gary-lasker/oneconf/wire-app-action |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Didier Roche-Tolomelli | Approve | ||
Review via email: mp+31660@code.launchpad.net |
Description of the change
Hello didrocks!
I had some time late yesterday to take a quick look at OneConf and I have a fix for the Install/Remove buttons not working bug. This should be all you need (I tested and it appears to do the trick, please let me know if any problems).
Oh, and by the way, we will be moving to using gtk.Action for implementing the package actions rather than the current signals, but it's currently a little tricky because of our custom views/button implementations, etc. -- however, once we go to that this should be a little cleaner for you.
One thing about this fix, if we will be getting refresh_hosts and creating new instances of OneConfPane, we'll want to detach the "application-
I think you also had a problem with show/hide apps not working. I'll try to take a look at that one asap. Are there any other remaining issues you'd like me to look at?
Thanks!
Gary
Thanks a lot for the fix!
well, I think we can keep it this way for now. I don't think we will destroy new panes during USC lifecycle, so it's safe to keep it this way, isn't it?
(we just create a default set of pane as you saw in the for loop, and then, can create more on the fly and hide some). Will we get still some garbage-collector issue?
gtk.Action sounds good. I'm looking forward to it!
Great, you patch is working (at least, with one pane). Merging it now.
For your question, yeah, the show/hide apps is the remaining things that needs work!
Thanks again!