[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