launchpad-reviewers team mailing list archive

["j.c.sackett" <jonathan.sackett@xxxxxxxxxxxxx>]

The proposal to merge lp:~jcsackett/launchpad/mergeproposal-status-icon-589584 into lp:launchpad has been updated.

Description changed to:

Summary
=======

On a merge proposal (for example, this one), there is both an edit icon link and a colored text section indicating the MP's status. Both are clickable and allow you to set the MP status (assuming you have permission to do so) but the mouse cursor only indicates that the edit icon is clickable.

This branch sets an anchor on the text as well, so it properly indicates that it's a control.

Implementation
==============

lib/lp/code/templates/branchmergeproposal-pagelet-summary.pt
------------------------------------------------------------

The span that contained the MP status text is now set only to display if the edit link is not enabled. If the link is enabled, an anchor with similar css class as the span is show instead. The span, new anchor, and the anchor around the edit icon are now inside a tal block defining the link variable, since all three elements require it.

Demo & QA
=========

Create an MP on launchpad.dev; when you hover the mouse over the status text, it should change to the cursor type indicating a clickable control (this is usually a hand). Clicking the link or the edit icon should allow you to set the status.

Lint
====

$ make lint

= Launchpad lint =

Checking for conflicts and issues in changed files.

Linting changed files:
  lib/lp/code/templates/branchmergeproposal-pagelet-summary.pt

For more details, see:
https://code.launchpad.net/~jcsackett/launchpad/mergeproposal-status-icon-589584/+merge/44531
-- 
https://code.launchpad.net/~jcsackett/launchpad/mergeproposal-status-icon-589584/+merge/44531
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~jcsackett/launchpad/mergeproposal-status-icon-589584 into lp:launchpad.