Ubuntu Community Web Themes

Merge lp://staging/~cjohnston/ubuntu-community-webthemes/top-nav into lp://staging/ubuntu-community-webthemes/light-django-theme

  1. top-nav
  2. Merge into light-django-theme
Proposed by Chris Johnston
Status: Merged
Approved by: Chris Johnston
Approved revision: 34
Merged at revision: 32
Proposed branch: lp://staging/~cjohnston/ubuntu-community-webthemes/top-nav
Merge into: lp://staging/ubuntu-community-webthemes/light-django-theme
Diff against target: 56 lines (+8/-10)
2 files modified
media/css/default.css (+4/-4)
templates/ubuntu_website_base.html (+4/-6)
To merge this branch: bzr merge lp://staging/~cjohnston/ubuntu-community-webthemes/top-nav
Reviewer Review Type Date Requested Status
Ronnie (community) Approve
Chris Johnston Needs Resubmitting
Review via email: mp+52791@code.staging.launchpad.net
To post a comment you must log in.
Revision history for this message
Chris Johnston (cjohnston) wrote :

After my meeting earlier this week, we are going to match the top-nav with the current global menu while research is being done on further changes.

lp://staging/~cjohnston/ubuntu-community-webthemes/top-nav updated
33. By Chris Johnston

fixes background color of top-nav

Revision history for this message
Ronnie (ronnie.vd.c) wrote :

The color of visited links is not right, and the margins differ:

Fix: http://paste.ubuntu.com/580222/

review: Needs Fixing
lp://staging/~cjohnston/ubuntu-community-webthemes/top-nav updated
34. By Chris Johnston

Made fixes suggested by Ronnie

Revision history for this message
Chris Johnston (cjohnston) :
review: Needs Resubmitting
Revision history for this message
Ronnie (ronnie.vd.c) wrote :

Good to go

review: Approve

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