The proposal to merge lp:~jcsackett/launchpad/sharing-details-breadcrumb into lp:launchpad has been updated.
Description changed to: Summary ======= This branch adds breadcrumb info for the sharing details page. It also alters the url for the sharing details page, slightly, from +sharingdetails/$personname to +sharing/$personname, which is closer to the url structure we agreed upon previously. Ideally, it would be +sharing/details/person, but that doesn't seem entirely possible within zope's traversal structure. The previous url (+sharingdetails) was believed to be necessary at the time, that's not true. Of note, this doesn't complete the breadcrumb setup as seen in mockups[1], but as the managing disclosure view doesn't have any corresponding interface beyond the pillar, it's not added into the traversed_objects for the Hierarchy interface to convert into a breadcrumb. There are probably ways around this, but sorting them out was delaying the useful bit of code contained in this branch, i.e. a proper breadcrumb for the sharing details page. [1]: http://people.canonical.com/~curtis/disclosure/details.html?context=albert Preimp ====== None Implementation ============== This branch adds a simple IBreadCrumb adapter for the PillarPerson interface, providing the name of the person in the pillar person as the breadcrumb text. The zcml for the details page has been updated, as has the stepthrough decorator for the NavigationMixin to alter the url from +sharingdetails to +sharing. Tests ===== bin/test -vvct pillar_sharing QA == Go to a sharing details page via +sharing/$personname; it should open fine, verifying the url change. Additionally, the page should have a breadcrumb of $pillarname >> $personname Lint ==== Checking for conflicts and issues in changed files. Linting changed files: lib/lp/registry/browser/configure.zcml lib/lp/registry/browser/pillar.py For more details, see: https://code.launchpad.net/~jcsackett/launchpad/sharing-details-breadcrumb/+merge/100498 -- https://code.launchpad.net/~jcsackett/launchpad/sharing-details-breadcrumb/+merge/100498 Your team Launchpad code reviewers is requested to review the proposed merge of lp:~jcsackett/launchpad/sharing-details-breadcrumb into lp:launchpad. _______________________________________________ Mailing list: https://launchpad.net/~launchpad-reviewers Post to : [email protected] Unsubscribe : https://launchpad.net/~launchpad-reviewers More help : https://help.launchpad.net/ListHelp
