Merge lp://staging/~coalwater/loco-team-portal/fix-783544 into lp://staging/loco-team-portal

Proposed by Mohammad AbuShady
Status: Merged
Approved by: Chris Johnston
Approved revision: 451
Merged at revision: 454
Proposed branch: lp://staging/~coalwater/loco-team-portal/fix-783544
Merge into: lp://staging/loco-team-portal
Diff against target: 36 lines (+18/-0)
2 files modified
loco_directory/media/css/newstyle.css (+11/-0)
loco_directory/templates/teams/team_list.html (+7/-0)
To merge this branch: bzr merge lp://staging/~coalwater/loco-team-portal/fix-783544
Reviewer Review Type Date Requested Status
Chris Johnston Approve
Review via email: mp+67899@code.staging.launchpad.net

Commit message

Added a legend in the team listing that explains the difference between colored and grayed ubuntu logos.

Description of the change

To post a comment you must log in.
Revision history for this message
Chris Johnston (cjohnston) wrote :

There is some sort of border that is not in the area on the About page.

review: Needs Fixing
451. By Mohammad AbuShady

Added a legend in the team listing that explains the difference between colored and grayed ubuntu logos.

Revision history for this message
Chris Johnston (cjohnston) wrote :

Looks good.

review: Approve
Revision history for this message
Mohammad AbuShady (coalwater) wrote :

The problem was a mistake in the CSS box shadow which made the shadow go all 4 directions.

Revision history for this message
Mohammad AbuShady (coalwater) wrote :

do i need the update the branch with the 2 revisions that were added on the trunk? seems i'm 2 revisions late

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches