[Merge] lp:~evfool/update-manager/fix727069 into lp:update-manager
Barry Warsaw
barry at canonical.com
Thu Mar 31 17:45:48 UTC 2011
Review: Needs Fixing
Thanks for your contribution Robert. I have some concerns about this patch.
When looking at a classic desktop under Natty (which I sadly have too many of at the moment ;), There is no "Applications Place". Setting aside whether I know that there is a Places menu item under the Ubuntu menu, even under Places there is no entry for Applications. AFAICT, you still have to go to System->Administration to get Update Manager.
Under my one Unity desktop, there is an Applications box, which I guess is the "Applications Place" you're talking about. However, clicking on that doesn't really help, since Update Manager is no where to be found. I need to select the 52 other installed application, then scroll all the way down to actually find Update Manager.
I don't really know what the best way to word all these multiple locations and options are, and I agree that on Unity, "Administration Menu" is meaningless (though still appropriate on classic desktop). But still, I think the patch needs some work before it could be accepted.
I'll paste the above into the bug report so MPT has a chance to weigh in.
--
https://code.launchpad.net/~evfool/update-manager/fix727069/+merge/55662
Your team Ubuntu Core Development Team is requested to review the proposed merge of lp:~evfool/update-manager/fix727069 into lp:update-manager.
More information about the Ubuntu-reviews
mailing list